[FFmpeg-devel] [PATCH] lavc/vvc: Stricter bound on pps_exp_slice_height_in_ctus_minus1

[FFmpeg-devel] [PATCH] lavc/vvc: Stricter bound on pps_exp_slice_height_in_ctus_minus1

[Frank Plowman]

When pps_num_exp_slices_in_tile[i] is nonzero, the ith tile is made up
of pps_num_exp_slices_in_tile[i] slices stacked atop one another, where
the height of the jth slice in the ith tile is given by
pps_exp_slice_height_in_ctus_minus1[i][j].  The sum of the heights of
the slices in the tile should not exceed the height of the tile itself.

Signed-off-by: Frank Plowman <post@frankplowman.com>
---
 libavcodec/cbs_h266_syntax_template.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavcodec/cbs_h266_syntax_template.c b/libavcodec/cbs_h266_syntax_template.c
index b84f3e29db..9f3ad0bb67 100644
--- a/libavcodec/cbs_h266_syntax_template.c
+++ b/libavcodec/cbs_h266_syntax_template.c
@@ -2020,7 +2020,7 @@ static int FUNC(pps) (CodedBitstreamContext *ctx, RWContext *rw,
                         for (j = 0; j < current->pps_num_exp_slices_in_tile[i];
                              j++) {
                             ues(pps_exp_slice_height_in_ctus_minus1[i][j], 0,
-                                current->row_height_val[tile_y] - 1, 2,
+                                remaining_height_in_ctbs_y - 1, 2,
                                 i, j);
                             slice_height_in_ctus =
                                 current->
-- 
2.47.0

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] lavc/vvc: Stricter bound on pps_exp_slice_height_in_ctus_minus1

[Nuo Mi]

On Sun, Mar 2, 2025 at 8:13 PM Frank Plowman <post@frankplowman.com> wrote:

> When pps_num_exp_slices_in_tile[i] is nonzero, the ith tile is made up
> of pps_num_exp_slices_in_tile[i] slices stacked atop one another, where
> the height of the jth slice in the ith tile is given by
> pps_exp_slice_height_in_ctus_minus1[i][j].  The sum of the heights of
> the slices in the tile should not exceed the height of the tile itself.
>
> Signed-off-by: Frank Plowman <post@frankplowman.com>
>
Applied.
Thank you, Frank

> ---
>  libavcodec/cbs_h266_syntax_template.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavcodec/cbs_h266_syntax_template.c
> b/libavcodec/cbs_h266_syntax_template.c
> index b84f3e29db..9f3ad0bb67 100644
> --- a/libavcodec/cbs_h266_syntax_template.c
> +++ b/libavcodec/cbs_h266_syntax_template.c
> @@ -2020,7 +2020,7 @@ static int FUNC(pps) (CodedBitstreamContext *ctx,
> RWContext *rw,
>                          for (j = 0; j <
> current->pps_num_exp_slices_in_tile[i];
>                               j++) {
>
>  ues(pps_exp_slice_height_in_ctus_minus1[i][j], 0,
> -                                current->row_height_val[tile_y] - 1, 2,
> +                                remaining_height_in_ctbs_y - 1, 2,
>                                  i, j);
>                              slice_height_in_ctus =
>                                  current->
> --
> 2.47.0
>
>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".