diff mbox series

[FFmpeg-devel,6/7] lavc/webvttenc: return more meaningful error codes

Message ID 20200410190223.134413-7-jstebbins@jetheaddev.com
State Accepted
Commit 5fcf620eeeab6d99f18a78965748ca2d2ebab7b6
Headers show
Series [FFmpeg-devel,1/7] lavc/dvbsub: fix potential encode buffer overflow | expand

Checks

Context Check Description
andriy/ffmpeg-patchwork success Make fate finished

Commit Message

John Stebbins April 10, 2020, 7:02 p.m. UTC
---
 libavcodec/webvttenc.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff mbox series

Patch

diff --git a/libavcodec/webvttenc.c b/libavcodec/webvttenc.c
index c84bbf4b4e..febf6ee370 100644
--- a/libavcodec/webvttenc.c
+++ b/libavcodec/webvttenc.c
@@ -168,7 +168,7 @@  static int webvtt_encode_frame(AVCodecContext *avctx,
 
         if (sub->rects[i]->type != SUBTITLE_ASS) {
             av_log(avctx, AV_LOG_ERROR, "Only SUBTITLE_ASS type supported.\n");
-            return AVERROR(ENOSYS);
+            return AVERROR(EINVAL);
         }
 
 #if FF_API_ASS_TIMING
@@ -200,7 +200,7 @@  static int webvtt_encode_frame(AVCodecContext *avctx,
 
     if (s->buffer.len > bufsize) {
         av_log(avctx, AV_LOG_ERROR, "Buffer too small for ASS event.\n");
-        return -1;
+        return AVERROR_BUFFER_TOO_SMALL;
     }
     memcpy(buf, s->buffer.str, s->buffer.len);