msi: Don't leak memory on failure.

Signed-off-by: Pierre Schweitzer <pierre@reactos.org> Signed-off-by: Hans Leidekker <hans@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

msi: Don't leak memory on failure.
a6ad46dc · Pierre Schweitzer · Alexandre Julliard · f621f410 · a6ad46dc
Commit a6ad46dc authored Oct 21, 2017 by Pierre Schweitzer Committed by Alexandre Julliard Oct 23, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 0 deletions

package.c dlls/msi/package.c +6 -0

No files found.
--- a/dlls/msi/package.c
+++ b/dlls/msi/package.c
@@ -2286,7 +2286,10 @@ static MSIRECORD *msi_get_property_row( MSIDATABASE *db, LPCWSTR name )

        row = MSI_CreateRecord(1);
        if (!row)
+        {
+            msi_free(buffer);
            return NULL;
+        }
        MSI_RecordSetStringW(row, 1, buffer);
        msi_free(buffer);
        return row;
@@ -2301,7 +2304,10 @@ static MSIRECORD *msi_get_property_row( MSIDATABASE *db, LPCWSTR name )

        row = MSI_CreateRecord(1);
        if (!row)
+        {
+            msi_free(buffer);
            return NULL;
+        }
        MSI_RecordSetStringW(row, 1, buffer);
        msi_free(buffer);
        return row;