[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