Re: kern/39364: wpa_cli scan_results shows '0' signal level

[christos%zoulas.com@localhost (Christos Zoulas)]

The following reply was made to PR kern/39364; it has been noted by GNATS.

From: christos%zoulas.com@localhost (Christos Zoulas)
To: gnats-bugs%NetBSD.org@localhost, kern-bug-people%netbsd.org@localhost, 
        gnats-admin%netbsd.org@localhost, netbsd-bugs%netbsd.org@localhost, 
jmcneill%invisible.ca@localhost
Cc: 
Subject: Re: kern/39364: wpa_cli scan_results shows '0' signal level
Date: Fri, 29 Aug 2008 00:57:10 -0400

 On Aug 29, 12:10am, jmcneill%invisible.ca@localhost ("Jared D. McNeill") wrote:
 -- Subject: Re: kern/39364: wpa_cli scan_results shows '0' signal level
 
 | The following reply was made to PR kern/39364; it has been noted by GNATS.
 | 
 | From: "Jared D. McNeill" <jmcneill%invisible.ca@localhost>
 | To: gnats-bugs%NetBSD.org@localhost
 | Cc: kern-bug-people%netbsd.org@localhost, gnats-admin%netbsd.org@localhost, 
 |  netbsd-bugs%netbsd.org@localhost
 | Subject: Re: kern/39364: wpa_cli scan_results shows '0' signal level
 | Date: Thu, 28 Aug 2008 20:06:14 -0400
 | 
 |  The following patch fixes the problem for me. Can someone verify its 
 |  correctness?
 |  
 |  Index: driver_netbsd.c
 |  ===================================================================
 |  RCS file: /cvsroot/src/usr.sbin/wpa/wpa_supplicant/driver_netbsd.c,v
 |  retrieving revision 1.6
 |  diff -u -r1.6 driver_netbsd.c
 |  --- driver_netbsd.c 15 Jun 2008 18:33:32 -0000      1.6
 |  +++ driver_netbsd.c 29 Aug 2008 00:05:48 -0000
 |  @@ -695,8 +695,8 @@
 |              wsr->ssid_len = sr->isr_ssid_len;
 |              wsr->freq = sr->isr_freq;
 |              wsr->noise = sr->isr_noise;
 |  -           wsr->qual = sr->isr_rssi;
 |  -           wsr->level = 0;         /* XXX? */
 |  +           wsr->qual = 0;                  /* XXX? */
 |  +           wsr->level = sr->isr_rssi;
 |              wsr->caps = sr->isr_capinfo;
 |              wsr->maxrate = getmaxrate(sr->isr_rates, sr->isr_nrates);
 |              vp = (u_int8_t *)(sr+1);
 
 Sure, if it works...
 
 christos