Commit 9f64df94 authored Sep 18, 2015 by Hante Meuleman Committed by Kalle Valo Sep 29, 2015

brcmfmac: Fix bug in flowring management.



The hash index stored in the flowrings is of type u16 but gets
stored in u8. This can result in incorrect indexing and possibly
result in crashes. This patch fixes the type.

Reviewed-by: Arend Van Spriel <arend@broadcom.com>
Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com>
Signed-off-by: Hante Meuleman <meuleman@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>

parent ae7c03f6

drivers/net/wireless/brcm80211/brcmfmac/flowring.h

+1 −1

Original line number	Original line	Diff line number	Diff line
	@@ -34,7 +34,7 @@ enum ring_status {
	};		};

	struct brcmf_flowring_ring {		struct brcmf_flowring_ring {
	u8 hash_id;		u16 hash_id;
	bool blocked;		bool blocked;
	enum ring_status status;		enum ring_status status;
	struct sk_buff_head skblist;		struct sk_buff_head skblist;