Thread (2 messages) flat view 2 messages, 1 author, 2011-08-30
STALE5484d

[PATCH 4/8] Null dereference fix

From: Pavel Raiskup <hidden>
Date: 2011-08-16 09:47:23
Subsystem: the rest · Maintainer: Linus Torvalds

Variable ifindex dereferenced on line 242 before null check on
line 249.
---
  network/common.c |    4 +++-
  1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/network/common.c b/network/common.c
index 6da9f0a..f8cd721 100644
--- a/network/common.c
+++ b/network/common.c
@@ -239,13 +239,15 @@ int bnep_if_down(const char *devname)

  int bnep_add_to_bridge(const char *devname, const char *bridge)
  {
-	int ifindex = if_nametoindex(devname);
+	int ifindex;
  	struct ifreq ifr;
  	int sk, err;

  	if (!devname || !bridge)
  		return -EINVAL;

+	ifindex = if_nametoindex(devname);
+
  	sk = socket(AF_INET, SOCK_STREAM, 0);
  	if (sk < 0)
  		return -1;
-- 
1.7.4.4
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help