[Bug 18071] Core Media Player - installer "Cannot read access control list"

http://bugs.winehq.org/show_bug.cgi?id=18071

Louis Lenders  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #3 from Louis Lenders   2009-04-16 02:06:03 ---
(In reply to comment #2)
> (In reply to comment #1)
> > (In reply to comment #0)
> > > First problem: on install, get this error. It's been reported on wine-users
> > > (http://www.winehq.org/pipermail/wine-users/2008-March/031475.html) but I
> > > haven't seen a bug for it. Looks like it may be related to NTFS ACL's.

Looks like it's not happy with what GetNamedSecurityInfo returns if descriptor
=0. Hack below gets rid of the errormessage. You'd have to test i guess what
happens on windows if descriptor=0.

diff --git a/dlls/advapi32/security.c b/dlls/advapi32/security.c
index f8715e0..2a3c29c 100644
--- a/dlls/advapi32/security.c
+++ b/dlls/advapi32/security.c
@@ -5082,8 +5082,8 @@ DWORD WINAPI GetNamedSecurityInfoW( LPWSTR name,
SE_OBJECT_TYPE type,
     TRACE( "%s %d %d %p %p %p %p %p\n", debugstr_w(name), type, info, owner,
            group, dacl, sacl, descriptor );

-    if (!name || !descriptor) return ERROR_INVALID_PARAMETER;
-
+    if (!name ) return ERROR_INVALID_PARAMETER;
+    if (!descriptor) descriptor = HeapAlloc(GetProcessHeap(), 0,
sizeof(PSECURITY_DESCRIPTOR));
     needed = sizeof(SECURITY_DESCRIPTOR_RELATIVE);
     if (info & OWNER_SECURITY_INFORMATION)
         needed += sizeof(sidWorld);