staging:r8188eu: avoid skb_clone for amsdu to msdu conversion (628cbd97) · Commits · 戴 / test

skb clones use same data buffer, so tail of one skb is corrupted by beginning of next skb. Signed-off-by:

drivers/staging/rtl8188eu/core/rtw_recv.c

+6 −13

Original line number	Diff line number	Diff line
		@@ -1525,20 +1525,13 @@ static int amsdu_to_msdu(struct adapter padapter, struct recv_frame prframe)

		/* Allocate new skb for releasing to upper layer */
		sub_skb = dev_alloc_skb(nSubframe_Length + 12);
		if (sub_skb) {
		skb_reserve(sub_skb, 12);
		skb_put_data(sub_skb, pdata, nSubframe_Length);
		} else {
		sub_skb = skb_clone(prframe->pkt, GFP_ATOMIC);
		if (sub_skb) {
		sub_skb->data = pdata;
		sub_skb->len = nSubframe_Length;
		skb_set_tail_pointer(sub_skb, nSubframe_Length);
		} else {
		DBG_88E("skb_clone() Fail!!! , nr_subframes=%d\n", nr_subframes);
		if (!sub_skb) {
		DBG_88E("dev_alloc_skb() Fail!!! , nr_subframes=%d\n", nr_subframes);
		break;
		}
		}

		skb_reserve(sub_skb, 12);
		skb_put_data(sub_skb, pdata, nSubframe_Length);

		subframes[nr_subframes++] = sub_skb;