On Thu, Oct 15, 2015 at 9:08 PM, Stephen Boyd <[email protected]> wrote:
> On 10/15, Linus Walleij wrote:
>> @@ -151,10 +174,19 @@ struct clk *icst_clk_register(struct device *dev,
>> init.flags = CLK_IS_ROOT;
>> init.parent_names = (parent_name ? &parent_name : NULL);
>> init.num_parents = (parent_name ? 1 : 0);
>> + icst->map = regmap_init_mmio(NULL, base, &icst_regmap_conf);
>> + if (IS_ERR(icst->map)) {
>> + int ret;
>> +
>> + pr_err("could not initialize ICST regmap\n");
>> + kfree(icst);
>> + ret = PTR_ERR(icst->map);
>
> drivers/clk/versatile/clk-icst.c:183
> icst_clk_register() error: dereferencing freed memory 'icst'
> drivers/clk/versatile/clk-icst.c:184
> icst_clk_register() warn: possible memory leak of 'pclone'
The pclone warning is correct, nice catch. (Fixing it.)
But for the second warning, whatever static checker you're
using for this is unable to handle error pointers:
clk = clk_register(dev, &icst->hw);
if (IS_ERR(clk))
kfree(icst);
return clk;
It is quite obvious that returning clk (which may be an error code)
is OK here.
If you want, I may need to add some specific annotation to shut
up the static checker, any hints?
Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-clk" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
