[U-Boot] [PATCH 11/12] sh_eth: Fix return value of sh_eth_initialize()
Mike Frysinger
vapier at gentoo.org
Sun Jan 24 11:57:52 CET 2010
On Sunday 24 January 2010 05:07:59 Matthias Kaehlcke wrote:
> --- a/drivers/net/sh_eth.c
> +++ b/drivers/net/sh_eth.c
> eth = (struct sh_eth_dev *)malloc(sizeof(struct sh_eth_dev));
> if (!eth) {
> printf(SHETHER_NAME ": %s: malloc failed\n", __func__);
> - ret = -ENOMEM;
> + ret = 0;
> goto err;
> }
>
> dev = (struct eth_device *)malloc(sizeof(struct eth_device));
> if (!dev) {
> printf(SHETHER_NAME ": %s: malloc failed\n", __func__);
> - ret = -ENOMEM;
> + ret = 0;
> goto err;
> }
these are error paths and thus should be -1
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20100124/23f3a6c3/attachment.pgp
More information about the U-Boot
mailing list