[PATCH] Dont exceed scan ssid max size advertised by driver

Jouni Malinen j at w1.fi
Thu Jun 30 12:07:05 PDT 2016


On Thu, Jun 30, 2016 at 10:21:38AM -0700, Dmitry Shmidt wrote:
> On Thu, Jun 30, 2016 at 9:29 AM, Undekari, Sunil Dutt
> <usdutt at qti.qualcomm.com> wrote:
> > I did discuss with Jouni earlier for a similar fix and his response was "Reserving an entry for WILDCARD_SSID_SCAN in this manner would be problematic since it would not allow even a single specific SSID to be scanned for with drivers that support only a single SSID per request.".
> 
> We can restrict proposed patch only for cases when max_ssid > 1.

That's probably fine, but it should be noted that reserving a slot for
the wildcard SSID in all scan requests would be pretty heavy extra
latency for drivers that support only a small number of SSIDs (say, 2).
Anyway, I'm not aware of such driver, so I think I'd accept such a patch
with the understanding that most drivers support something closer to 16
SSIDs where the impact would be minimal.

-- 
Jouni Malinen                                            PGP id EFC895FA



More information about the Hostap mailing list