Re: Fixed crash in debugger with wxWidget 3.0

Поиск
Список
Период
Сортировка
От Dave Page
Тема Re: Fixed crash in debugger with wxWidget 3.0
Дата
Msg-id CA+OCxoxBr5HheSeG+=U0_2MXtys2xfkooe5FbQb-Ob0-Jk37+A@mail.gmail.com
обсуждение исходный текст
Ответ на Fixed crash in debugger with wxWidget 3.0  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Список pgadmin-hackers
Дерево обсуждения
Thanks - applied.


On Fri, May 23, 2014 at 9:05 AM, Akshay Joshi <akshay.joshi@enterprisedb.com> 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 




--
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

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Dave Page
Дата:
Сообщение: pgAdmin III commit: Fix a potential crash in the debugger.
Следующее
От: Dave Page
Дата:
Сообщение: pgAdmin III commit: Correctly initialise the datadir variable when usin