Issue 169 in mp4v2: Modifying tags with mp4tags -m "test" corrupts some specific MP4 files

[mp...@googlecode.com]

Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 169 by e...@bizmodeller.com: Modifying tags with mp4tags
-m "test" corrupts some specific MP4 files
http://code.google.com/p/mp4v2/issues/detail?id=169

Note this is obviously not a generic problem, it is specific to some
specific MP4 files. A test file that this issue is reproducible with is
referenced below.

What steps will reproduce the problem?
1. Download this specific MP4 file:

https://s3.amazonaws.com/bizmodeller/Staging/test.mp4

2. Run mp4info test.mp4:

c:\mp4v2\bin\Windows-x64\Debug>mp4info test.mp4
mp4info version 2.0-r477
test.mp4:
mp4v2::impl::MP4Atom::ReadAtom: "test.mp4": atom type is suspect
Track Type Info
1 video H264 High@3, 2484.582 secs, 1068 kbps, 686x344 @ 29.970031
fps
2 audio MPEG-4 AAC LC, 2484.480 secs, 154 kbps, 48000 Hz
mp4v2::impl::MP4Atom::ReadAtom: "test.mp4": atom type is suspect
Name: Amish.Mafia.S03E05.The.Bear
Encoded with: HandBrake 0.9.9 2013051800
TV Show: Amish Mafia
TV Episode: 5
TV Season: 3

3. Run:

mp4tags -m "test" test.mp4

4. Run mp4info test.mp4 (again):

c:\mp4v2\bin\Windows-x64\Debug>mp4info test.mp4
mp4info version 2.0-r477
test.mp4:
mp4v2::impl::MP4Atom::ReadAtom: "test.mp4": atom type is suspect
mp4v2::impl::MP4Atom::ReadChildAtoms: "test.mp4": In atom missing child
atom moov
mp4v2::impl::MP4File::FindIntegerProperty: no such property -
moov.mvhd.modificationTime (..\..\src\mp4file.cpp,746)
mp4info: can't open test.mp4

5. The file is now corrupted and unplayable

Note: it does not appear to matter which tag is updated.
Note 2: Performing an optimize before updating the tag allows the tag
update to succeed without corruption

What is the expected output? What do you see instead?

The expected output is that either MP4Tags/libmp4v2.dll should update the
tag successfully, or it should not corrupt the file.

What version of the product are you using? On what operating system?

version 2.0-r477 on Windows 8 64-bit

Please provide any additional information below.

A dump of the file using mp4file --dump (prior to tag modification) is
attached for your reference.


Attachments:
dump.txt 15.3 KB

--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings