<div dir="ltr">Hi Lionel,<div><br></div><div>Thanks for the review. Can you please push the patch.</div><div><br></div><div>Regards,</div><div>Nayan.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Sep 25, 2016 at 10:18 PM, Lionel Landwerlin <span dir="ltr"><<a href="mailto:lionel.g.landwerlin@intel.com" target="_blank">lionel.g.landwerlin@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Reviewed-by: Lionel Landwerlin <<a href="mailto:lionel.g.landwerlin@intel.com" target="_blank">lionel.g.landwerlin@intel.com</a><wbr>><div class="HOEnZb"><div class="h5"><br>
<br>
On 25/09/16 16:06, Nayan Deshmukh wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
CovID: 1373370<br>
v2: fix indentation<br>
<br>
Signed-off-by: Nayan Deshmukh <<a href="mailto:nayan26deshmukh@gmail.com" target="_blank">nayan26deshmukh@gmail.com</a>><br>
---<br>
src/intel/tools/disasm.c | 4 +++-<br>
1 file changed, 3 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/src/intel/tools/disasm.c b/src/intel/tools/disasm.c<br>
index e527615..4ac7b90 100644<br>
--- a/src/intel/tools/disasm.c<br>
+++ b/src/intel/tools/disasm.c<br>
@@ -101,8 +101,10 @@ gen_disasm_create(int pciid)<br>
if (gd == NULL)<br>
return NULL;<br>
- if (!gen_get_device_info(pciid, &gd->devinfo))<br>
+ if (!gen_get_device_info(pciid, &gd->devinfo)) {<br>
+ free(gd);<br>
return NULL;<br>
+ }<br>
brw_init_compaction_tables(&g<wbr>d->devinfo);<br>
<br>
</blockquote>
<br>
<br>
</div></div></blockquote></div><br></div>