lkml.org 
[lkml]   [2019]   [Apr]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [1/7] cpufreq: ap806: Checking implementation ofarmada_8k_cpufreq_init()
From
Date
>> https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/drivers/cpufreq/armada-8k-cpufreq.c?id=05d08e2995cbe6efdb993482ee0d38a77040861a#n137
>
> Thank you for your comments.
> Adding a null pointer check here is indeed safer.

I wonder then why such a return value check was omitted so far.


> However, the purpose of series of patches is to add missing of_node_put
> to avoid memory leaks.

A few lines might become also more interesting besides these source code places.


> So after that, we will also add a null pointer check later.

Would we like to develop another script for the semantic patch language
to detect similarly questionable implementation details?

Regards,
Markus

\
 
 \ /
  Last update: 2019-04-03 14:55    [W:0.032 / U:0.444 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site