diff options
| author | Myungho Jung <mhjungk@gmail.com> | 2017-04-25 11:58:15 -0700 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2017-04-26 14:47:14 -0400 |
| commit | 9899886d5e8ec5b343b1efe44f185a0e68dc6454 (patch) | |
| tree | 85732488c432e6e70b0fe73e62cfc46370162dae /net/core/dev.c | |
| parent | 5294b83086cc1c35b4efeca03644cf9d12282e5b (diff) | |
| download | op-kernel-dev-9899886d5e8ec5b343b1efe44f185a0e68dc6454.zip op-kernel-dev-9899886d5e8ec5b343b1efe44f185a0e68dc6454.tar.gz | |
net: core: Prevent from dereferencing null pointer when releasing SKB
Added NULL check to make __dev_kfree_skb_irq consistent with kfree
family of functions.
Link: https://bugzilla.kernel.org/show_bug.cgi?id=195289
Signed-off-by: Myungho Jung <mhjungk@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/core/dev.c')
| -rw-r--r-- | net/core/dev.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/net/core/dev.c b/net/core/dev.c index 533a6d6..9b58753 100644 --- a/net/core/dev.c +++ b/net/core/dev.c @@ -2450,6 +2450,9 @@ void __dev_kfree_skb_irq(struct sk_buff *skb, enum skb_free_reason reason) { unsigned long flags; + if (unlikely(!skb)) + return; + if (likely(atomic_read(&skb->users) == 1)) { smp_rmb(); atomic_set(&skb->users, 0); |
