[Mesa-dev] [PATCH v2] intel/i965: fix resource leak

Nayan Deshmukh nayan26deshmukh at gmail.com
Sun Sep 25 18:22:42 UTC 2016


Hi Lionel,

Thanks for the review. Can you please push the patch.

Regards,
Nayan.

On Sun, Sep 25, 2016 at 10:18 PM, Lionel Landwerlin <
lionel.g.landwerlin at intel.com> wrote:

> Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
>
>
> On 25/09/16 16:06, Nayan Deshmukh wrote:
>
>> CovID: 1373370
>> v2: fix indentation
>>
>> Signed-off-by: Nayan Deshmukh <nayan26deshmukh at gmail.com>
>> ---
>>   src/intel/tools/disasm.c | 4 +++-
>>   1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/src/intel/tools/disasm.c b/src/intel/tools/disasm.c
>> index e527615..4ac7b90 100644
>> --- a/src/intel/tools/disasm.c
>> +++ b/src/intel/tools/disasm.c
>> @@ -101,8 +101,10 @@ gen_disasm_create(int pciid)
>>      if (gd == NULL)
>>         return NULL;
>>   -   if (!gen_get_device_info(pciid, &gd->devinfo))
>> +   if (!gen_get_device_info(pciid, &gd->devinfo)) {
>> +      free(gd);
>>         return NULL;
>> +   }
>>        brw_init_compaction_tables(&gd->devinfo);
>>
>>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20160925/46c6194b/attachment.html>


More information about the mesa-dev mailing list