Change #269836
| Category | ffmpeg |
| Changed by | Jun Zhao <barryjzhao@tencent.com> |
| Changed at | Mon 08 Jun 2026 04:18:32 |
| Repository | https://git.ffmpeg.org/ffmpeg.git |
| Project | ffmpeg |
| Branch | master |
| Revision | c75701a62f6ee6267b4a1d5c3ec72f13aa799704 |
Comments
lavf/mov: read multi-valued metadata tags When a metadata tag (e.g. ©ART) contains multiple values, either as multiple 'data' child atoms within one tag or as multiple sibling tag atoms with the same key, only the first value was read. Fix by joining multiple values with semicolons using AV_DICT_APPEND, consistent with Ogg Vorbis Comment handling in oggparsevorbis.c, and reusing the existing 'goto retry' loop that covr already uses. Also add the missing atom.size -= str_size to correctly track remaining bytes in the tag atom, matching the covr path. Limitation: on remux the joined string is written back as a single value, same lossy behavior as Ogg Vorbis. Lossless round-trip would require AV_DICT_MULTIKEY support throughout the metadata pipeline. Fix #22367 Signed-off-by: Jun Zhao <barryjzhao@tencent.com>
Changed files
- libavformat/mov.c