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 < [email protected]> wrote: > Reviewed-by: Lionel Landwerlin <[email protected]> > > > On 25/09/16 16:06, Nayan Deshmukh wrote: > >> CovID: 1373370 >> v2: fix indentation >> >> Signed-off-by: Nayan Deshmukh <[email protected]> >> --- >> 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); >> >> > > >
_______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
