From: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
---|---|
To: | Magnus Hagander <magnus(at)hagander(dot)net> |
Cc: | Dave Page <dpage(at)pgadmin(dot)org>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: pgadmin crash on refresh |
Date: | 2011-01-22 00:19:11 |
Message-ID: | 4D3A227F.1080804@lelarge.info |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Le 21/01/2011 19:36, Magnus Hagander a écrit :
> On Fri, Jan 21, 2011 at 13:35, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>> On Fri, Jan 21, 2011 at 12:20 PM, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
>>> On Wed, Jan 19, 2011 at 14:55, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>> On Wed, Jan 19, 2011 at 12:18 PM, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
>>>>> Opened database->catalogs->pg_catalogs->tables. Right click + refresh = crash.
>>>>>
>>>>> Don't have time to dig in right now myself, but here's a backtrace if
>>>>> somebody wants to get started, or perhaps has a clue what it could
>>>>> be...
>>>>>
>>>>> this is with current git master tip, debugging enabled.
>>>>
>>>> Hmm, can't reproduce it on Windows. It does look an awful lot like
>>>
>>> Oh, additional point: you need to actually *expand* the tables node,
>>> or it doesn't crash. Did you by any chance not do that?
>>
>> I did do that.
>>
>>>> this though: http://git.postgresql.org/gitweb?p=pgadmin3.git;a=commit;h=9ec6b29fe3fbd9cc196d5e91b55f3697f02bcad8
>>>
>>> Nope. I reverted that patch (well, due to the style patch i couldn't,
>>> but i commented out that one line it seems to add) and it still
>>> crashes for me.
>>
>> There's little chance that that patch would cause (or fix) the problem
>> you see - but a similar patch in a different place might be the fix.
>>
>
> I've created ticket 299 for this for now, so we don't loose track of it.
>
Thanks Magnus.
--
Guillaume
http://www.postgresql.fr
http://dalibo.com
From | Date | Subject | |
---|---|---|---|
Next Message | Guillaume Lelarge | 2011-01-22 00:33:20 | Re: Patch to support SSL certificate connections |
Previous Message | Guillaume Lelarge | 2011-01-22 00:18:19 | Re: Number of selected characters. |