Re: [PATCH 2/2] msi: Don't leak memory on failure.

[Nikolay Sivov]

On 21.10.2017 12:50, Pierre Schweitzer wrote:
> row = MSI_CreateRecord(1);
>          if (!row)
> +        {
> +            msi_free(buffer);
>              return NULL;
> +        }
>          MSI_RecordSetStringW(row, 1, buffer);
>          msi_free(buffer);
>          return row;

You can replace this with:

---
row = MSI_CreateRecord(1);
if (row)
    MSI_RecordSetStringW(row, 1, buffer);
msi_free(buffer);
return row;
---