avcodec/subviewerdec: add some memory checks
authorClément Bœsch <u@pkh.me>
Sat, 20 Sep 2014 20:17:29 +0000 (22:17 +0200)
committerClément Bœsch <u@pkh.me>
Sun, 21 Sep 2014 16:41:45 +0000 (18:41 +0200)
libavcodec/subviewerdec.c

index 63be418..b41f7a9 100644 (file)
@@ -51,6 +51,7 @@ static int subviewer_decode_frame(AVCodecContext *avctx,
                                   void *data, int *got_sub_ptr, AVPacket *avpkt)
 {
     char c;
+    int ret = 0;
     AVSubtitle *sub = data;
     const char *ptr = avpkt->data;
     AVBPrint buf;
@@ -67,9 +68,11 @@ static int subviewer_decode_frame(AVCodecContext *avctx,
     // note: no need to rescale pts & duration since they are in the same
     // timebase as ASS (1/100)
     if (ptr && avpkt->size > 0 && !subviewer_event_to_ass(&buf, ptr))
-        ff_ass_add_rect(sub, buf.str, avpkt->pts, avpkt->duration, 0);
-    *got_sub_ptr = sub->num_rects > 0;
+        ret = ff_ass_add_rect_bprint(sub, &buf, avpkt->pts, avpkt->duration, 0);
     av_bprint_finalize(&buf, NULL);
+    if (ret < 0)
+        return ret;
+    *got_sub_ptr = sub->num_rects > 0;
     return avpkt->size;
 }