<div dir="ltr">Hi Emil,<div>    I have split out patches and sent out the patches.</div><div>    Please review them. Thanks!</div></div><div class="gmail_extra"><br><div class="gmail_quote">2016-09-06 11:03 GMT+08:00 Weng Chuanbo <span dir="ltr"><<a href="mailto:chuanbo.weng@gmail.com" target="_blank">chuanbo.weng@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra">Got it. Thanks for your patient explanation!</div><div><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">2016-09-06 1:14 GMT+08:00 Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On 5 September 2016 at 17:21, Weng Chuanbo <<a href="mailto:chuanbo.weng@gmail.com" target="_blank">chuanbo.weng@gmail.com</a>> wrote:<br>
<br>
> [****Chuanbo****] Could you explain " we want these NULL checks split out<br>
> and ported to older loader " more detailed?<br>
><br>
> And what's older loaders? What's newer dri modules?<br>
><br>
> From my understanding, the only path in mesa code invokes mapImage is<br>
> gbm_dri_bo_map. So if apply my patch code above, no NULL deref will happen.<br>
><br>
</span>For this particular exercise we can say that there are three loaders -<br>
src/{gbm,glx,egl}, the interface is in include/ and everything else<br>
can be considered* as dri module/drivers.<br>
<br>
So here you want to split out the null checks into a separate patches,<br>
which precede the functional (EGLimage) change. This way the former<br>
can be applied (as they are bugfixes) for older versions of mesa.<br>
<span><font color="#888888"><br>
-Emil<br>
</font></span></blockquote></div><br></div></div></div></div>
</blockquote></div><br></div>