Re: [FFmpeg-devel] [PATCH v2 2/2] avcodec/av1dec: Return error for unsupported tile list OBU

2024-02-25 Thread Xiang, Haihao
On Do, 2023-12-28 at 09:24 +0800, fei.w.wang-at-intel@ffmpeg.org wrote:
> From: Fei Wang 
> 
> Otherwise decoding maybe successful but output result is incorrect.
> 
> Signed-off-by: Fei Wang 
> ---
>  libavcodec/av1dec.c | 6 ++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/libavcodec/av1dec.c b/libavcodec/av1dec.c
> index 198fe75aa0..41746b1f11 100644
> --- a/libavcodec/av1dec.c
> +++ b/libavcodec/av1dec.c
> @@ -1202,6 +1202,12 @@ static int av1_receive_frame_internal(AVCodecContext
> *avctx, AVFrame *frame)
>  
>  av_log(avctx, AV_LOG_DEBUG, "OBU idx:%d, type:%d, content
> available:%d.\n", i, unit->type, !!obu);
>  
> +    if (unit->type == AV1_OBU_TILE_LIST) {
> +    av_log(avctx, AV_LOG_ERROR, "Large scale tile decoding is
> unsupported.\n");
> +    ret = AVERROR_PATCHWELCOME;
> +    goto end;
> +    }
> +
>  if (!obu)
>  continue;
>  

Patchset LGTM, I'll merge them if there are no objections

Thanks
Haihao


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

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


[FFmpeg-devel] [PATCH v2 2/2] avcodec/av1dec: Return error for unsupported tile list OBU

2023-12-27 Thread fei . w . wang-at-intel . com
From: Fei Wang 

Otherwise decoding maybe successful but output result is incorrect.

Signed-off-by: Fei Wang 
---
 libavcodec/av1dec.c | 6 ++
 1 file changed, 6 insertions(+)

diff --git a/libavcodec/av1dec.c b/libavcodec/av1dec.c
index 198fe75aa0..41746b1f11 100644
--- a/libavcodec/av1dec.c
+++ b/libavcodec/av1dec.c
@@ -1202,6 +1202,12 @@ static int av1_receive_frame_internal(AVCodecContext 
*avctx, AVFrame *frame)
 
 av_log(avctx, AV_LOG_DEBUG, "OBU idx:%d, type:%d, content 
available:%d.\n", i, unit->type, !!obu);
 
+if (unit->type == AV1_OBU_TILE_LIST) {
+av_log(avctx, AV_LOG_ERROR, "Large scale tile decoding is 
unsupported.\n");
+ret = AVERROR_PATCHWELCOME;
+goto end;
+}
+
 if (!obu)
 continue;
 
-- 
2.25.1

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

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