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>

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>
Helmut Raiger · Wolfgang Denk
1 parent 24e1664472
Showing 1 changed file with 2 additions and 0 deletions Side-by-side Diff
net/eth.c
@@ -108,6 +108,8 @@
 {
 	struct eth_device *dev, *target_dev;
  
+	BUG_ON(devname == NULL);
+
 	if (!eth_devices)
 		return NULL;
...	...	@@ -108,6 +108,8 @@
108	108	{
109	109	struct eth_device dev, target_dev;
110	110
	111	+ BUG_ON(devname == NULL);
	112	+
111	113	if (!eth_devices)
112	114	return NULL;
113	115