Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1624900 > unrolled thread

[PATCH] nl80211: Fix enum type of variable in nl80211_put_sta_rate()

Back to article view | Back to linux.kernel

Contents

  [PATCH] nl80211: Fix enum type of variable in nl80211_put_sta_rate() Matthias Kaehlcke <mka@chromium.org> - 2017-04-18 01:10 +0200
    Re: [PATCH] nl80211: Fix enum type of variable in  nl80211_put_sta_rate() Johannes Berg <johannes@sipsolutions.net> - 2017-04-18 11:10 +0200

#1624900 — [PATCH] nl80211: Fix enum type of variable in nl80211_put_sta_rate()

rate_flg is of type 'enum nl80211_attrs', however it is assigned with
'enum nl80211_rate_info' values. Change the type of rate_flg accordingly.

Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
---
 net/wireless/nl80211.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/net/wireless/nl80211.c b/net/wireless/nl80211.c
index 2312dc2ffdb9..9af21a21ea6b 100644
--- a/net/wireless/nl80211.c
+++ b/net/wireless/nl80211.c
@@ -4151,7 +4151,7 @@ static bool nl80211_put_sta_rate(struct sk_buff *msg, struct rate_info *info,
 	struct nlattr *rate;
 	u32 bitrate;
 	u16 bitrate_compat;
-	enum nl80211_attrs rate_flg;
+	enum nl80211_rate_info rate_flg;
 
 	rate = nla_nest_start(msg, attr);
 	if (!rate)
-- 
2.12.2.762.g0e3151a226-goog

[toc] | [next] | [standalone]

#1625165 — Re: [PATCH] nl80211: Fix enum type of variable in nl80211_put_sta_rate()

On Mon, 2017-04-17 at 15:59 -0700, Matthias Kaehlcke wrote:
> rate_flg is of type 'enum nl80211_attrs', however it is assigned with
> 'enum nl80211_rate_info' values. Change the type of rate_flg
> accordingly.

Applied this, and the other two patches you had (IBSS enum and array-
bounds)

johannes

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web