Postgres Pro
Downloads
Home   >   mailing lists

Re: Patch for Race Condition. - Mailing list pgadmin-hackers

From Ashesh Vashi
Subject Re: Patch for Race Condition.
Date
Msg-id CAG7mmoxSj6Xb6PZiVqsQeMCnONP6somamM2merenuuw+OJfwFQ@mail.gmail.com
Whole thread Raw
In response to Re: Patch for Race Condition.  (Ashesh Vashi <ashesh.vashi@enterprisedb.com>)
List pgadmin-hackers
Tree view
Sorry - that was wrong file (it was from my working branching, hence - wrong number shown for the line number.) 
diff --git a/pgadmin/frm/events.cpp b/pgadmin/frm/events.cpp
index a73fa6a..276a6de 100644
--- a/pgadmin/frm/events.cpp
+++ b/pgadmin/frm/events.cpp
@@ -745,6 +745,10 @@ void frmMain::OnSelRightClick(wxTreeEvent &event)
                s_currentObjectMutex.Lock();
                currentObject = browser->GetObject(item);
        }
+       else
+       {
+               s_currentObjectMutex.Lock();
+       }
 
        if (currentObject)
                doPopup(browser, event.GetPoint(), currentObject);


On Thu, Oct 3, 2013 at 10:29 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
I think - the fix may not work on every platform.
We do unlock the mutex, which is not locked always.

diff --git a/pgadmin/frm/events.cpp b/pgadmin/frm/events.cpp
index 359e372..1bc4b04 100644
--- a/pgadmin/frm/events.cpp
+++ b/pgadmin/frm/events.cpp
@@ -791,6 +791,10 @@ void frmMain::OnSelRightClick(wxTreeEvent &event)
                s_currentObjectMutex.Lock();
                currentObject = browser->GetObject(item);
        }
+       else
+       {
+               s_currentObjectMutex.Lock();
+       }
 
        if (currentObject)
                doPopup(browser, event.GetPoint(), currentObject);


On Fri, Sep 27, 2013 at 8:22 PM, Dave Page <dpage@pgadmin.org> wrote:
Thanks - applied with minor changes to the comments and mutex name.


On Fri, Sep 27, 2013 at 8:51 AM, Dinesh Kumar <dinesh.kumar@enterprisedb.com> wrote:
Hi Dave,

Sorry, it's my bad. The patch is build on another branch.

Please find the attached patch which has been built on the pgAdmin master branch.

Thanks in advance.


Dinesh

-- 
Dinesh Kumar
Software Engineer
Skype ID: dinesh.kumar432
www.enterprisedb.com

Follow us on Twitter
@EnterpriseDB 

Visit EnterpriseDB for tutorials, webinars, whitepapers and more


On Thu, Sep 26, 2013 at 7:54 PM, Dave Page <dpage@pgadmin.org> wrote:



On Tue, Sep 24, 2013 at 12:37 PM, Dinesh Kumar <dinesh.kumar@enterprisedb.com> wrote:
Hi Dave/Team,

While working on one of the assigned task, i have been noticed the race condition on object "currentObject", which is concurrently updating by "onSelRightClick" and "execSelChange" methods. Due to this, sometimes, i am getting "n" Node context menu on "m" node.

Ex:- When i right click on "server node", it is showing the "database node" popup menu options.

I have been attaching the patch along with this request, and will be awaiting for your inputs.

This doesn't apply - can you check the patch is correct please? 

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company




--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



--
--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company

 

http://www.linkedin.com/in/asheshvashi




--
--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company

 

http://www.linkedin.com/in/asheshvashi

pgadmin-hackers by date:

Previous
From: Ashesh Vashi
Date:
Subject: Re: Patch for Race Condition.
Next
From: Dave Page
Date:
Subject: Re: Patch for Race Condition.