Re: advapi32/security: Free descriptor if it isn't returned from GetSecurityInfo()

[James Hawkins]

On Fri, Jan 8, 2010 at 10:08 AM, Nikolay Sivov  wrote:
> Free descriptor if it isn't returned from GetSecurityInfo() (Valgrind).

This is very wrong :-)  You should read the comment above the test
that causes the leak.  Dan has notice to suppress this valgrind error,
so it shouldn't show up in the logs after he adds the suppression.

Thanks,
James Hawkins

advapi32/security: Free descriptor if it isn't returned from GetSecurityInfo()

[Nikolay Sivov]

On 1/8/2010 21:52, James Hawkins wrote:
> On Fri, Jan 8, 2010 at 10:08 AM, Nikolay Sivov  wrote:
> > Free descriptor if it isn't returned from GetSecurityInfo() (Valgrind).
>
> This is very wrong :-)  You should read the comment above the test
> that causes the leak.  Dan has notice to suppress this valgrind error,
> so it shouldn't show up in the logs after he adds the suppression.
>
> Thanks,
> James Hawkins

Ah. ok. Sorry for that. Looks like I'm using outdated suppression file.

Thanks.