On Sat, 26 Aug 2017 08:09:43 +0200 Peter Rosin <p...@axentia.se> wrote:
> On 2017-07-09 19:10, Jonathan Cameron wrote: > > On Fri, 7 Jul 2017 11:26:35 +0200 > > Peter Rosin <p...@axentia.se> wrote: > > > >> On 2017-07-07 06:57, Peter Rosin wrote: > >>> On 2017-07-07 06:53, Gustavo A. R. Silva wrote: > >>>> Check return values from call to devm_kzalloc() and devm_kmemup() > >>> > >>> If someone cares enough: s/devm_kmemup/evm_kmemdup/ > >> > >> Strange, there seems to be some inherent shortage of 'd' characters... > >> > >> Cheers, > >> pea :-) > >> > >>> > >>>> in order to prevent a NULL pointer dereference. > >>>> > >>>> This issue was detected using Coccinelle and the following semantic > >>>> patch: > >>>> > >>>> @@ > >>>> expression x; > >>>> identifier fld; > >>>> @@ > >>>> > >>>> * x = devm_kzalloc(...); > >>>> ... when != x == NULL > >>>> x->fld > >>>> > >>>> Cc: Peter Rosin <p...@axentia.se> > >>>> Signed-off-by: Gustavo A. R. Silva <garsi...@embeddedor.com> > >>> > >>> Either way, > >>> > >>> Reviewed-by: Peter Rosin <p...@axentia.se> > >>> > >>> Thanks! > >>> > > The relevant patch adding the file in questions hasn't yet worked it's > > way back to the iio tree so I can't apply this until it does. > > > > That is likely to be a few weeks away yet. Please give me a poke if > > I seem to have forgotten it! > > Hi Jonathan! > > I saw a patch from Christophe JAILLET [1] and it seemed familiar so I > did a bit of digging. Now, I did go back and check this a couple of > weeks after you request, but iio-mux.c still wasn't in the iio tree, > and then I did forgot about it. Sorry. But here's the poke... > Applied. Thanks - this one had completely dropped off the back of my patch queue. Oops. Jonathan > Cheers, > Peter > > [1] https://lkml.org/lkml/2017/8/26/3 > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html
