Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 372837 - dev-util/codeblocks crashes when a wxChartCtrl is added with wxSmith
Summary: dev-util/codeblocks crashes when a wxChartCtrl is added with wxSmith
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo wxWidgets project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 419567
  Show dependency tree
 
Reported: 2011-06-24 14:35 UTC by Gert Wollny
Modified: 2012-11-02 06:00 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
Proposed fix as sent upstream (wxwidgets-eclass-inherit-base.patch,400 bytes, patch)
2011-06-24 14:36 UTC, Gert Wollny
Details | Diff
Proposed fix as sent upstream (codeblocks-10.05-release-wxCharWindow-fix.patch,1.14 KB, patch)
2011-06-24 14:37 UTC, Gert Wollny
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gert Wollny 2011-06-24 14:35:58 UTC
When one tries to add a wxChartCtrl to a form, codeblocks crashed with a sigsegv. This is because in the chartwindow.cpp a pointer sizes variable is set by calling GetSizes() which may return a NULL pointer. However, this is not checked before the pointer is de-referenced. 

The bug has been reported upstream as 
https://developer.berlios.de/bugs/?func=detailbug&bug_id=18182&group_id=5358
with a patch provided as 
https://developer.berlios.de/patch/?func=detailpatch&patch_id=3177&group_id=5358


Reproducible: Always

Steps to Reproduce:
1. Create a new Project using wxSmith as GUI builder
2. add a wxChartCtrl to the newly created Dialog/Form 
Actual Results:  
Program crashes with a sigsegv

Expected Results:  
control is added to the form
Comment 1 Gert Wollny 2011-06-24 14:36:22 UTC
Created attachment 278013 [details, diff]
Proposed fix as sent upstream
Comment 2 Gert Wollny 2011-06-24 14:37:45 UTC
Created attachment 278015 [details, diff]
Proposed fix as sent upstream

Sorry, hit okay too fast.
Comment 3 Ryan Hill (RETIRED) gentoo-dev 2011-07-11 23:49:22 UTC
What version?
Comment 4 Gert Wollny 2011-07-12 08:34:27 UTC
10.05, btw, the bug is now fixed upstream in HEAD of their wxSmith branch.
Comment 5 Marco Ziebell 2012-06-10 09:57:38 UTC
Report is obsolete in dev-util/codeblocks-10.05, so it should be closed
Comment 6 Gert Wollny 2012-06-10 12:04:59 UTC
Care to elaborate? v10.05 is the latest unstable version in the portage tree, what made the bug obsolete? If there was some change to the ebuild that would change the behaviour with respect to the bug, shouldn't there be a version -r1?
Comment 7 Marco Ziebell 2012-06-15 13:29:47 UTC
My Mistake, forget what I wrote
Comment 8 Ryan Hill (RETIRED) gentoo-dev 2012-07-02 02:42:35 UTC
What was the actual fix that was applied to upstream?  I'm having trouble locating it in their tree.
Comment 9 Gert Wollny 2012-07-02 16:14:27 UTC
According to the patch tracker 

https://developer.berlios.de/patch/?func=detailpatch&patch_id=3177&group_id=5358

it is the patch I gave above, their link is:  

https://developer.berlios.de/patch/download.php?id=3177
Comment 10 Ryan Hill (RETIRED) gentoo-dev 2012-09-16 02:41:15 UTC
The "applied partially" and your comment about not knowing if it is the correct fix make me want to see the actual commit.
Comment 11 Ryan Hill (RETIRED) gentoo-dev 2012-11-02 06:00:55 UTC
Fixed in 10.05-r1.  Sorry about the wait.