Bug 372837 – dev-util/codeblocks crashes when a wxChartCtrl is added with wxSmith

• Product: Gentoo Linux
• Status: RESOLVED FIXED

[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) 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) 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) 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) 2012-11-02 06:00:55 UTC]

Fixed in 10.05-r1.  Sorry about the wait.