[FFmpeg-devel] [PATCH] fix crash in ffplay.c #2

Michael Niedermayer michaelni
Tue Dec 30 01:33:47 CET 2008


On Mon, Dec 29, 2008 at 06:27:54PM -0500, Ronald S. Bultje wrote:
> Hi,
> 
> when av_find_stream_info() completes but not all variables have been
> found, some values (e.g. a channel count) can be left unset (0).
> ffplay.c doesn't take that into account and uses channels as a
> denominator in a division in line 783 (and following). The attached
> patch fixes this (wrongly, but I couldn't think of anything better and
> maybe someone interested can actually do something useful with it :-)
> ).

This is certainly not a correct fix ...
And its hard to say how to fix it without any clue about the kind of
file you try to play, or why channels are not set.

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081230/80ccd06e/attachment.pgp>



More information about the ffmpeg-devel mailing list