ConvertStringSecurityDescriptorToSecurityDescriptor - more tests (pass)

[Paul Bryan Roberts]

A second attempt that, I hope, takes into account feedback from James 
Hawkins.

Original correction to ParseStringAclToAcl() to set missing error code 
stands.

Gratuitous setting of invalid parameter withdrawn.

Replaced with check for NULL on StringSecurityDescriptor parameter in 
both A and W forms of ConvertStringSecurityDescriptorToSecurityDescriptor.

New checks backed up by additional test cases.

Two instances of "highly superfluous" TRACE removed as suggested by JH,

Issue - unable to add test cases that show

if (GetVersion() & 0x80000000)
    SetLastError(ERROR_CALL_NOT_IMPLEMENTED);

should precede checks for NULL parameters.  What to do ?  Drop patch 
altogether ?



-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-ConvertStringSecurityDescriptorToSecurityDescriptor.patch
Type: text/x-diff
Size: 0 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-patches/attachments/20081013/0a078ebc/attachment.patch