GAS: Fix reporting of GAS query timeouts
authorJouni Malinen <j@w1.fi>
Mon, 13 Aug 2012 18:33:00 +0000 (21:33 +0300)
committerJouni Malinen <j@w1.fi>
Mon, 13 Aug 2012 18:33:00 +0000 (21:33 +0300)
GAS_QUERY_TIMEOUT value was used for two different things - enum for
status callback and #define for internal eloop timeout). The latter
overwrites the former and as such, the timeout reported ended up going
out with value 5 which matches with GAS_QUERY_CANCELLED instead of
GAS_QUERY_TIMEOUT. This value was not used in existing code, so this
should not modify the current behavior. Anyway, the correct reason for
the failure should be reported. Rename the internal #define for the
timeout period to avoid the name conflict. [Bug 463]

Signed-hostap: Jouni Malinen <j@w1.fi>

wpa_supplicant/gas_query.c

index 1fd6e00..efa9be8 100644 (file)
@@ -19,7 +19,7 @@
 #include "gas_query.h"
 
 
-#define GAS_QUERY_TIMEOUT 5
+#define GAS_QUERY_TIMEOUT_PERIOD 5
 
 
 struct gas_query_pending {
@@ -457,7 +457,7 @@ int gas_query_req(struct gas_query *gas, const u8 *dst, int freq,
                return -1;
        }
 
-       eloop_register_timeout(GAS_QUERY_TIMEOUT, 0, gas_query_timeout,
+       eloop_register_timeout(GAS_QUERY_TIMEOUT_PERIOD, 0, gas_query_timeout,
                               gas, query);
 
        return dialog_token;