[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