Re: GetTokenInformation() and FreeSid() at port/exec.c

Andrew Chernow <ac@esilo.com>

From: Andrew Chernow <ac@esilo.com>
To: Magnus Hagander <magnus@hagander.net>
Cc: TAKATSUKA Haruka <harukat@sraoss.co.jp>, pgsql-bugs@postgresql.org
Date: 2009-06-23T15:07:06Z
Lists: pgsql-bugs
> 
> How about something like this? I switched to using LocalAlloc() in all
> places to be consistent, instead of mixing heap and local. (Though per
> doc, LocalAlloc is actually a wrapper for HeapAlloc in win32).

Our patches crossed.  Although, in my patch I left the allocation scheme 
alone since I wasn't sure if someone wanted that way.  I'd suggest 
malloc and free if your going to change it.  The only time I use an MS 
allocater is when a win32 api function specifically states it must be used.

-- 
Andrew Chernow
eSilo, LLC
every bit counts
http://www.esilo.com/