[Skiboot] [PATCH 3/3] hdata: Prevent NULL dereference on duplicate slot map info
stewart at linux.ibm.com
Mon Mar 25 10:28:51 AEDT 2019
Vasant Hegde <hegdevasant at linux.vnet.ibm.com> writes:
> 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.
Although that would be it in the PCI hotplug code path, and being unable
to process the new device shouldn't be a fatal error.
OPAL Architect, IBM.
More information about the Skiboot