On Tue, Jun 11, 2019 at 08:38:14PM -0400, Neil Horman wrote: > 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. > > --- This marker can't be here, as it causes git to loose everything below. > 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 > > Signed-off-by: Neil Horman <nhorman@xxxxxxxxxxxxx> > Reported-by: syzbot+f7e9153b037eac9b1df8@xxxxxxxxxxxxxxxxxxxxxxxxx > CC: Marcelo Ricardo Leitner <marcelo.leitner@xxxxxxxxx> > CC: Xin Long <lucien.xin@xxxxxxxxx> > CC: "David S. Miller" <davem@xxxxxxxxxxxxx> > CC: netdev@xxxxxxxxxxxxxxx Anyhow, LGTM and reproducer didn't give any hits in 2 runs of 50mins. Acked-by: Marcelo Ricardo Leitner <marcelo.leitner@xxxxxxxxx>
