On Wed, Jun 12, 2019 at 03:07:15PM -0300, Marcelo Ricardo Leitner wrote: > 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. > thats intentional so that, when Dave commits it, the change notes arent carried into the changelog (I.e. the change notes are useful for email review, but not especially useful in the permanent commit history). Neil > > 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> >
