46bd1dc2ffb42eebe1c715b1f00d9f11d8e2ee0e
[oweals/openwrt.git] /
1 From: Felix Fietkau <nbd@nbd.name>
2 Date: Tue, 12 Feb 2019 14:22:43 +0100
3 Subject: [PATCH v2] wpa_supplicant: fix race condition in mesh mpm new peer
4  handling
5
6 When wpa_supplicant receives another new peer event before the first one
7 has been processed, it tries to add a station to the driver a second time
8 (which fails) and then tears down the station entry until another event
9 comes in.
10 Fix this by only adding a station to the driver if it didn't exist already.
11
12 Signed-off-by: Felix Fietkau <nbd@nbd.name>
13 ---
14
15 --- a/wpa_supplicant/mesh_mpm.c
16 +++ b/wpa_supplicant/mesh_mpm.c
17 @@ -685,11 +685,12 @@ static struct sta_info * mesh_mpm_add_pe
18         }
19  
20         sta = ap_get_sta(data, addr);
21 -       if (!sta) {
22 -               sta = ap_sta_add(data, addr);
23 -               if (!sta)
24 -                       return NULL;
25 -       }
26 +       if (sta)
27 +               return NULL;
28 +
29 +       sta = ap_sta_add(data, addr);
30 +       if (!sta)
31 +               return NULL;
32  
33         /* Set WMM by default since Mesh STAs are QoS STAs */
34         sta->flags |= WLAN_STA_WMM;