[FFmpeg-devel] [RFC] switch to poll()

Luca Barbato lu_zero
Tue Nov 16 18:06:48 CET 2010


On 11/16/2010 12:33 AM, Martin Storsj? wrote:
> On Tue, 16 Nov 2010, Martin Storsj? wrote:
> 
>> On Mon, 15 Nov 2010, Luca Barbato wrote:
>>
>>>> Do you want me to test run some of the modified codepaths that you haven't 
>>>> tested yourself, e.g. SAP?
>>>
>>> Yes please
>>
>> Ok, will do.
> 
> The two attached patches need to be squashed into this, then after that, 
> it seems to work fine with both RTSP and SAP.
> 
> The bug with max_p not being reinitialized was particularly nasty - it 
> lead to kernel panics on OS X, when poll was fed with too long pollfd 
> list. (Not sure at which point it happened though, if the bug was due to 
> the kernel reading out of bounds, or if it just was "too many"...)

I wonder why I didn't notice it before =\ shame on me =P

Regarding moving out the for the initialization code, I think it should
be done.

lu


-- 

Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero




More information about the ffmpeg-devel mailing list