Thread: Fixed pgAdmin crash issue

Fixed pgAdmin crash issue

From
Neel Patel
Date:
Hi Dave,

Please find the patch file in attachment for below described issue in linux.

Steps to reproduce:-

1. Open pgAdmin and go to database node of larger size.
2. Right Click on that database and go to "Backup"
3. Give the Filename = "test" and Format = "Custom".
4. Click on "Backup" button and immediately click "Cancel" button to cancel the 
     operation.
5. pgAdmin will crash.


Findings:-

As we are creating the new process with "wxExecute" to create the Backup and user immediately press on "Cancel" then we should  first kill the process which is created and proper way to delete the process is once we get the callback of "EndProcess" not in Abort() method so changed it accordingly.

Kindly let me know for any modification.

Thanks,
Neel Patel


Attachment

Re: Fixed pgAdmin crash issue

From
Dave Page
Date:
Thanks - applied to the master and 1.18 branches.

On Tue, Sep 10, 2013 at 1:09 PM, Neel Patel <neel.patel@enterprisedb.com> wrote:
> Hi Dave,
>
> Please find the patch file in attachment for below described issue in linux.
>
> Steps to reproduce:-
>
> 1. Open pgAdmin and go to database node of larger size.
> 2. Right Click on that database and go to "Backup"
> 3. Give the Filename = "test" and Format = "Custom".
> 4. Click on "Backup" button and immediately click "Cancel" button to cancel
> the
>      operation.
> 5. pgAdmin will crash.
>
>
> Findings:-
>
> As we are creating the new process with "wxExecute" to create the Backup and
> user immediately press on "Cancel" then we should  first kill the process
> which is created and proper way to delete the process is once we get the
> callback of "EndProcess" not in Abort() method so changed it accordingly.
>
> Kindly let me know for any modification.
>
> Thanks,
> Neel Patel
>
>



--
Dave Page
Chief Architect, Tools & Installers
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake