From: Helmut Raiger <helmut.raiger@hale.at>
Date: Mon, 22 Aug 2011 00:17:17 +0000 (+0000)
Subject: net/eth.c: throw BUG for eth_get_dev_by_name(NULL)
X-Git-Tag: v2011.09-rc1~28
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=7e7f903fcd7883d3646223e3cb30b50a7dfe6248;p=oweals%2Fu-boot.git

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 dbd1e2dbae..5911b1c786 100644
--- a/net/eth.c
+++ b/net/eth.c
@@ -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;