[PATCH] wpa_supplicant: Work around odd driver-reported ssid lengths

[Dan Williams]

On Thu, 2006-01-12 at 18:54 +0100, Henrik Brix Andersen wrote:
> On Thu, Jan 12, 2006 at 11:30:37AM -0500, Dan Williams wrote:
> > At least the atmel and airo drivers do odd, technically incorrect,
> > things in their SIOCGIWESSID:
> 
> As you've pointed out yourself, the drivers do "technically incorrect
> things" whereas wpa_supplicant does the right thing.
> 
> Please fix the offending drivers instead of breaking wpa_supplicant.

Right, so I've got a patch (attached) that does this that I'll be
submitting to netdev at vger.  But two questions:

1) What if upstream rejects the patch?  The patch may break existing
user-space tools.  Turns out that airo, atmel, prism54, ray_cs, and
wavelan_cs all have this problem.  If upstream rejects, what will
wpa_supplicant do?  Continue to fail with these drivers?

2) If the patch is accepted, will it be noted somewhere that
wpa_supplicant is unreliable with the previously stated cards and
kernels older than (and including) 2.6.15?

"fix the offending drivers instead of breaking wpa_supplicant" is an
entirely inconsistent line, since there are already quite a few
compromises in wpa_supplicant.  For example, working with non-WEXT
drivers.  I'm just trying to encourage keeping an open mind when it
comes to workarounds here, since kernel drivers, wpa_supplicant, and
wireless in general are not perfect but must still work with each other
as well as possible.

Dan


-------------- next part --------------
A non-text attachment was scrubbed...
Name: wireless-ssid-len-fix.patch
Type: text/x-patch
Size: 2163 bytes
Desc: not available
Url : http://lists.shmoo.com/pipermail/hostap/attachments/20060112/8d0b0341/attachment.bin