From: | Sanket Mehta <sanket(dot)mehta(at)enterprisedb(dot)com> |
---|---|
To: | John Obaterspok <john(dot)obaterspok(at)gmail(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [PATCH] Fix crash when disabling auto commit |
Date: | 2015-06-12 11:25:14 |
Message-ID: | CA+yw=mMH0fTn13RAmCRQSBx4Evv1L5sd5_HTTy7hqCmvYk3WdA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
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 | Jacek Wielemborek | 2015-06-13 15:20:15 | valgrind complains a lot when trying to establish an SSH tunnel while giving wrong key |
Previous Message | Sanket Mehta | 2015-06-12 06:20:46 | Re: [PATCH] Fix crash when disabling auto commit |