Re: advapi32: Added test and fix for GetFileSecurity

[James Hawkins]

On 4/16/07, Nigel Nia-Chiang Liang  wrote:
> Hi,
>
> Added a simple test case and fix for GetFileSecurity with NULL
> pSecurityDescriptor parameter as suggested by James Hawkins.

SetLastError(NO_ERROR);
+
     result = pGetFileSecurityA(
directory,OWNER_SECURITY_INFORMATION,buffer,0x40,&outSize);

Please don't change the whitespace.  The SetLastError call is a part
of that test block, so there's no need for an empty line.

+    result = pGetFileSecurityA(
directory,OWNER_SECURITY_INFORMATION,NULL,0x40,&outSize);
+    ok(result, "GetFileSecurityA should not fail for NULL
pSecurityDescriptor parameter\n");

What is the value of outSize in this case?  You need to set it to some
value before the call and check if it's changed after the call.