[weiden] 31161: Fix a memory leak in AccpLookupSidByName

Author: weiden
Date: Tue Dec 11 21:56:55 2007
New Revision: 31161

URL: http://svn.reactos.org/svn/reactos?rev=31161&view=rev
Log:
Fix a memory leak in AccpLookupSidByName

Modified:
    trunk/reactos/dll/win32/ntmarta/ntmarta.c

Modified: trunk/reactos/dll/win32/ntmarta/ntmarta.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/ntmarta/ntmarta.c...
==============================================================================
--- trunk/reactos/dll/win32/ntmarta/ntmarta.c (original)
+++ trunk/reactos/dll/win32/ntmarta/ntmarta.c Tue Dec 11 21:56:55 2007
@@ -366,7 +366,10 @@
         return LsaNtStatusToWinError(Status);

if (TranslatedSid->Use == SidTypeUnknown || TranslatedSid->Use == SidTypeInvalid)
-        return LsaNtStatusToWinError(STATUS_NONE_MAPPED); /* FIXME- what error code? */
+    {
+        Ret = LsaNtStatusToWinError(STATUS_NONE_MAPPED); /* FIXME- what error code? */
+        goto Cleanup;
+    }

SidLen = GetLengthSid(TranslatedSid->Sid);
     ASSERT(SidLen != 0);
@@ -387,6 +390,7 @@
     else
         Ret = ERROR_NOT_ENOUGH_MEMORY;

+Cleanup:
     LsaFreeMemory(ReferencedDomains);
     LsaFreeMemory(TranslatedSid);