[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Fri, 7 Jul 2017 11:26:35 +0200
From: Peter Rosin <peda@...ntia.se>
To: "Gustavo A. R. Silva" <garsilva@...eddedor.com>,
Jonathan Cameron <jic23@...nel.org>,
Hartmut Knaack <knaack.h@....de>,
Lars-Peter Clausen <lars@...afoo.de>,
Peter Meerwald-Stadler <pmeerw@...erw.net>
Cc: linux-iio@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] iio: multiplexer: add NULL check on devm_kzalloc() and
devm_kmemdup() return values
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 <peda@...ntia.se>
>> Signed-off-by: Gustavo A. R. Silva <garsilva@...eddedor.com>
>
> Either way,
>
> Reviewed-by: Peter Rosin <peda@...ntia.se>
>
> Thanks!
>
Powered by blists - more mailing lists