diff mbox series

[FFmpeg-devel,4/7] avcodec/vc2enc: Fix slice length

Message ID AS8P250MB074423DF71813D281F21FC2A8FF12@AS8P250MB0744.EURP250.PROD.OUTLOOK.COM
State Accepted
Commit 02ecf8d7f3fe6ad6bc493a37bfff65bfe9150113
Headers show
Series [FFmpeg-devel,1/7] avcodec/vc2enc: Avoid void* where possible | expand

Checks

Context Check Description
yinshiyou/make_loongarch64 success Make finished
yinshiyou/make_fate_loongarch64 success Make fate finished
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Andreas Rheinhardt May 28, 2024, 2:49 a.m. UTC 
args->bytes here already includes prefix_bytes (see
SSIZE_ROUND macro), so including it here again and
forgetting it when offsetting skip seems wrong.
This only works because prefix_bytes is currently
always zero in this encoder.
(This has been added in b88be742fac7a77a8095e8155ba8790db4b77568
without any reason.)

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 libavcodec/vc2enc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/libavcodec/vc2enc.c b/libavcodec/vc2enc.c
index f39f368181..3285218724 100644
--- a/libavcodec/vc2enc.c
+++ b/libavcodec/vc2enc.c
@@ -788,7 +788,7 @@  static int encode_slices(VC2EncContext *s)
     for (slice_y = 0; slice_y < s->num_y; slice_y++) {
         for (slice_x = 0; slice_x < s->num_x; slice_x++) {
             SliceArgs *args = &enc_args[s->num_x*slice_y + slice_x];
-            init_put_bits(&args->pb, buf + skip, args->bytes+s->prefix_bytes);
+            init_put_bits(&args->pb, buf + skip, args->bytes);
             skip += args->bytes;
         }
     }