[SCM] libav/experimental: h263dec: Fix order of initialization

siretart at users.alioth.debian.org siretart at users.alioth.debian.org
Sat Aug 30 15:49:52 UTC 2014


The following commit has been merged in the experimental branch:
commit b869eea7ea8f5d8331fcd6355f848bb6a6e06b14
Author: John Stebbins <stebbins at jetheaddev.com>
Date:   Fri Aug 8 08:21:02 2014 -0700

    h263dec: Fix order of initialization
    
    ff_MPV_common_init requires the frame dimensions which get parsed in
    *_decode_picture_header.

diff --git a/libavcodec/h263dec.c b/libavcodec/h263dec.c
index a6d16b7..c4db99e 100644
--- a/libavcodec/h263dec.c
+++ b/libavcodec/h263dec.c
@@ -416,21 +416,9 @@ int ff_h263_decode_frame(AVCodecContext *avctx, void *data, int *got_frame,
     if (ret < 0)
         return ret;
 
-    if (!s->context_initialized) {
+    if (!s->context_initialized)
         // we need the idct permutaton for reading a custom matrix
         ff_mpv_idct_init(s);
-        if ((ret = ff_MPV_common_init(s)) < 0)
-            return ret;
-    }
-
-    /* We need to set current_picture_ptr before reading the header,
-     * otherwise we cannot store anyting in there */
-    if (s->current_picture_ptr == NULL || s->current_picture_ptr->f->data[0]) {
-        int i = ff_find_unused_picture(s, 0);
-        if (i < 0)
-            return i;
-        s->current_picture_ptr = &s->picture[i];
-    }
 
     /* let's go :-) */
     if (CONFIG_WMV2_DECODER && s->msmpeg4_version == 5) {
@@ -465,6 +453,17 @@ int ff_h263_decode_frame(AVCodecContext *avctx, void *data, int *got_frame,
         return ret;
     }
 
+    if (!s->context_initialized)
+        if ((ret = ff_MPV_common_init(s)) < 0)
+            return ret;
+
+    if (s->current_picture_ptr == NULL || s->current_picture_ptr->f->data[0]) {
+        int i = ff_find_unused_picture(s, 0);
+        if (i < 0)
+            return i;
+        s->current_picture_ptr = &s->picture[i];
+    }
+
     avctx->has_b_frames = !s->low_delay;
 
 #define SET_QPEL_FUNC(postfix1, postfix2)                           \

-- 
Libav/FFmpeg packaging



More information about the pkg-multimedia-commits mailing list