bug 456 [REGRESSION 0.7.3-6 => 1.0-2] WPA-PSK + openmoko: unable to get DHCPACK

Timo Juhani Lindfors timo.lindfors
Thu Jun 28 09:42:03 PDT 2012


I bisected this down to

commit 180cdf45a415969b851aab530e9b23606032941a
Author: Jouni Malinen <jouni at qca.qualcomm.com>
Date:   Tue Oct 18 23:04:36 2011 +0300

    wext: Increase scan timeout from 5 to 10 seconds
    Some dualband cards can use more than five seconds to run through
    a full scan, so increase the timeout to avoid hitting the missing
    scan completed event workaround.

diff --git a/src/drivers/driver_wext.c b/src/drivers/driver_wext.c
index 9fa356b..e44aeac 100644
--- a/src/drivers/driver_wext.c
+++ b/src/drivers/driver_wext.c
@@ -1007,7 +1007,7 @@ int wpa_driver_wext_scan(void *priv, struct wpa_driver_scan_params *params)
        /* Not all drivers generate "scan completed" wireless event, so try to
         * read results after a timeout. */
-       timeout = 5;
+       timeout = 10;
        if (drv->scan_complete_events) {
                 * The driver seems to deliver SIOCGIWSCAN events to notify

Here's the log for completeness:

git bisect start
# good: [0b86f67a2970403d1b1b6d34ce49f5485ef5dbb3] Add wpa_gui-qt4 translation files into build
git bisect good 0b86f67a2970403d1b1b6d34ce49f5485ef5dbb3
# bad: [106233504ab2f55dd5f5b6454e2443aed7914383] Update version number for 1.0 release.
git bisect bad 106233504ab2f55dd5f5b6454e2443aed7914383
# good: [2a0cd0670b08ae84796352e17279b792e26fa8ba] OpenSSL: Add access to Android certificates
git bisect good 2a0cd0670b08ae84796352e17279b792e26fa8ba
# good: [a4cba8f1e208c95218f89daec01cbcb59d97dc6c] Use sched_scan in driver init
git bisect good a4cba8f1e208c95218f89daec01cbcb59d97dc6c
# bad: [1dc77cda118410904886776194f56f3623f830b4] Make crypto_hash_init() easier for static analyzers
git bisect bad 1dc77cda118410904886776194f56f3623f830b4
# bad: [3803bd331de647bb99a80db00cf19361b0b04be2] TLS: Validate RSA ClientKeyExchange length field
git bisect bad 3803bd331de647bb99a80db00cf19361b0b04be2
# good: [ea08bfe3847c7075c7fab802441a519dd02c7654] netlink: Do not use void pointer for pointer arithmetic
git bisect good ea08bfe3847c7075c7fab802441a519dd02c7654
# bad: [9fb0407055112d9f24cd451ce880326197988a61] nl80211: Use a wrapper for genlmsg_put
git bisect bad 9fb0407055112d9f24cd451ce880326197988a61
# bad: [9b90955ec706905d1c5b864885d252ebf7ddba4e] AP: Pass only bssid/addr/wds to EVENT_RX_FROM_UNKNOWN
git bisect bad 9b90955ec706905d1c5b864885d252ebf7ddba4e
# bad: [7a25f29d890f56c4df81143f8b5eb42fb5eeeb07] Interworking: Use unsigned integer for bitfield
git bisect bad 7a25f29d890f56c4df81143f8b5eb42fb5eeeb07
# good: [54e9c5fc6976c4cd1fd4cccf2b7dadaad2c4e437] EAP: Clear ClientTimeout back to default value in INITIALIZE
git bisect good 54e9c5fc6976c4cd1fd4cccf2b7dadaad2c4e437
# bad: [180cdf45a415969b851aab530e9b23606032941a] wext: Increase scan timeout from 5 to 10 seconds
git bisect bad 180cdf45a415969b851aab530e9b23606032941a
# good: [0597a5b59d731142d53bd854b18f802fb2e01046] Add log_level command
git bisect good 0597a5b59d731142d53bd854b18f802fb2e01046

Any idea what the _real_ issue here could be? Surely increasing timeout
should not cause regressions?

More information about the Hostap mailing list