From: Andrey Konovalov <andreyk...@gmail.com> Date: Mon, 7 Mar 2016 22:50:41 +0300
> On Mon, Mar 7, 2016 at 10:11 PM, David Miller <da...@davemloft.net> wrote: >> From: Linus Torvalds <torva...@linux-foundation.org> >> Date: Mon, 7 Mar 2016 10:13:09 -0800 >> >>> On Sat, Mar 5, 2016 at 11:53 AM, Bjørn Mork <bj...@mork.no> wrote: >>>> >>>> >>>> Definitely. The patch is so obviously correct that we can only wonder how >>>> it was possible to miss it it the first place :) >>>> >>>> Will take a look to see if we could do a better job cleaning up in other >>>> places. >>> >>> What should I do for 4.5? Will there be a pull request for this, or >>> should I just commit my cdc_ncm_bind() patch directly? >> >> Yes I plan to send you a pull request today with Oliver's fix. >> >> Assuming this is what you guys are referring to: >> >> commit 1666984c8625b3db19a9abc298931d35ab7bc64b >> Author: Oliver Neukum <oneu...@suse.com> >> Date: Mon Mar 7 11:31:10 2016 +0100 >> >> usbnet: cleanup after bind() in probe() >> >> In case bind() works, but a later error forces bailing >> in probe() in error cases work and a timer may be scheduled. >> They must be killed. This fixes an error case related to >> the double free reported in >> http://www.spinics.net/lists/netdev/msg367669.html >> and needs to go on top of Linus' fix to cdc-ncm. >> >> Signed-off-by: Oliver Neukum <oneu...@suse.com> >> Signed-off-by: David S. Miller <da...@davemloft.net> > > Could you also add: > Reported-by: Andrey Konovalov <andreyk...@gmail.com> > ? Sorry it's already committed to my tree and I can't redo the commit message once that happens since my tree has static history.
