Re: [PATCH] ath11k: Fix some resource leaks in error path in 'ath11k_thermal_register()'

2020-05-18 Thread Kalle Valo
Christophe JAILLET wrote: > If 'thermal_cooling_device_register()' fails, we must undo what has been > allocated so far. So we must go to 'err_thermal_destroy' instead of > returning directly > > In case of error in 'ath11k_thermal_register()', the previous > 'thermal_cooling_device_register()'

Re: [PATCH] ath11k: Fix some resource leaks in error path in 'ath11k_thermal_register()'

2020-05-13 Thread Rajkumar Manoharan
On 2020-05-13 13:14, Christophe JAILLET wrote: If 'thermal_cooling_device_register()' fails, we must undo what has been allocated so far. So we must go to 'err_thermal_destroy' instead of returning directly In case of error in 'ath11k_thermal_register()', the previous 'thermal_cooling_device_re

[PATCH] ath11k: Fix some resource leaks in error path in 'ath11k_thermal_register()'

2020-05-13 Thread Christophe JAILLET
If 'thermal_cooling_device_register()' fails, we must undo what has been allocated so far. So we must go to 'err_thermal_destroy' instead of returning directly In case of error in 'ath11k_thermal_register()', the previous 'thermal_cooling_device_register()' call must also be undone. Move the 'ar->