]> granicus.if.org Git - taglib/commit
Update mp4properties.cpp (#893)
authorTim Malseed <t.malseed@gmail.com>
Sun, 17 Mar 2019 13:22:19 +0000 (00:22 +1100)
committerStephen F. Booth <me@sbooth.org>
Sun, 17 Mar 2019 13:22:19 +0000 (08:22 -0500)
commit6455671ecef7014ee83e91e1fd4e1f9b93dff53a
tree4181fda717c8ca4de5b4b68d9572550ae3bd7a49
parent660748210fed054e886095c75c8e9f62276f4320
Update mp4properties.cpp (#893)

When parsing mp4 media header version 1 (mdhd) atoms, the timescale (unit) is parsed as a `LongLong` (8 bytes), but instead should be a `UInt` (4 bytes). This results in an incorrect timescale, and also pushes the offset of the duration (length) off by 4 bytes.

The end result being that the AudioProperties track length for mp4's with mdhd v1 comes back as 0.

See: https://wiki.multimedia.cx/index.php/QuickTime_container

|  Entry | Bytes (v0) | Bytes (v1) |
| :---         |     :---:      | :---: |
| size  | 4  | 4 |
| type  | 4  | 4 |
| version | 1 | 1 |
| flags | 3 | 3 |
| creation time* | 4 | **8** |
| modification time* | 4 | **8** |
| time scale | 4 | 4 |
| duration* | 4 | **8** |
| language | 2 | 2 |
| quality | 2 | 2 |
taglib/mp4/mp4properties.cpp