[FFmpeg-devel] [PATCH 17/20] avcodec/snow: Remove outdated assert

Andreas Rheinhardt andreas.rheinhardt at outlook.com
Sat Mar 8 21:28:18 EET 2025


Andreas Rheinhardt:
> It comes from a time before frames were refcounted; it has indeed been
> added in a follow-up commit to c13e490dce1a66d79e1f053d8a38fb9b2eb53267:
> "codec_release_buffer: fix handling of non user buffers". This type
> of check is obsolete now.
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
> ---
>  libavcodec/snow.c | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/libavcodec/snow.c b/libavcodec/snow.c
> index 7e93005492..dcd6d67ad5 100644
> --- a/libavcodec/snow.c
> +++ b/libavcodec/snow.c
> @@ -607,9 +607,6 @@ av_cold void ff_snow_common_end(SnowContext *s)
>      av_freep(&s->scratchbuf);
>  
>      for(i=0; i<MAX_REF_FRAMES; i++){
> -        if(s->last_picture[i] && s->last_picture[i]->data[0]) {
> -            av_assert0(s->last_picture[i]->data[0] != s->current_picture->data[0]);
> -        }
>          av_frame_free(&s->last_picture[i]);
>      }
>  

Will apply this patch tomorrow unless there are objections.

- Andreas



More information about the ffmpeg-devel mailing list