[Skiboot] [PATCH] mambo: Add NULL checks and error messages

Andrew Donnellan andrew.donnellan at au1.ibm.com
Mon Jul 11 14:17:35 AEST 2016


On 11/07/16 14:13, Alastair D'Silva wrote:
> This looks like it can result in a free attompt on a NULL pointer.

Which is fine - free(NULL) is defined as a no-op, and in this case if 
only one of bl or bdi failed to allocate, we want to free the other one 
which was successfully allocated.

-- 
Andrew Donnellan              OzLabs, ADL Canberra
andrew.donnellan at au1.ibm.com  IBM Australia Limited



More information about the Skiboot mailing list