On Sun, Jul 12, 2020 at 08:25:43AM -0500, Tim Chase wrote:
> That said, I haven't yet applied your patch below (the second patch
> you sent me) since everything was working with the first one. Would
> you still like me to apply that patch and test too? Or are you
> satisfied with the patch you sent to tech@ ?
I expect that either patch in isolation would fix it. I'll commit both.
Generally, 'ic' represents the wireless interface and 'ni' is used when
referring to a peer on the wifi network. So this second diff is valid,
even though combined with the other patch it boils down to a style fix.
Thank you for your patience in getting this resolved :)
> > diff 21633c8848e72769b1658114d9c706c177040a2a /usr/src
> > blob - 70dbaf422bd5ddc4567ef61322718cb15e7453ce
> > file + sys/dev/ic/ar5008.c
> > --- sys/dev/ic/ar5008.c
> > +++ sys/dev/ic/ar5008.c
> > @@ -1005,7 +1005,7 @@ ar5008_rx_process(struct athn_softc *sc,
> > struct mbuf_l (ni->ni_flags & IEEE80211_NODE_RXPROT) &&
> > (ni->ni_rsncipher == IEEE80211_CIPHER_CCMP ||
> > (IEEE80211_IS_MULTICAST(wh->i_addr1) &&
> > - ic->ic_rsngroupcipher == IEEE80211_CIPHER_CCMP))) {
> > + ni->ni_rsngroupcipher == IEEE80211_CIPHER_CCMP))) {
> > if (ar5008_ccmp_decap(sc, m, ni) != 0) {
> > ifp->if_ierrors++;
> > ieee80211_release_node(ic, ni);
>
>
>
