mac80211: Fix addition of mesh configuration element
authorIlan peer <ilan.peer@intel.com>
Mon, 26 Dec 2016 16:17:36 +0000 (18:17 +0200)
committerSasha Levin <alexander.levin@microsoft.com>
Wed, 17 Jan 2018 17:55:32 +0000 (12:55 -0500)
[ Upstream commit 57629915d568c522ac1422df7bba4bee5b5c7a7c ]

The code was setting the capabilities byte to zero,
after it was already properly set previously. Fix it.

The bug was found while debugging hwsim mesh tests failures
that happened since the commit mentioned below.

Fixes: 76f43b4c0a93 ("mac80211: Remove invalid flag operations in mesh TSF synchronization")
Signed-off-by: Ilan Peer <ilan.peer@intel.com>
Reviewed-by: Masashi Honma <masashi.honma@gmail.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Sasha Levin <alexander.levin@microsoft.com>
net/mac80211/mesh.c

index 48257f17688f8ee55600d972395e76db8103947f..2ce35082a33524fd605250a0c78331000c77193e 100644 (file)
@@ -289,8 +289,6 @@ int mesh_add_meshconf_ie(struct ieee80211_sub_if_data *sdata,
        /* Mesh PS mode. See IEEE802.11-2012 8.4.2.100.8 */
        *pos |= ifmsh->ps_peers_deep_sleep ?
                        IEEE80211_MESHCONF_CAPAB_POWER_SAVE_LEVEL : 0x00;
-       *pos++ = 0x00;
-
        return 0;
 }