| From: | Andrew Chernow <ac(at)esilo(dot)com> |
|---|---|
| To: | TAKATSUKA Haruka <harukat(at)sraoss(dot)co(dot)jp> |
| Cc: | pgsql-bugs(at)postgresql(dot)org |
| Subject: | Re: GetTokenInformation() and FreeSid() at port/exec.c |
| Date: | 2009-06-23 15:01:42 |
| Message-ID: | 4A40EE56.6000703@esilo.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-bugs |
>
> DWORD len = GetLengthSid(pTokenUser->User.Sid)
> *ppSidUser = (PSID) HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, len);
> CopySid(len, *ppSidUser, pTokenUser->User.Sid);
>
I attached a patch for this. Although, I did not use CopySid. Instead,
I changed GetUserSid to GetTokenUser. AddUserToDacl() is the only
function making use of GetUserSid(), so this change won't break
anything. The benefit to this approach over my first suggestion is that
it avoids an unneeded HeapAlloc(sid), CopySid(sid) ... and its cleaner.
--
Andrew Chernow
eSilo, LLC
every bit counts
http://www.esilo.com/
| Attachment | Content-Type | Size |
|---|---|---|
| freesid.patch | text/x-patch | 7.0 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Armin Jenewein | 2009-06-23 15:06:15 | BUG #4875: /etc/init.d/postgresql status shows postmaster is stopped when executed as user |
| Previous Message | Magnus Hagander | 2009-06-23 14:58:01 | Re: GetTokenInformation() and FreeSid() at port/exec.c |