diff options
| author | Herbert Xu <herbert@gondor.apana.org.au> | 2016-05-16 17:28:16 +0800 |
|---|---|---|
| committer | Mister Oyster <oysterized@gmail.com> | 2017-04-13 12:35:36 +0200 |
| commit | c0cc863130b945a47410b97d617c278748aa9b0b (patch) | |
| tree | cbd1667447644f339e3525b960d48181239f9c32 /scripts/const_structs.checkpatch | |
| parent | db41e4e1c30d890f95b5747a55f570ef9b077666 (diff) | |
netlink: Fix dump skb leak/double free
When we free cb->skb after a dump, we do it after releasing the
lock. This means that a new dump could have started in the time
being and we'll end up freeing their skb instead of ours.
This patch saves the skb and module before we unlock so we free
the right memory.
Change-Id: Icdf8adc86b334c32d2f820b5b203989b294fb19f
Fixes: 16b304f3404f ("netlink: Eliminate kmalloc in netlink dump operation.")
Reported-by: Baozeng Ding <sploving1@gmail.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Acked-by: Cong Wang <xiyou.wangcong@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'scripts/const_structs.checkpatch')
0 files changed, 0 insertions, 0 deletions