Re: [RESEND PATCH v2 10/16] clk: move checking .git_parent to __clk_core_init()
From: Stephen Boyd
Date: Tue Feb 02 2016 - 15:43:43 EST
On 02/02, Masahiro Yamada wrote:
> 2016-02-02 11:39 GMT+09:00 Stephen Boyd <sboyd@xxxxxxxxxxxxxx>:
> >
> > Right, this patch moves the check to __clk_core_init(), and it
> > will print an error and then keep going (because we lost the goto
> > out part). Once we keep going in __clk_core_init() we'll call
> > __clk_init_parent() and NULL pointer deref.
> >
>
> Ah, I missed that.
>
> We need
>
> ret = -EINVAL;
> goto out;
>
> here, I think.
>
> Shall I send the v3 for 08/16?
>
> Or, if it is too late, shall I send a fix-up patch?
>
I'll split the patch out again and squash in those two lines.
Nothing more to do. Thanks.
--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project