| commit | 7e7f903fcd7883d3646223e3cb30b50a7dfe6248 | [log] [tgz] |
|---|---|---|
| author | Helmut Raiger <helmut.raiger@hale.at> | Mon Aug 22 00:17:17 2011 +0000 |
| committer | Wolfgang Denk <wd@denx.de> | Fri Sep 09 23:58:47 2011 +0200 |
| tree | c8c1b3aa1dfec9534d1b9f90aa858fdd54222ba5 | |
| parent | 24e1664472deef23abb3246da8573ddefc0c7971 [diff] [blame] |
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>
diff --git a/net/eth.c b/net/eth.c index dbd1e2d..5911b1c 100644 --- a/net/eth.c +++ b/net/eth.c
@@ -108,6 +108,8 @@ { struct eth_device *dev, *target_dev; + BUG_ON(devname == NULL); + if (!eth_devices) return NULL;