Skip site navigation (1) Skip section navigation (2)

Re: pgadmin3-1.6.1 - core dumped - Fixed

From: Dhanaraj M <Dhanaraj(dot)M(at)Sun(dot)COM>
To: Dhanaraj M <Dhanaraj(dot)M(at)Sun(dot)COM>
Cc: Dave Page <dpage(at)postgresql(dot)org>, Zdenek(dot)Kotala(at)Sun(dot)COM, pgadmin-hackers(at)postgresql(dot)org
Subject: Re: pgadmin3-1.6.1 - core dumped - Fixed
Date: 2006-12-14 10:07:06
Message-ID: 4581224A.1000403@sun.com (view raw or flat)
Thread:
Lists: pgadmin-hackers
Hi Dave

I tested in sparc/x-86 and this fix works well.
However, I like to know whether there will be regressions due to this fix.
Can you please review my fix?

Thanks
Dhanaraj

Dhanaraj M wrote:
> Hi Dave
>
> I dont know whether the problem is with pgAdmin or not.
> But I have a fix for this.
>
> File name:-  src/frm/frmQuery.cpp
>
> void frmQuery::OnChangeConnection(wxCommandEvent &ev)
> {
>     unsigned int sel=cbConnection->GetCurrentSelection();
>
> +  if(!IsVisible())
> +       return;
>
>     if (sel == cbConnection->GetCount()-1)
>     {
>         // new Connection
>         dlgSelectConnection dlg(this, mainForm);
>         int rc=dlg.Go(conn, cbConnection);
> ---------------------------------------------------------------------------- 
>
> After closing the query tool, this event is called.
> So I am checking whether the window is destroyed already or not.
> Is it the right fix?
> I am testing in different machines(sparc/x-86) and report you soon.
> Please comment on this?
>
> Thanks
> Dhanaraj
>
>
>
>
> Dave Page wrote:
>
>> Dhanaraj M wrote:
>>
>>> Sorry for the wrong work around.
>>> But it crahes in this step...
>>>     if (s->GetConnected() && s->GetConnection()->GetHost() == 
>>> conn->GetHost() && s->GetConnection()->GetPort() == conn->GetPort())^M
>>>                      I think 'conn' is destroyed already... or some 
>>> other problem.
>>>
>>> Any idea??
>>
>>
>> Sorry - just building wx now....
>>
>> Try testing conn at the beginning of the if statement:
>>
>> if (conn && s->GetConnected() && s->GetConnection()->GetHost() ==
>>
>> I'm still not sure how you're getting into that code when you close 
>> the form though...
>>
>> Regards, Dave.
>>
>> ---------------------------(end of broadcast)---------------------------
>> TIP 4: Have you searched our list archives?
>>
>>               http://archives.postgresql.org
>
>
>
> ---------------------------(end of broadcast)---------------------------
> TIP 5: don't forget to increase your free space map settings


In response to

pgadmin-hackers by date

Next:From: Zdenek KotalaDate: 2006-12-14 13:11:47
Subject: Re: pgadmin3-1.6.1 - core dumped - Work around
Previous:From: svnDate: 2006-12-14 09:33:57
Subject: SVN Commit by dpage: r5790 - trunk/pgadmin3

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group