diff options
author | green <green@FreeBSD.org> | 2003-11-04 00:31:58 +0000 |
---|---|---|
committer | green <green@FreeBSD.org> | 2003-11-04 00:31:58 +0000 |
commit | 482fc15052e75fb79667ea8cc7f8d4253315f3cd (patch) | |
tree | e7160285885d002a5890324ad3347f237b5c0891 /sys/dev/wi | |
parent | 6f7aa7fb6c3d294fc1263891bcf3cb61c2ae70ba (diff) | |
download | FreeBSD-src-482fc15052e75fb79667ea8cc7f8d4253315f3cd.zip FreeBSD-src-482fc15052e75fb79667ea8cc7f8d4253315f3cd.tar.gz |
Fix wi(4)'s WI_RID_SCAN_RES ioctl (wicontrol -L). The wrong length
is computed, so the user thinks that for non-PRISM cards there are
more APs represented than exist.
Diffstat (limited to 'sys/dev/wi')
-rw-r--r-- | sys/dev/wi/if_wi.c | 22 |
1 files changed, 13 insertions, 9 deletions
diff --git a/sys/dev/wi/if_wi.c b/sys/dev/wi/if_wi.c index 9703010..5c5c716 100644 --- a/sys/dev/wi/if_wi.c +++ b/sys/dev/wi/if_wi.c @@ -1838,6 +1838,7 @@ wi_get_cfg(struct ifnet *ifp, u_long cmd, caddr_t data) struct ifreq *ifr = (struct ifreq *)data; struct wi_req wreq; struct wi_scan_res *res; + size_t reslen; int len, n, error, mif, val, off, i; error = copyin(ifr->ifr_data, &wreq, sizeof(wreq)); @@ -1964,11 +1965,16 @@ wi_get_cfg(struct ifnet *ifp, u_long cmd, caddr_t data) break; } n = sc->sc_naps; - off = sc->sc_firmware_type != WI_LUCENT ? - sizeof(struct wi_scan_p2_hdr) : 0; - if (len < off + sizeof(struct wi_scan_res) * n) - n = (len - off) / sizeof(struct wi_scan_res); - len = off + sizeof(struct wi_scan_res) * n; + if (sc->sc_firmware_type == WI_LUCENT) { + off = 0; + reslen = WI_WAVELAN_RES_SIZE; + } else { + off = sizeof(struct wi_scan_p2_hdr); + reslen = WI_PRISM2_RES_SIZE; + } + if (len < off + reslen * n) + n = (len - off) / reslen; + len = off + reslen * n; if (off != 0) { struct wi_scan_p2_hdr *p2 = (struct wi_scan_p2_hdr *)wreq.wi_val; /* @@ -1982,7 +1988,7 @@ wi_get_cfg(struct ifnet *ifp, u_long cmd, caddr_t data) p2->wi_rsvd = 0; p2->wi_reason = n; /* XXX */ } - for (i = 0; i < n; i++) { + for (i = 0; i < n; i++, off += reslen) { const struct wi_apinfo *ap = &sc->sc_aps[i]; res = (struct wi_scan_res *)((char *)wreq.wi_val + off); @@ -2001,9 +2007,7 @@ wi_get_cfg(struct ifnet *ifp, u_long cmd, caddr_t data) sizeof(res->wi_srates)); res->wi_rate = ap->rate; res->wi_rsvd = 0; - off += WI_PRISM2_RES_SIZE; - } else - off += WI_WAVELAN_RES_SIZE; + } } break; |