By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.
Bug 1322 - ksim crashes when configuration dialog launched.
Summary: ksim crashes when configuration dialog launched.
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: tdeutils (show other bugs)
Version: R14.0.0 [Trinity]
Hardware: amd64 Linux
: P5 normal
Assignee: Timothy Pearson
URL:
Depends on:
Blocks:
 
Reported: 2012-11-15 00:27 CST by Alexander Golubev (Fat-Zer)
Modified: 2013-03-02 22:57 CST (History)
2 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:

Attachments
backtrace (2.85 KB, text/plain)
2012-11-15 00:40 CST, Alexander Golubev (Fat-Zer) 		Details
fix patch (1.25 KB, patch)
2012-11-15 00:53 CST, Alexander Golubev (Fat-Zer) 		Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Golubev (Fat-Zer) 2012-11-15 00:27:10 CST 
See the discussion on BUG 1310 starting on comment 12: http://bugs.pearsoncomputing.net/show_bug.cgi?id=1310#c12
Comment 1 Alexander Golubev (Fat-Zer) 2012-11-15 00:40:02 CST 
Created attachment 980 [details]
backtrace

Backtrace can vary from system to system and even from build to build.
Comment 2 Alexander Golubev (Fat-Zer) 2012-11-15 00:53:41 CST 
Created attachment 981 [details]
fix patch

the patch fixes the crash for me.

Generally the crash should appear only when KSim::Plugin::fileName() returns TQString(), but for some reason even if d is not NULL d->name is copied to the stack and the link to the on stack copy is returned. This copy is corrupted by later calls. And leads to SIGSEGV.

The by-value return prevents that.
Comment 3 Darrell 2012-11-15 14:03:38 CST 
The patch works for me with Slackware 13.1 32-bit. I'm not a C++ guru therefore Tim or Slavek will have to review and approve before pushing to GIT. :)

Good work!
Comment 4 Darrell 2013-03-02 22:57:09 CST 
Patch in attachment 981 [details] pushed to GIT in commit 4ed913cf. Resolved.