]> www.infradead.org Git - users/dwmw2/linux.git/commitdiff
mac80211: validate extended element ID is present
authorJohannes Berg <johannes.berg@intel.com>
Sat, 11 Dec 2021 19:10:24 +0000 (20:10 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 22 Dec 2021 08:19:03 +0000 (09:19 +0100)
commit 768c0b19b50665e337c96858aa2b7928d6dcf756 upstream.

Before attempting to parse an extended element, verify that
the extended element ID is present.

Fixes: 41cbb0f5a295 ("mac80211: add support for HE")
Reported-by: syzbot+59bdff68edce82e393b6@syzkaller.appspotmail.com
Link: https://lore.kernel.org/r/20211211201023.f30a1b128c07.I5cacc176da94ba316877c6e10fe3ceec8b4dbd7d@changeid
Cc: stable@vger.kernel.org
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
net/mac80211/util.c

index 7fa9871b1db9fe325dea47bc81a15418ea1cbb81..9c1a20ca6344544178c074662e575a75695b0217 100644 (file)
@@ -1102,6 +1102,8 @@ u32 ieee802_11_parse_elems_crc(const u8 *start, size_t len, bool action,
                                elems->max_idle_period_ie = (void *)pos;
                        break;
                case WLAN_EID_EXTENSION:
+                       if (!elen)
+                               break;
                        if (pos[0] == WLAN_EID_EXT_HE_MU_EDCA &&
                            elen >= (sizeof(*elems->mu_edca_param_set) + 1)) {
                                elems->mu_edca_param_set = (void *)&pos[1];