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 a1f94e5f11a334c94c92f510708255d5f74cf4b6..ab7072cf65a5edb0aee13a0794330f370bb69cf5 100644 (file)
--- 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");