Thanks - applied.
On Fri, May 23, 2014 at 9:05 AM, Akshay Joshi <[email protected] > wrote: > 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 <%2B91%2020-3058-9517> Mobile: +91 976-788-8246 > <%2B91%20976-788-8246>* > > > -- > Sent via pgadmin-hackers mailing list ([email protected]) > 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
