15/01/2019 11:01, David Hunt: > From: Liang Ma <[email protected]> > > Fixes: e6c6dc0f96c8 ("power: add p-state driver compatibility") > Coverity issue: 328528 > > v2: focus just on the resource leak. Remove additional code around > Turbo Boost frequency handling. > > Signed-off-by: Liang Ma <[email protected]> > Reviewed-by: Lei Yao <[email protected]> > Tested-by: Lei Yao <[email protected]> > Signed-off-by: David Hunt <[email protected]> > --- > --- a/lib/librte_power/power_pstate_cpufreq.c > +++ b/lib/librte_power/power_pstate_cpufreq.c > f_max = fopen(fullpath_max, "rw+"); > + > + if (f_max == NULL) > + fclose(f_min);
No need to insert a blank line before "if". Removing on apply. Applied, thanks
