DGNum/hostapd
6
0
Fork 0
Code Issues Pull requests 1 Projects Releases Packages Wiki Activity Actions

DPP: Fix @CONF-OBJ-SEP@ parsing for multiple configs

Browse source 
The first call to dpp_configuration_parse_helper() was supposed to use
the separately prepared tmp string with only the first configuration
entry, but it ended up using the full string that included both
configuration entries. This could result in the first configObject
getting a mix of parameters from both entries.

Fix the parsing to use only the text before the @CONF-OBJ-SEP@ separator
for the first entry.

Fixes: 7eb06a3369 ("DPP2: Allow multiple Config Objects to be build on Configurator")
Signed-off-by: Jouni Malinen <quic_jouni@quicinc.com>
This commit is contained in:
Jouni Malinen 2023-02-22 17:01:58 +02:00 • committed by Jouni Malinen
parent 596d602de8
commit 7292e30b7f
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/common/dpp.c
View file

@ -1342,7 +1342,7 @@ static int dpp_configuration_parse(struct dpp_authentication *auth,
goto fail; goto fail;
os_memcpy(tmp, cmd, len); os_memcpy(tmp, cmd, len);
tmp[len] = '\0'; tmp[len] = '\0';
res = dpp_configuration_parse_helper(auth, cmd, 0); res = dpp_configuration_parse_helper(auth, tmp, 0);
str_clear_free(tmp); str_clear_free(tmp);
if (res) if (res)
goto fail; goto fail;