[FFmpeg-devel] [PATCH 11/12] asfdec: properly handle multiple index objects

Anton Khirnov anton
Fri Feb 11 14:42:59 CET 2011


On Thu, Feb 10, 2011 at 09:44:53PM -0500, Ronald S. Bultje wrote:
> Hi,
> 
> On Wed, Feb 9, 2011 at 3:56 PM, Anton Khirnov <anton at khirnov.net> wrote:
> > Current code assumes there's always only one and that it's always the
> > right one for the stream we're seeking in.
> > ---
> > ?libavformat/asfdec.c | ? 25 ++++++++++++++++++++-----
> > ?1 files changed, 20 insertions(+), 5 deletions(-)
> 
> Is there a file that this causes issues for, or is this
> hypothetical/security/...?

No, the only sample with multiple video tracks i have is so broken, that
this patch has no visible effect on it.
However it doesn't break anything either, so I guess we might as well do
it by the specs. I'll try to do more tests after I fix asfenc to
properly work with multiple video tracks, so feel free to postpone this
patch if you're feeling overly cautious ;)

-- 
Anton Khirnov
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20110211/33aae25b/attachment.pgp>



More information about the ffmpeg-devel mailing list