[U-Boot] Bombs away! Was: [PATCH 02/10] net: rtl8139: Add initialized eth_device structure
Reinhard Meyer
u-boot at emk-elektronik.de
Thu Oct 14 08:31:47 CEST 2010
Dear All,
> dev = (struct eth_device *)malloc(sizeof *dev);
> + memset(dev, 0, sizeof(*dev));
>
> sprintf (dev->name, "RTL8139#%d", card_number);
Apparently its quite common NOT to check malloc()'s possible
NULL return value... At least most NET drivers don't seem to...
Maybe another RFC to avoid duplicating code:
malloc_cleared_panic() to allocate and clear memory for a
really required structure and put a proper panic message if that
fails. Assuming that continuing u-boot once a driver cannot even
be initialized is futile, that would save even more code in each
driver.
Reinhard
More information about the U-Boot
mailing list