Hi Jonathan,

Quoting Jonathan Cameron <ji...@kernel.org>:

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!


OK, I will do that. :)

Thanks!
--
Gustavo A. R. Silva





Reply via email to