Date: Fri, 13 Oct 2017 09:11:04 +0200 From: Hans Petter Selasky <hps@selasky.org> To: Adrian Chadd <adrian@FreeBSD.org>, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r324580 - head/sys/net80211 Message-ID: <b4ade0cf-8658-508d-f806-7db54d605b57@selasky.org> In-Reply-To: <201710130649.v9D6n7cI036282@repo.freebsd.org> References: <201710130649.v9D6n7cI036282@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 10/13/17 08:49, Adrian Chadd wrote:
> if (ieee80211_check_rxseq_amsdu_more(rxs)) {
^^^ does this line also need a NULL check?
> ni->ni_rxseqs[tid] = rxseq;
> - if (ieee80211_check_rxseq_amsdu(rxs))
> + if ((rxs != NULL) && ieee80211_check_rxseq_amsdu(rxs))
> IEEE80211_NODE_STAT(ni, rx_amsdu_more_end);
--HPS
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?b4ade0cf-8658-508d-f806-7db54d605b57>