diff options
| -rw-r--r-- | meta/recipes-connectivity/wpa-supplicant/wpa-supplicant/0001-AP-WMM-Fix-integer-underflow-in-WMM-Action-frame-par.patch | 45 |
1 files changed, 45 insertions, 0 deletions
diff --git a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant/0001-AP-WMM-Fix-integer-underflow-in-WMM-Action-frame-par.patch b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant/0001-AP-WMM-Fix-integer-underflow-in-WMM-Action-frame-par.patch new file mode 100644 index 0000000000..882674fe5b --- /dev/null +++ b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant/0001-AP-WMM-Fix-integer-underflow-in-WMM-Action-frame-par.patch | |||
| @@ -0,0 +1,45 @@ | |||
| 1 | Upstream-Status: Backport | ||
| 2 | |||
| 3 | Signed-off-by: Fan Xin <fan.xin@jp.fujitsu.com> | ||
| 4 | |||
| 5 | From ef566a4d4f74022e1fdb0a2addfe81e6de9f4aae Mon Sep 17 00:00:00 2001 | ||
| 6 | From: Jouni Malinen <j@w1.fi> | ||
| 7 | Date: Wed, 29 Apr 2015 02:21:53 +0300 | ||
| 8 | Subject: [PATCH] AP WMM: Fix integer underflow in WMM Action frame parser | ||
| 9 | |||
| 10 | The length of the WMM Action frame was not properly validated and the | ||
| 11 | length of the information elements (int left) could end up being | ||
| 12 | negative. This would result in reading significantly past the stack | ||
| 13 | buffer while parsing the IEs in ieee802_11_parse_elems() and while doing | ||
| 14 | so, resulting in segmentation fault. | ||
| 15 | |||
| 16 | This can result in an invalid frame being used for a denial of service | ||
| 17 | attack (hostapd process killed) against an AP with a driver that uses | ||
| 18 | hostapd for management frame processing (e.g., all mac80211-based | ||
| 19 | drivers). | ||
| 20 | |||
| 21 | Thanks to Kostya Kortchinsky of Google security team for discovering and | ||
| 22 | reporting this issue. | ||
| 23 | |||
| 24 | Signed-off-by: Jouni Malinen <j@w1.fi> | ||
| 25 | --- | ||
| 26 | src/ap/wmm.c | 3 +++ | ||
| 27 | 1 file changed, 3 insertions(+) | ||
| 28 | |||
| 29 | diff --git a/src/ap/wmm.c b/src/ap/wmm.c | ||
| 30 | index 6d4177c..314e244 100644 | ||
| 31 | --- a/src/ap/wmm.c | ||
| 32 | +++ b/src/ap/wmm.c | ||
| 33 | @@ -274,6 +274,9 @@ void hostapd_wmm_action(struct hostapd_data *hapd, | ||
| 34 | return; | ||
| 35 | } | ||
| 36 | |||
| 37 | + if (left < 0) | ||
| 38 | + return; /* not a valid WMM Action frame */ | ||
| 39 | + | ||
| 40 | /* extract the tspec info element */ | ||
| 41 | if (ieee802_11_parse_elems(pos, left, &elems, 1) == ParseFailed) { | ||
| 42 | hostapd_logger(hapd, mgmt->sa, HOSTAPD_MODULE_IEEE80211, | ||
| 43 | -- | ||
| 44 | 1.9.1 | ||
| 45 | |||
