From: | John Obaterspok <john(dot)obaterspok(at)gmail(dot)com> |
---|---|
To: | Sanket Mehta <sanket(dot)mehta(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [PATCH] Fix crash when disabling auto commit |
Date: | 2015-06-14 08:30:16 |
Message-ID: | CAOscVdJZmXqoQErk5YqRLp3RT8zEHqmYMhZ1obdwFeE-yT8OPg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hello Sanket,
Just enter "rollback" and exec
The
while(wxIsalpha(query.GetChar(wordlen)))
wordlen++;
As the 'k' in rollback is a char it also tries the next character (worklen
= 8) which causes out of bounds check.
-- john
2015-06-12 13:25 GMT+02:00 Sanket Mehta <sanket(dot)mehta(at)enterprisedb(dot)com>:
> Hi John,
>
> I have tried to reproduce the scenario but not able to reproduce the crash
> in my system.
> can you please provide your steps which causes crash on your machine?
>
> Regards,
> Sanket Mehta
> Sr Software engineer
> Enterprisedb
>
> On Fri, Jun 12, 2015 at 11:50 AM, Sanket Mehta <
> sanket(dot)mehta(at)enterprisedb(dot)com> wrote:
>
>> Hi,
>>
>> I am looking into the same and few other issues in code and will send the
>> patch soon.
>>
>>
>> Regards,
>> Sanket Mehta
>> Sr Software engineer
>> Enterprisedb
>>
>> On Fri, Jun 12, 2015 at 1:49 AM, John Obaterspok <
>> john(dot)obaterspok(at)gmail(dot)com> wrote:
>>
>>> Fix crash when string has only alphas (like 'rollback')
>>>
>>> diff --git a/pgadmin/frm/frmQuery.cpp b/pgadmin/frm/frmQuery.cpp
>>> index b5a2f56..110bbc7 100644
>>> --- a/pgadmin/frm/frmQuery.cpp
>>> +++ b/pgadmin/frm/frmQuery.cpp
>>> @@ -2522,7 +2522,7 @@ bool frmQuery::isBeginNotRequired(wxString query)
>>> /*
>>> * Check word length (since "beginx" is not "begin").
>>> */
>>> - while(wxIsalpha(query.GetChar(wordlen)))
>>> + while(wordlen < query.Length() && wxIsalpha(query.GetChar(wordlen)))
>>> wordlen++;
>>>
>>> /*
>>>
>>
>>
>
From | Date | Subject | |
---|---|---|---|
Next Message | Sanket Mehta | 2015-06-14 09:10:55 | Re: [PATCH] Fix crash when disabling auto commit |
Previous Message | Jacek Wielemborek | 2015-06-13 15:22:10 | SSH error messages not shown |