Re: Bug report: OIDs, roles and Windows (signedness issue?) - Mailing list pgadmin-hackers

From Ian Lawrence Barwick
Subject Re: Bug report: OIDs, roles and Windows (signedness issue?)
Date
Msg-id CAB8KJ=gjLLUo3GC6ecxtWmvAWURjB1idLKiwU2qj_cKpdzx_TA@mail.gmail.com
Whole thread Raw
In response to Re: Bug report: OIDs, roles and Windows (signedness issue?)  (Ian Lawrence Barwick <barwick@gmail.com>)
Responses Re: Bug report: OIDs, roles and Windows (signedness issue?)
List pgadmin-hackers
2013/11/22 Ian Lawrence Barwick <barwick@gmail.com>:
> 2013/11/22 Ian Lawrence Barwick <barwick@gmail.com>:
>
>>>
>>> BTW other objects (tables, schemas etc.) with OIDs greater than 256^4/2 - 1
>>> don't seem to be affected by this.
>>
>> It's been a while since I've looked at the pgAdmin source, but some wild
>> grepping shows that in:
>>
>>   pgadmin/schema/pgRole.cpp
>>
>> the role OID is set with:
>>
>>  role->iSetOid(roles->GetLong(wxT("oid")));
>>
>> whereas almost everywhere else it's set with GetOid(), e.g.
>>
>>  table->iSetOid(tables->GetOid(wxT("oid")));
>>
>> which may or may not have something to do with the issue.
>
> If that is the problem, then here's a patch. Untested as I don't have a Windows
> build environment available, though I might be able to cobble one together
> next week.

Just wondering - has anyone been able to look into this? Unfortunately I still
don't have a Windows build environment available to test.


Regards

Ian Barwick


pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: Re: [PATCH] Added a menu option to disable code folding
Next
From: Ashesh Vashi
Date:
Subject: Re: Bug report: OIDs, roles and Windows (signedness issue?)