Got it. Thanks for your patient explanation! 2016-09-06 1:14 GMT+08:00 Emil Velikov <[email protected]>:
> On 5 September 2016 at 17:21, Weng Chuanbo <[email protected]> wrote: > > > [****Chuanbo****] Could you explain " we want these NULL checks split out > > and ported to older loader " more detailed? > > > > And what's older loaders? What's newer dri modules? > > > > From my understanding, the only path in mesa code invokes mapImage is > > gbm_dri_bo_map. So if apply my patch code above, no NULL deref will > happen. > > > For this particular exercise we can say that there are three loaders - > src/{gbm,glx,egl}, the interface is in include/ and everything else > can be considered* as dri module/drivers. > > So here you want to split out the null checks into a separate patches, > which precede the functional (EGLimage) change. This way the former > can be applied (as they are bugfixes) for older versions of mesa. > > -Emil >
_______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
