[Skiboot] [PATCH 3/3] hdata: Prevent NULL dereference on duplicate slot map info

Vasant Hegde hegdevasant at linux.vnet.ibm.com
Wed Mar 20 19:13:28 AEDT 2019


On 03/20/2019 12:03 PM, Stewart Smith wrote:
> Vasant Hegde <hegdevasant at linux.vnet.ibm.com> writes:
>> On 03/18/2019 11:22 AM, Stewart Smith wrote:
>>> Signed-off-by: Stewart Smith <stewart at linux.ibm.com>
>>
>> Patch looks good.
>>
>> I see few other places inside iohub.c where we don't check dt_new* return value.
>> May be fix them as well?
> 
> Huh, we probably should. afl-fuzz is pretty good at crashing things
> here, and I have mixed feelings about how much effort to put into it...
> 

May be we can add assert() inside dt_new* functions. If we fail to allocate memory
for device tree then we have bigger problem anyway.

-Vasant




More information about the Skiboot mailing list