svn commit: r324580 - head/sys/net80211

Adrian Chadd adrian at freebsd.org
Fri Oct 13 07:45:26 UTC 2017


no, it checks it.


-a


On 13 October 2017 at 00:11, Hans Petter Selasky <hps at selasky.org> wrote:
> 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


More information about the svn-src-head mailing list