[FFmpeg-devel] [PATCH 3/3] id3v2: don't overwrite existing tags

Måns Rullgård mans
Thu Jan 20 14:40:46 CET 2011


Anton Khirnov <anton at khirnov.net> writes:

> On Thu, Jan 20, 2011 at 11:45:55AM +0000, M?ns Rullg?rd wrote:
>> Anton Khirnov <anton at khirnov.net> writes:
>> 
>> > Apparrently some broken taggers append a new ID3v2 tag leaving the
>> > existing one intact. Our parser currently reads all tags and overwrites
>> > existing values with supposedly outdated ones.
>> 
>> Did you mean prefix rather than append?  I can't make sense of this
>> any other way.
> Yes, I meant prepend. Updated patch attached.
>
> -- 
> Anton Khirnov
>
> From 350536916798ac4df144172a1f4539945e7a201c Mon Sep 17 00:00:00 2001
> From: Anton Khirnov <anton at khirnov.net>
> Date: Thu, 20 Jan 2011 09:06:58 +0100
> Subject: [PATCH] id3v2: don't overwrite existing tags
>
> Apparrently some broken taggers prepend a new ID3v2 tag leaving the
> existing one intact. Our parser currently reads all tags and overwrites
> existing values with supposedly outdated ones.
>
> fixes issue2419

Thanks, queued.

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-devel mailing list