Re: [PATCH v1 1/2] tests/chamelium/audio: Fix ALSA handle leak on skipped audio config

Sebastian Brzezinka
2026-03-04T11:18:53Z

Hi Mohammed,

On Wed Mar 4, 2026 at 11:05 AM CET, Mohammed Bilal wrote:
> When the audio configuration check fails, the loop continues without
> closing previously opened ALSA handles. This leads to handle leaks
> across iterations and may eventually cause file descriptor exhaustion
> or configuration failures.
>
> Fix by ensuring the handles are closed before continuing.
>
> Signed-off-by: Mohammed Bilal <mohammed.bilal@intel.com>
> ---
This is a clean fix, I have no concerns.

Reviewed-by: Sebastian Brzezinka <sebastian.brzezinka@intel.com>

--
Best regards,
Sebastian