From: Jouni Malinen <jouni@codeaurora.org>
Date: Fri, 11 Oct 2019 10:04:42 +0000 (+0300)
Subject: DPP: Fix confusing debug entry from Configurator
X-Git-Tag: hostap_2_10~2306
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=9c08bfbd91566794b14da873859bff8190d7b5ff;p=thirdparty%2Fhostap.git

DPP: Fix confusing debug entry from Configurator

The logic for displaying the message about no configuration being
available for the Enrollee was reversed when support for multiple Config
Objects was added. This was supposed to be shown only if the first
Config Object fails (i.e., when no configuration is available); not when
second attempt fails (since the first one had already been successful in
that case).

Fixes: 7eb06a33697f ("DPP2: Allow multiple Config Objects to be build on Configurator")
Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
---

diff --git a/src/common/dpp.c b/src/common/dpp.c
index a1f94e5f1..ab7072cf6 100644
--- a/src/common/dpp.c
+++ b/src/common/dpp.c
@@ -5013,7 +5013,7 @@ dpp_build_conf_obj(struct dpp_authentication *auth, int ap, int idx)
 	else
 		conf = NULL;
 	if (!conf) {
-		if (idx != 0)
+		if (idx == 0)
 			wpa_printf(MSG_DEBUG,
 				   "DPP: No configuration available for Enrollee(%s) - reject configuration request",
 				   ap ? "ap" : "sta");