bin/98220: wpa_supplicant operation does not match documentation

[Kevin Oberman]

>Number:         98220
>Category:       bin
>Synopsis:       wpa_supplicant operation does not match documentation
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    freebsd-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Wed May 31 15:10:18 GMT 2006
>Closed-Date:
>Last-Modified:
>Originator:     Kevin Oberman
>Release:        FreeBSD 7.0-CURRENT i386
>Organization:
ESnet-The Energy Science Network
>Environment:
System: FreeBSD slan.es.net 7.0-CURRENT FreeBSD 7.0-CURRENT #15: Tue May 23 14:19:22 PDT 2006 root at slan.es.net:/usr/obj/usr/src/sys/IBM-T43 i386


	
>Description:
	The wpa_supplicant man page claims that if several SSIDs are
contained in a single priority group, they will be preferred in the
order they appear. This is not the case. the selection is two 'for'
loops:
list of scanned BSSes
	check for matches with configured networks
Thus, the first entry in the list of scanned networks (which is,
essentially random) will match with any of the the networks in the
priority group with no attempt to match any other BSSes to the earlier
entries.

>How-To-Repeat:
	Build a wpa_supplicant.conf file with several networks in the
preferred order. Start the supplicant in a location where available
BSSes match both the first and last network entry. Start the
supplicant in debug. It will choose the first BSS in the list of
available BSSes that matches any entry. Restart the supplicant for a
repeat with a re-ordered BSSes and the first match is chosen again.

>Fix:
Either:
	The code in wpa_supplicant_select_bss in events.c needs to be
re-ordered so that the testing of each network ssid against the first
BSS seen is done before moving on to the next BSS.
or:
	Modify the documentation to state that a match to any SSID listed
in a priority group will be chosen at random with no regard to the order
of the entries in the configuration.
>Release-Note:
>Audit-Trail:
>Unformatted: