]> git.sesse.net Git - ffmpeg/blobdiff - libavcodec/h264_sei.c
vdpau: Add missing deprecation guards
[ffmpeg] / libavcodec / h264_sei.c
index 361d4deb1adb55414f180b01ee9c4469d22a9798..1fb1fc5db4ffc0784f2d9ba603ab036a3d1a3eb9 100644 (file)
@@ -171,7 +171,7 @@ static int decode_registered_user_data_closed_caption(H264Context *h, int size)
         }
     } else {
         int i;
-        avpriv_request_sample("Subtitles with data type 0x%02x",
+        avpriv_request_sample(h->avctx, "Subtitles with data type 0x%02x",
                               user_data_type_code);
         for (i = 0; i < size - 1; i++)
             skip_bits(&h->gb, 8);
@@ -215,13 +215,17 @@ static int decode_registered_user_data(H264Context *h, int size)
 
 static int decode_unregistered_user_data(H264Context *h, int size)
 {
-    uint8_t user_data[16 + 256];
+    uint8_t *user_data;
     int e, build, i;
 
-    if (size < 16)
+    if (size < 16 || size >= INT_MAX - 16)
         return AVERROR_INVALIDDATA;
 
-    for (i = 0; i < sizeof(user_data) - 1 && i < size; i++)
+    user_data = av_malloc(16 + size + 1);
+    if (!user_data)
+        return AVERROR(ENOMEM);
+
+    for (i = 0; i < size + 16; i++)
         user_data[i] = get_bits(&h->gb, 8);
 
     user_data[i] = 0;
@@ -229,12 +233,10 @@ static int decode_unregistered_user_data(H264Context *h, int size)
     if (e == 1 && build > 0)
         h->x264_build = build;
 
-    if (h->avctx->debug & FF_DEBUG_BUGS)
+    if (strlen(user_data + 16) > 0)
         av_log(h->avctx, AV_LOG_DEBUG, "user data:\"%s\"\n", user_data + 16);
 
-    for (; i < size; i++)
-        skip_bits(&h->gb, 8);
-
+    av_free(user_data);
     return 0;
 }