lk/linux-kernel-test
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

zd1211rw: Fix multicast filtering.

Browse Source 
If multicast parameter (as returned by zd_op_prepare_multicast) has
changed, no bit in changed_flags is set. To handle this situation, we do
not return if changed_flags is 0. If we do so, we will have some issue
with IPv6 which uses multicast for link layer address resolution.

Signed-off-by: Benoit Papillault <benoit.papillault@free.fr>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
This commit is contained in:
Benoit Papillault
2010-01-03 10:20:01 +01:00
committed by John W. Linville
parent 8a9ac160e8
commit 7de3c5dc0a
1 changed files with 7 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
drivers/net/wireless/zd1211rw/zd_mac.c
View File

@@ -987,12 +987,13 @@ static void zd_op_configure_filter(struct ieee80211_hw *hw,
changed_flags &= SUPPORTED_FIF_FLAGS; changed_flags &= SUPPORTED_FIF_FLAGS;
*new_flags &= SUPPORTED_FIF_FLAGS; *new_flags &= SUPPORTED_FIF_FLAGS;
/* changed_flags is always populated but this driver /*
* doesn't support all FIF flags so its possible we don't * If multicast parameter (as returned by zd_op_prepare_multicast)
* need to do anything */ * has changed, no bit in changed_flags is set. To handle this
if (!changed_flags) * situation, we do not return if changed_flags is 0. If we do so,
return; * we will have some issue with IPv6 which uses multicast for link
* layer address resolution.
*/
if (*new_flags & (FIF_PROMISC_IN_BSS | FIF_ALLMULTI)) if (*new_flags & (FIF_PROMISC_IN_BSS | FIF_ALLMULTI))
zd_mc_add_all(&hash); zd_mc_add_all(&hash);