Hi Emil,
I have split out patches and sent out the patches.
Please review them. Thanks!
2016-09-06 11:03 GMT+08:00 Weng Chuanbo <chuanbo.w...@gmail.com>:
> Got it. Thanks for your patient explanation!
>
> 2016-09-06 1:14 GMT+08:00 Emil Velikov <emil.l.veli...@gmail.com>:
>
>> On 5 September 2016 at 17:21, Weng Chuanbo <chuanbo.w...@gmail.com>
>> 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
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
