[FFmpeg-devel] [PATCH] dxva2_hevc: don't use frames as reference if they are not marked as such

[Hendrik Leppkes]

On Wed, Mar 23, 2022 at 2:54 PM Steve Lhomme <robux4 at ycbcr.xyz> wrote:
>
> Similar to how a frame is considered for referencing for the RefPicList array.
> This will do the same for RefPicSetStCurrBefore, RefPicSetStCurrAfter and
> RefPicSetLtCurr.
>
> Fixes playback of http://www.gbbsoft.pl/!download/!/Film1.mp4
> Ref. VLC issue https://code.videolan.org/videolan/vlc/-/issues/26738
>
> Signed-off-by: Steve Lhomme <robux4 at ycbcr.xyz>
> ---
>  libavcodec/dxva2_hevc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavcodec/dxva2_hevc.c b/libavcodec/dxva2_hevc.c
> index c91bcf3eeb..6b239d9917 100644
> --- a/libavcodec/dxva2_hevc.c
> +++ b/libavcodec/dxva2_hevc.c
> @@ -184,7 +184,7 @@ static void fill_picture_parameters(const AVCodecContext *avctx, AVDXVAContext *
>              const HEVCFrame *frame = NULL; \
>              while (!frame && j < rpl->nb_refs) \
>                  frame = rpl->ref[j++]; \
> -            if (frame) \
> +            if (frame && frame->flags & (HEVC_FRAME_FLAG_LONG_REF | HEVC_FRAME_FLAG_SHORT_REF)) \
>                  pp->ref_list[i] = get_refpic_index(pp, ff_dxva2_get_surface_index(avctx, ctx, frame->frame)); \
>              else \
>                  pp->ref_list[i] = 0xff; \
> --
> 2.29.2
>

LGTM