On Fri, May 16, 2025 at 1:42 PM Johan Hovold johan@kernel.org wrote:
On Fri, May 16, 2025 at 12:40:23PM +0200, Bartosz Golaszewski wrote:
From: Bartosz Golaszewski bartosz.golaszewski@linaro.org
We initialize the data->mutex in gpiod_export() but lack the corresponding mutex_destroy() in gpiod_unexport() causing a resource leak with mutex debugging enabled. Add the call right before kfreeing the GPIO data.
No, there's no resource leak and it's perfectly fine not to call mutex_destroy().
No, there's no leak but with lock debugging it still warns if the mutex is locked when it's being destroyed so the change still makes sense with a modified commit message.
You can't just make shit up and then pretend to fix it...
There's no need for this kind of comment. You made your point clear in the first sentence.
Bartosz