Re: [PATCH] Fix a sndio crash when exiting in muted state

Brad Smith <[email protected]> Sat, 25 Mar 2023 04:02:23 -0400
Newsgroups gmane.comp.video.mplayer.devel
Message-ID <[email protected]>
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.
_______________________________________________
MPlayer-dev-eng mailing list
[email protected]
https://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng