Thread: Fixed crash in debugger with wxWidget 3.0
Hi Dave
--

During testing of pgAdmin3 with wxWidget 3.0, I have found one crash in debugger. Cause of the crash is "m_progress" variable is not instantiated before the function call "wxStatusBar::SetFieldsCount(fields);" at line no 49 in ctlProgressStatusBar.cpp.
Function call "wxStatusBar::SetFieldsCount(fields)" internally calls "ctlProgressStatusBar::SetStatusWidths" which internally calls "ctlProgressStatusBar::OnSize" and that time "m_progress" variable is not instantiated, but used that's why application got crashed.
I have just moved up the call to instantiate the "m_progress" variable. Attached is the patch file to fix the issue. Please review it and if it looks good to you then can you please commit the code.
Akshay Joshi
Principal Software Engineer

Phone: +91 20-3058-9517
Mobile: +91 976-788-8246
Mobile: +91 976-788-8246
Attachment
Thanks - applied.
On Fri, May 23, 2014 at 9:05 AM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi DaveDuring testing of pgAdmin3 with wxWidget 3.0, I have found one crash in debugger. Cause of the crash is "m_progress" variable is not instantiated before the function call "wxStatusBar::SetFieldsCount(fields);" at line no 49 in ctlProgressStatusBar.cpp.Function call "wxStatusBar::SetFieldsCount(fields)" internally calls "ctlProgressStatusBar::SetStatusWidths" which internally calls "ctlProgressStatusBar::OnSize" and that time "m_progress" variable is not instantiated, but used that's why application got crashed.I have just moved up the call to instantiate the "m_progress" variable. Attached is the patch file to fix the issue. Please review it and if it looks good to you then can you please commit the code.
--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company