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'


> +             return ERR_PTR(ret);

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project
--
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

Reply via email to