diff options
author | Helmut Raiger <helmut.raiger@hale.at> | 2011-08-22 00:17:17 (GMT) |
---|---|---|
committer | Wolfgang Denk <wd@denx.de> | 2011-09-09 21:58:47 (GMT) |
commit | 7e7f903fcd7883d3646223e3cb30b50a7dfe6248 (patch) | |
tree | c8c1b3aa1dfec9534d1b9f90aa858fdd54222ba5 | |
parent | 24e1664472deef23abb3246da8573ddefc0c7971 (diff) | |
download | u-boot-fsl-qoriq-7e7f903fcd7883d3646223e3cb30b50a7dfe6248.tar.xz |
net/eth.c: throw BUG for eth_get_dev_by_name(NULL)
eth_get_dev_by_name() is not safe to use for devname being NULL
as it uses strcmp. This patch makes it fail with a BUG().
Signed-off-by: Helmut Raiger <helmut.raiger@hale.at>
Acked-by: Mike Frysinger <vapier@gentoo.org>
-rw-r--r-- | net/eth.c | 2 |
1 files changed, 2 insertions, 0 deletions
@@ -108,6 +108,8 @@ struct eth_device *eth_get_dev_by_name(const char *devname) { struct eth_device *dev, *target_dev; + BUG_ON(devname == NULL); + if (!eth_devices) return NULL; |