[MPlayer-dev-eng] [PATCH] Fix a sndio crash when exiting in muted state
Brad Smith
brad at comstyle.com
Sat Mar 25 10:02:23 EET 2023
On 2023-03-24 4:38 p.m., Alexander Strasser wrote:
> Hi Brad,
>
> On 2023-03-22 22:58 +0100, Reimar Döffinger wrote:
>>> Index: libao2/ao_sndio.c
>>> ===================================================================
>>> --- libao2/ao_sndio.c (revision 38412)
>>> +++ libao2/ao_sndio.c (working copy)
>>> @@ -159,6 +159,7 @@
>>> pfds = NULL;
>>> sio_close(hdl);
>>> hdl = NULL;
>>> + havevol = 0;
>>> return 0;
>>> }
>>>
>>> @@ -172,6 +173,7 @@
>>> hdl = NULL;
>>> free(pfds);
>>> pfds = NULL;
>>> + havevol = 0;
>>> }
>> Well, it's probably a good idea to do that, but a aoctl happening after uninit (if I understand right) is the far worse issue an probably needs to be fixed by itself.
>
> Could you test if the attached patch also fixes the issues you observed?
>
> Only lightly tested but it would help if you could confirm if it fixes
> the problem.
>
>
> Thanks,
> Alexander
According to feedback MPlayer no longer crashes with this applied and
the other patch
removed.
More information about the MPlayer-dev-eng
mailing list