diff options
author | Jean-Michel Trivi <jmtrivi@google.com> | 2011-08-29 14:06:51 -0700 |
---|---|---|
committer | Jean-Michel Trivi <jmtrivi@google.com> | 2011-08-29 18:56:55 -0700 |
commit | 59fe406b3a475648828d4e44ca47427fa4829677 (patch) | |
tree | d5296976a92e721548c1784898a4634cc3a1ab8e /docs/html/sdk/api_diff/14/changes | |
parent | 890be6b919ce0c65d018a2dc40a6a1d59bc2b14d (diff) |
Bug 5156756 Fix AAC ADTS header size computation
This previous frame size computation code had two incorrect
assumptions:
1/ the ADTS frame length value already contains the header size
2/ the ADTS header is not of fixed size: it is 2 bytes bigger
if it contains the CRC value.
For 1/, the code worked because when the header size was added
to the frame size, the + operator is of stronger precedence
than the ? operator, so the size added was always 0 (instead
of ADTS_HEADER_LENGTH.
For 2/, the code worked as long as there was no CRC in the ADTS
data.
The fix consists in:
- documenting what the frame length computation code returns
- fixing the frame length computation
- when computing the frame length, also returning the header
size so the correct data can be sent to the decoder.
Change-Id: I92df72a9e531f594f762e63d62f9dee7b0109904
Diffstat (limited to 'docs/html/sdk/api_diff/14/changes')
0 files changed, 0 insertions, 0 deletions