]> git.ipfire.org Git - thirdparty/hostap.git/commit
WMM: Do not modify input TSPEC buffer during processing
authorJouni Malinen <j@w1.fi>
Sat, 23 Nov 2019 15:39:55 +0000 (17:39 +0200)
committerJouni Malinen <j@w1.fi>
Sat, 23 Nov 2019 15:39:55 +0000 (17:39 +0200)
commit5b50265e133d2ce111211e69d584ba71bf0551e4
tree98f1985ae225d8bc1c5ba767271eeb3a3b630a00
parente8ccbef25123e66ab600fef0e1f9da6ded96cddb
WMM: Do not modify input TSPEC buffer during processing

The WMM TSPEC processor used the input buffer for processing the request
and building the response. This was fine for the FT case, but for the
WMM Action frame case, the input buffer is marked const, so it should
not really be modified. This modification could not really cause any
noticeable harm, but it can result in error reports from fuzzing and
potentially even from some static analyzers.

Fix this by marking the input arguments const more consistently (the
parsed IE was able to drop the const) and copy the const input data to a
temporary buffer for processing and modification instead of allowing the
input data to be modified.

Credit to OSS-Fuzz: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=19050
Signed-off-by: Jouni Malinen <j@w1.fi>
src/ap/wmm.c