* [FFmpeg-devel] [PATCH] lavc/vaapi_decode: Reject decoding of frames with no slices
@ 2024-05-10 9:55 David Rosca
2024-05-13 6:43 ` Xiang, Haihao
0 siblings, 1 reply; 2+ messages in thread
From: David Rosca @ 2024-05-10 9:55 UTC (permalink / raw)
To: ffmpeg-devel; +Cc: David Rosca
Matches other hwaccels.
---
libavcodec/vaapi_decode.c | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/libavcodec/vaapi_decode.c b/libavcodec/vaapi_decode.c
index 8e9f647c20..3c4030c073 100644
--- a/libavcodec/vaapi_decode.c
+++ b/libavcodec/vaapi_decode.c
@@ -157,6 +157,11 @@ int ff_vaapi_decode_issue(AVCodecContext *avctx,
VAStatus vas;
int err;
+ if (pic->nb_slices <= 0) {
+ err = -1;
+ goto fail;
+ }
+
av_log(avctx, AV_LOG_DEBUG, "Decode to surface %#x.\n",
pic->output_surface);
--
2.45.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".
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [FFmpeg-devel] [PATCH] lavc/vaapi_decode: Reject decoding of frames with no slices
2024-05-10 9:55 [FFmpeg-devel] [PATCH] lavc/vaapi_decode: Reject decoding of frames with no slices David Rosca
@ 2024-05-13 6:43 ` Xiang, Haihao
0 siblings, 0 replies; 2+ messages in thread
From: Xiang, Haihao @ 2024-05-13 6:43 UTC (permalink / raw)
To: ffmpeg-devel; +Cc: nowrep
On Vr, 2024-05-10 at 11:55 +0200, David Rosca wrote:
> Matches other hwaccels.
> ---
> libavcodec/vaapi_decode.c | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/libavcodec/vaapi_decode.c b/libavcodec/vaapi_decode.c
> index 8e9f647c20..3c4030c073 100644
> --- a/libavcodec/vaapi_decode.c
> +++ b/libavcodec/vaapi_decode.c
> @@ -157,6 +157,11 @@ int ff_vaapi_decode_issue(AVCodecContext *avctx,
> VAStatus vas;
> int err;
>
> + if (pic->nb_slices <= 0) {
> + err = -1;
Better to return a valid averror code, e.g. AVERROR(EINVAL).
Thanks
Haihao
> + goto fail;
> + }
> +
> av_log(avctx, AV_LOG_DEBUG, "Decode to surface %#x.\n",
> pic->output_surface);
>
_______________________________________________
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".
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2024-05-13 6:43 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-05-10 9:55 [FFmpeg-devel] [PATCH] lavc/vaapi_decode: Reject decoding of frames with no slices David Rosca
2024-05-13 6:43 ` Xiang, Haihao
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
This inbox may be cloned and mirrored by anyone:
git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
ffmpegdev@gitmailbox.com
public-inbox-index ffmpegdev
Example config snippet for mirrors.
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git