[FFmpeg-devel] [PATCH] asfenc: fix extended content header

Michael Niedermayer michaelni
Fri Feb 19 23:50:08 CET 2010


On Thu, Feb 18, 2010 at 07:15:47PM +0100, Anton Khirnov wrote:
> On Wed, Feb 17, 2010 at 12:32:28AM +0100, Michael Niedermayer wrote:
> > 
> > see issue1065
> > 
> oops, that patch was wrong. it should have applied only to unicode tags.
> 
> as for the file in that issue, it's an abomination containing broken
> ID3. imo the best/simplest solution is to skip byte array tags
> completely.

do we loose some metadata that way? if so rejected
if no elaborate


> 
> patches for this attached.
> 
> Anton Khirnov

>  asfdec.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> c16b78d0761a73eb9d0f5a8712e9bd18dc85cac5  0001-asfdec-only-unicode-tags-must-have-even-length.patch
> From 7f52208a755d624f0b0e7b66050d1a021240b632 Mon Sep 17 00:00:00 2001
> From: Anton Khirnov <wyskas at gmail.com>
> Date: Thu, 18 Feb 2010 19:04:17 +0100
> Subject: [PATCH 1/2] asfdec: only unicode tags must have even length.

ok if tested

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Old school: Use the lowest level language in which you can solve the problem
            conveniently.
New school: Use the highest level language in which the latest supercomputer
            can solve the problem without the user falling asleep waiting.
-------------- 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/20100219/a109a079/attachment.pgp>



More information about the ffmpeg-devel mailing list