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