| Message ID | 20191006050120.26807-3-andreas.rheinhardt@gmail.com |
|---|---|
| State | Accepted |
| Commit | 5e546864b09379910721b35a14713982d933d9dd |
| Headers | show |
diff --git a/libavcodec/flac_parser.c b/libavcodec/flac_parser.c index 2721286464..5d0705ce63 100644 --- a/libavcodec/flac_parser.c +++ b/libavcodec/flac_parser.c @@ -221,7 +221,7 @@ static int find_headers_search(FLACParseContext *fpc, uint8_t *buf, int buf_size } for (; i < buf_size - 1; i += 4) { - x = AV_RB32(buf + i); + x = AV_RN32(buf + i); if (((x & ~(x + 0x01010101)) & 0x80808080)) { for (j = 0; j < 4; j++) { if ((AV_RB16(buf + i + j) & 0xFFFE) == 0xFFF8)
FLAC sync codes contain a byte equal to 0xFF and so the function that searches for sync codes first searched for this byte. It did this by checking four bytes at once; these bytes have been read via AV_RB32, but the test works just as well with native endianness. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> --- libavcodec/flac_parser.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)