diff mbox series

[FFmpeg-devel,5/7] lavc/srtenc: return more meaninful error codes

Message ID 20200410190223.134413-6-jstebbins@jetheaddev.com
State Accepted
Commit 9ac715a36783af7d52e23463bcb8ce2c502ca6e7
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/srtenc.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff mbox series

Patch

diff --git a/libavcodec/srtenc.c b/libavcodec/srtenc.c
index 34f0f0d5e6..655b65679e 100644
--- a/libavcodec/srtenc.c
+++ b/libavcodec/srtenc.c
@@ -241,7 +241,7 @@  static int 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
@@ -276,7 +276,7 @@  static int 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);