[MPlayer-dev-eng] [PATCH] fix for segmentation fault in ao_nas
Guillaume POIRIER
poirierg at gmail.com
Mon Jun 13 18:12:38 CEST 2005
Hi,
On 5/27/05, Erik Auerswald <auerswal at unix-ag.uni-kl.de> wrote:
> Hi,
>
> audio output using ao_nas sometimes crashes mplayer with a segmentation
> fault. This is caused by NAS server messages requesting large amounts of
> data. One error in libao2/ao_nas.c is subtracting an AuUint32 from an int
> value. The other is not checking if enough data is in the buffer to
> fulfill the request.
>
> The latter problem is solved by limiting the sent data to the available
> amount of data. The former can be solved by either using 64bit integer
> types or checking the AuUint32 for values that overflow an int in the
> current implementation.
>
> I've attached 2 alternative patches, one for each of the overflow fixes,
> because I'm not sure which way is "better".
>
> Erik
>
> P.S. I've already sent alternative 1 to this list, but I didn't use the
> [PATCH] subject line, so it probably was overlooked.
Has anyone had a look at it?
Guillaume
--
I should have invented the tape recorder,
that way people would have more quotes of
me to put at the bottom of their e-mails
-- Benjamin Franklin
More information about the MPlayer-dev-eng
mailing list