diff mbox series

[FFmpeg-devel,8/8] avpriv_find_start_code(): reduce the number of iterations

Message ID 20220201212056.29712-9-scott.the.elm@gmail.com
State New
Headers show
Series rewrite avpriv_find_start_code() for clarity | expand

Checks

Context Check Description
andriy/make_aarch64_jetson success Make finished
andriy/make_fate_aarch64_jetson fail Make fate failed
andriy/make_x86 success Make finished
andriy/make_fate_x86 fail Make fate failed
andriy/make_ppc success Make finished
andriy/make_fate_ppc fail Make fate failed
andriy/make_armv7_RPi4 success Make finished
andriy/make_fate_armv7_RPi4 fail Make fate failed

Commit Message

Scott Theisen Feb. 1, 2022, 9:20 p.m. UTC 
by correctly starting with three new bytes on the next iteration,
instead of keeping byte p[-3] which is invalid, i.e. known to be 01
when it must be 00.
---
 libavcodec/utils.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/libavcodec/utils.c b/libavcodec/utils.c
index b4c5fa5009..d485d0c96b 100644
--- a/libavcodec/utils.c
+++ b/libavcodec/utils.c
@@ -965,7 +965,7 @@  const uint8_t *avpriv_find_start_code(const uint8_t *av_restrict p,
         // i.e. p[-1] == 0 is equivalent to p[-1] < 1
 
         // UU UU 01
-        else if (p[-2] != 0) p += 2;    // we have UU YY 01, so increment by 2
+        else if (p[-2] != 0) p += 3;    // we have UU YY 01, so increment by 3
                                         // to start check over with 3 new bytes
         // UU 00 01
         else if (p[-3] != 0) p += 3;    // we have YY 00 01, so increment by 3