Skip to contentSkip to conversation

mac80211: fix printk() format string

PATCH In review Németh Márton 3 participants 3 messages Oct 28, 2007 mac80211

lore
Share & tools
lore

Németh Márton

author
#0 lore raw

PATCH 1/1 mac80211: fix printk() format string

From: Márton Németh <[email protected]>

Get rid of the following compiler warning message:

net/mac80211/ieee80211_sta.c: In function 'ieee80211_rx_mgmt_assoc_resp':
net/mac80211/ieee80211_sta.c:1187: warning: format '%d' expects type 'int', but argument 7 has type 'long unsigned int'

Signed-off-by Márton Németh <[email protected]>

---

net/mac80211/ieee80211_sta.c renamed from net/mac80211/ieee80211_sta.c.orig +1 -1
@@ -1182,7 +1182,7 @@ static void ieee80211_rx_mgmt_assoc_resp
 	aid = le16_to_cpu(mgmt->u.assoc_resp.aid);
 
 	printk(KERN_DEBUG "%s: RX %sssocResp from %s (capab=0x%x "
-	       "status=%d aid=%d)\n",
+	       "status=%d aid=%lu)\n",
 	       dev->name, reassoc ? "Rea" : "A", print_mac(mac, mgmt->sa),
 	       capab_info, status_code, aid & ~(BIT(15) | BIT(14)));
maintainer
#1 lore raw
Quoted 6 lines

Get rid of the following compiler warning message:

net/mac80211/ieee80211_sta.c: In function 'ieee80211_rx_mgmt_assoc_resp':
net/mac80211/ieee80211_sta.c:1187: warning: format '%d' expects type 'int', but argument 7 has type 'long unsigned int'

a different fix is already applied.

johannes

#2 lore raw
In reply to Németh Márton
Quoted 5 lines · N=E9meth M=E1rton wrote:

From: M=E1rton N=E9meth <[email protected]>
=20
Get rid of the following compiler warning message:
=20
net/mac80211/ieee80211_sta.c: In function 'ieee80211_rx_mgmt_assoc_re=

sp':

Quoted 1 line

net/mac80211/ieee80211_sta.c:1187: warning: format '%d' expects type =

'int', but argument 7 has type 'long unsigned int'

Quoted 4 lines

=20

Signed-off-by M=E1rton N=E9meth <[email protected]>

---
--- linux-2.6.24-rc1/net/mac80211/ieee80211_sta.c.orig 2007-10-24 05:=

50:57.000000000 +0200

Quoted 1 line

+++ linux-2.6.24-rc1/net/mac80211/ieee80211_sta.c 2007-10-28 11:20:23=

=2E000000000 +0100

Review

Quoted 8 lines
(quoted diff) +0 -0
@@ -1182,7 +1182,7 @@ static void ieee80211_rx_mgmt_assoc_resp
 	aid =3D le16_to_cpu(mgmt->u.assoc_resp.aid);

=20

(quoted diff) +1 -1
 	printk(KERN_DEBUG "%s: RX %sssocResp from %s (capab=3D0x%x "
-	       "status=3D%d aid=3D%d)\n",
+	       "status=3D%d aid=3D%lu)\n",
 	       dev->name, reassoc ? "Rea" : "A", print_mac(mac, mgmt->sa),
 	       capab_info, status_code, aid & ~(BIT(15) | BIT(14)));

=46ix already went upstream...

-
To unsubscribe from this list: send the line "unsubscribe linux-wireles= s" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html