Fixed memory leak from LocalGroupIsMember function

Author: ms-rs

NetAdmin.xs

@@ -1715,21 +1715,29 @@ XS(XS_NT__NetAdmin_LocalGroupIsMember)
 	AllocWideName((char*)SvPV(ST(1),n_a), lpwGroup);
 	do {
 	    PLOCALGROUP_MEMBERS_INFO_0 pwzMembersInfo;
+	    pwzMembersInfo = NULL;
 	    lastError = NetLocalGroupGetMembers(lpwServer, lpwGroup, 0,
 						(LPBYTE*)&pwzMembersInfo,
 		    				PREFLEN, &entriesRead,
 						&totalEntries, &resumeHandle);
-	    if (lastError != 0 && lastError != ERROR_MORE_DATA)
+	    if (lastError != 0 && lastError != ERROR_MORE_DATA) {
+	        if (pwzMembersInfo != NULL) {
+		    NetApiBufferFree(pwzMembersInfo);
+	        }
 		break;
-	    for (index = 0; index < entriesRead; ++index)
-		if (EqualSid(pSid, pwzMembersInfo[index].lgrmi0_sid) != 0){
-		    bReturn = TRUE;
-		    break;
-		}
+	    }
+
+            if (bReturn == FALSE) {
+                for (index = 0; index < entriesRead; ++index) {
+                    if (EqualSid(pSid, pwzMembersInfo[index].lgrmi0_sid) != 0){
+		      bReturn = TRUE;
+		      break;
+		    }
+	        }
+	    }
 
 	    NetApiBufferFree(pwzMembersInfo);
-	} while(bReturn == FALSE &&
-		(lastError == ERROR_MORE_DATA ||  resumeHandle != 0));
+	} while(lastError == ERROR_MORE_DATA || resumeHandle != 0);
 
 	free(pSid);
 	FreeWideName(lpwServer);