Re: [PATCH v2] clocksource/drivers: Fix memory leak in ttc_setup_clockevent

2019-10-23 Thread Markus Elfring
> In the implementation of ttc_setup_clockevent() release the allocated > memory for ttcce if clk_notifier_register() fails. I got other wording preferences. Thus I imagine that such a change description can still be improved another bit. Would you like to express the addition of a jump target

Re: [PATCH v2] clocksource/drivers: Fix memory leak in ttc_setup_clockevent

2019-10-23 Thread Michal Simek
On 23. 10. 19 6:31, Navid Emamdoost wrote: > In the implementation of ttc_setup_clockevent() release the allocated > memory for ttcce if clk_notifier_register() fails. > > Fixes: 70504f311d4b ("clocksource/drivers/cadence_ttc: Convert init function > to return error") > Signed-off-by: Navid

[PATCH v2] clocksource/drivers: Fix memory leak in ttc_setup_clockevent

2019-10-22 Thread Navid Emamdoost
In the implementation of ttc_setup_clockevent() release the allocated memory for ttcce if clk_notifier_register() fails. Fixes: 70504f311d4b ("clocksource/drivers/cadence_ttc: Convert init function to return error") Signed-off-by: Navid Emamdoost --- Changes in v2: - Added goto label