Search Linux Wireless

Re: [PATCH] cfg80211: Check for channel HT capabilities in an IBSS

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Le 04/05/2010 20:14, Luis R. Rodriguez a écrit :
On Mon, May 3, 2010 at 11:47 PM, Benoit Papillault
<benoit.papillault@xxxxxxx>  wrote:
When configuring an HT IBSS, we need to check if the specified channel
is capable of ht40+, ht40- or ht20.

Signed-off-by: Benoit Papillault<benoit.papillault@xxxxxxx>
---
  net/wireless/nl80211.c |    8 ++++----
  1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/net/wireless/nl80211.c b/net/wireless/nl80211.c
index 8da9823..0a82623 100644
--- a/net/wireless/nl80211.c
+++ b/net/wireless/nl80211.c
@@ -3858,11 +3858,11 @@ static int nl80211_join_ibss(struct sk_buff *skb, struct genl_info *info)
                ibss.ie_len = nla_len(info->attrs[NL80211_ATTR_IE]);
        }

-       ibss.channel = ieee80211_get_channel(wiphy,
-               nla_get_u32(info->attrs[NL80211_ATTR_WIPHY_FREQ]));
+       ibss.channel = rdev_freq_to_chan(rdev,
+               nla_get_u32(info->attrs[NL80211_ATTR_WIPHY_FREQ]),
+               channel_type);
        if (!ibss.channel ||
-           ibss.channel->flags&  IEEE80211_CHAN_NO_IBSS ||
-           ibss.channel->flags&  IEEE80211_CHAN_DISABLED) {
+           ibss.channel->flags&  IEEE80211_CHAN_NO_IBSS) {
                err = -EINVAL;

Why is the disabled channel check being removed here?

   Luis


Hi Luis,

It's because the check is already done in rdev_freq_to_chan.

Regards,
Benoit
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux