commit | 7d450d6152835e65b742a014733d19f92cb6ca6f | [log] [tgz] |
---|---|---|
author | Vignesh Venkatasubramanian <vigneshv@google.com> | Tue Nov 03 15:07:18 2020 -0800 |
committer | Vignesh Venkatasubramanian <vigneshv@google.com> | Tue Nov 03 15:07:18 2020 -0800 |
tree | f11b37fd99c1cbf1a9b4d76e933db27fec6b8d3a | |
parent | 7fe9ab77464bb96f3d568356a758b11850f7ad06 [diff] |
mpeg4/av1: Update the minimum size check for AV1 CSD The minimum required size for the AV1C box is 4 (and not 5). The specification can be found in [1]. The sum of all the mandatory fields in the AV1C box will fit in 4 bytes (note that the "configObu" entries are optional). The similar check that we do in Utils.cpp [2] when parsing AV1C is correct. This bug was introduced in ag/6271779. [1] https://aomediacodec.github.io/av1-isobmff/#av1codecconfigurationbox-syntax [2] https://cs.android.com/android/platform/superproject/+/master:frameworks/av/media/libstagefright/Utils.cpp;l=644 Bug: 141654151 Test: AV1 media cts tests still pass Change-Id: Iadc9a1b643d55ac88dd00c4e0face05f464e8d05