diff options
| author |   Neil Horman <nhorman@tuxdriver.com> | 2019-06-13 06:35:59 -0400 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2019-06-14 19:27:24 -0700 |
| commit | ce950f1050cece5e406a5cde723c69bba60e1b26 (patch) | |
| tree | 8aecfaeb0ff4506107ead6fad531a4b30c31633b /drivers/net/dsa/rtl8366rb.c | |
| parent | net: dsa: microchip: Don't try to read stats for unused ports (diff) | |
| download | linux-dev-ce950f1050cece5e406a5cde723c69bba60e1b26.tar.xz linux-dev-ce950f1050cece5e406a5cde723c69bba60e1b26.zip | |
sctp: Free cookie before we memdup a new one
Based on comments from Xin, even after fixes for our recent syzbot
report of cookie memory leaks, its possible to get a resend of an INIT
chunk which would lead to us leaking cookie memory.
To ensure that we don't leak cookie memory, free any previously
allocated cookie first.
Change notes
v1->v2
update subsystem tag in subject (davem)
repeat kfree check for peer_random and peer_hmacs (xin)
v2->v3
net->sctp
also free peer_chunks
v3->v4
fix subject tags
v4->v5
remove cut line
Signed-off-by: Neil Horman <nhorman@tuxdriver.com>
Reported-by: syzbot+f7e9153b037eac9b1df8@syzkaller.appspotmail.com
CC: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>
CC: Xin Long <lucien.xin@gmail.com>
CC: "David S. Miller" <davem@davemloft.net>
CC: netdev@vger.kernel.org
Acked-by: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to '')
0 files changed, 0 insertions, 0 deletions