* [PATCH] nl80211: fix a bug caused by a missing asterisk
@ 2007-08-11 8:21 Johannes Berg
0 siblings, 0 replies; only message in thread
From: Johannes Berg @ 2007-08-11 8:21 UTC (permalink / raw)
To: John Linville; +Cc: linux-wireless
I really dislike double pointers. Oh well.
Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
---
Just hit it yesterday, oopses the kernel under RNTL so you're screwed.
Not nice.
--- wireless-dev.orig/net/wireless/nl80211.c 2007-08-11 10:19:57.999563528 +0200
+++ wireless-dev/net/wireless/nl80211.c 2007-08-11 10:19:58.559563528 +0200
@@ -55,7 +55,7 @@ static int get_drv_dev_by_info_ifindex(s
ifindex = nla_get_u32(info->attrs[NL80211_ATTR_IFINDEX]);
*dev = dev_get_by_index(ifindex);
- if (!dev)
+ if (!*dev)
return -ENODEV;
*drv = cfg80211_get_dev_from_ifindex(ifindex);
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2007-08-11 8:18 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-08-11 8:21 [PATCH] nl80211: fix a bug caused by a missing asterisk Johannes Berg
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox