server: Fix file_set_sd to handle NULL DACLs.

These mean that the file should have full permissions for everyone.

server: Fix file_set_sd to handle NULL DACLs.
8d6093e6 · Rob Shearman · Alexandre Julliard · 1c7891f8 · 8d6093e6
Commit 8d6093e6 authored Oct 20, 2007 by Rob Shearman Committed by Alexandre Julliard Oct 22, 2007
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 2 deletions

file.c server/file.c +3 -2

No files found.
--- a/server/file.c
+++ b/server/file.c
@@ -435,9 +435,10 @@ static int file_set_sd( struct object *obj, const struct security_descriptor *sd

    if (set_info & DACL_SECURITY_INFORMATION)
    {
-        if (sd->control & SE_DACL_PRESENT)
+        int present;
+        const ACL *dacl = sd_get_dacl( sd, &present );
+        if (present && dacl)
        {
-            const ACL *dacl = (const ACL *)((char *)sd + sd->owner_len + sd->group_len + sd->sacl_len);
            const ACE_HEADER *ace = (const ACE_HEADER *)(dacl + 1);
            ULONG i;
            for (i = 0; i < dacl->AceCount; i++)