Re: [PATCH] brcmfmac: check return from kzalloc in brcmf_fweh_process_event

["Arend van Spriel" <arend@broadcom.com>]

On 11/19/2012 04:53 PM, John W. Linville wrote:
> From: "John W. Linville" <linville@tuxdriver.com>
>
> Signed-off-by: John W. Linville <linville@tuxdriver.com>
> Reported-by: Fengguang Wu <fengguang.wu@intel.com>
> ---
>   drivers/net/wireless/brcm80211/brcmfmac/fweh.c | 3 +++
>   1 file changed, 3 insertions(+)
>
> diff --git a/drivers/net/wireless/brcm80211/brcmfmac/fweh.c b/drivers/net/wireless/brcm80211/brcmfmac/fweh.c
> index 1e4188c..fa8fc44 100644
> --- a/drivers/net/wireless/brcm80211/brcmfmac/fweh.c
> +++ b/drivers/net/wireless/brcm80211/brcmfmac/fweh.c
> @@ -494,6 +494,9 @@ void brcmf_fweh_process_event(struct brcmf_pub *drvr,
>   		alloc_flag = GFP_ATOMIC;
>
>   	event = kzalloc(sizeof(*event) + datalen, alloc_flag);
> +	if (!event)

My fix included a debug statement about the discarded event from firmware.

> +		return;
> +
>   	event->code = code;
>   	event->ifidx = *ifidx;
>
>

I will deal with the merge so:

Acked-by: Arend van Spriel <arend@broadcom.com>

Gr. AvS