Re: [PATCH v12 0/3] MR414: fltmgr.sys: Implement FltBuildDefaultSecurityDescriptor

[Zebediah Figura (＠zfigura)]

Zebediah Figura (@zfigura) commented about dlls/ntoskrnl.exe/tests/driver.c:
> +    }
> +    ok(sd != NULL, "Failed to return descriptor\n");
> +
> +    status = RtlGetGroupSecurityDescriptor(sd, &group, &isdefault);
> +    ok(status == STATUS_SUCCESS, "got %#lx\n", status);
> +    ok(group == NULL, "group isn't NULL\n");
> +
> +    status = RtlGetOwnerSecurityDescriptor(sd, &owner, &isdefault);
> +    ok(status == STATUS_SUCCESS, "got %#lx\n", status);
> +    ok(owner == NULL, "owner isn't NULL\n");
> +
> +    status = RtlGetDaclSecurityDescriptor(sd, &present, &acl, &isdefault);
> +    ok(status == STATUS_SUCCESS, "got %#lx\n", status);
> +    ok(acl != NULL, "acl is NULL\n");
> +    ok(acl->AceCount == 2, "got %d\n", acl->AceCount);
> +
> Should we test "isdefault" here as well?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/414#note_65054