Re: [PATCH 3/7] cfg80211: handle failed skb allocation
On Wed, 2016-07-06 at 13:51 +0200, Johannes Berg wrote: > On Tue, 2016-07-05 at 15:23 +0300, Luca Coelho wrote: > > From: Gregory Greenman > > > > Handle the case when dev_alloc_skb returns NULL. > > > Applied. A Fixes: tag would have been nice, but I added one :) Oh, did I miss that one? Sorry. I tried to add "Fixes" to all the relevant patches I sent... but probably missed some. -- Luca. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 3/7] cfg80211: handle failed skb allocation
On Tue, 2016-07-05 at 15:23 +0300, Luca Coelho wrote: > From: Gregory Greenman > > Handle the case when dev_alloc_skb returns NULL. > Applied. A Fixes: tag would have been nice, but I added one :) johannes -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH 3/7] cfg80211: handle failed skb allocation
From: Gregory Greenman Handle the case when dev_alloc_skb returns NULL. Signed-off-by: Gregory Greenman Signed-off-by: Luca Coelho --- net/wireless/util.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/net/wireless/util.c b/net/wireless/util.c index 2443ee3..b7d1592 100644 --- a/net/wireless/util.c +++ b/net/wireless/util.c @@ -721,6 +721,8 @@ __ieee80211_amsdu_copy(struct sk_buff *skb, unsigned int hlen, * alignment since sizeof(struct ethhdr) is 14. */ frame = dev_alloc_skb(hlen + sizeof(struct ethhdr) + 2 + cur_len); + if (!frame) + return NULL; skb_reserve(frame, hlen + sizeof(struct ethhdr) + 2); skb_copy_bits(skb, offset, skb_put(frame, cur_len), cur_len); -- 2.8.1 -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html