[PATCH] ntdll: Fix some memory leaks (Valgrind).

Sven Baars sbaars at codeweavers.com
Sun Sep 20 11:25:27 CDT 2020


Signed-off-by: Sven Baars <sbaars at codeweavers.com>
---
 dlls/ntdll/unix/security.c | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/dlls/ntdll/unix/security.c b/dlls/ntdll/unix/security.c
index daecc5e0591..09a1d787c09 100644
--- a/dlls/ntdll/unix/security.c
+++ b/dlls/ntdll/unix/security.c
@@ -779,8 +779,16 @@ NTSTATUS WINAPI NtSetSecurityObject( HANDLE handle, SECURITY_INFORMATION info, P
     InitializeObjectAttributes( &attr, NULL, 0, 0, descr );
     if ((status = alloc_object_attributes( &attr, &objattr, &len ))) return status;
     sd = (struct security_descriptor *)(objattr + 1);
-    if (info & OWNER_SECURITY_INFORMATION && !sd->owner_len) return STATUS_INVALID_SECURITY_DESCR;
-    if (info & GROUP_SECURITY_INFORMATION && !sd->group_len) return STATUS_INVALID_SECURITY_DESCR;
+    if (info & OWNER_SECURITY_INFORMATION && !sd->owner_len)
+    {
+        free( objattr );
+        return STATUS_INVALID_SECURITY_DESCR;
+    }
+    if (info & GROUP_SECURITY_INFORMATION && !sd->group_len)
+    {
+        free( objattr );
+        return STATUS_INVALID_SECURITY_DESCR;
+    }
     if (info & (SACL_SECURITY_INFORMATION | LABEL_SECURITY_INFORMATION)) sd->control |= SE_SACL_PRESENT;
     if (info & DACL_SECURITY_INFORMATION) sd->control |= SE_DACL_PRESENT;
 
-- 
2.25.1




More information about the wine-devel mailing list