[FFmpeg-cvslog] avcodec/options: avcodec_copy_context() Check subtitle_header_size instead of setting it

Michael Niedermayer git at videolan.org
Thu May 15 19:49:48 CEST 2014


ffmpeg | branch: master | Michael Niedermayer <michaelni at gmx.at> | Thu May 15 19:41:38 2014 +0200| [97f856a4c27129f99f2c7a41f7eb33a0d86917f8] | committer: Michael Niedermayer

avcodec/options: avcodec_copy_context() Check subtitle_header_size instead of setting it

The value is already copied in the generic code so it should always match

Signed-off-by: Michael Niedermayer <michaelni at gmx.at>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=97f856a4c27129f99f2c7a41f7eb33a0d86917f8
---

 libavcodec/options.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavcodec/options.c b/libavcodec/options.c
index 88b3b12..b31483b 100644
--- a/libavcodec/options.c
+++ b/libavcodec/options.c
@@ -207,7 +207,7 @@ int avcodec_copy_context(AVCodecContext *dest, const AVCodecContext *src)
     alloc_and_copy_or_fail(inter_matrix, 64 * sizeof(int16_t), 0);
     alloc_and_copy_or_fail(rc_override,  src->rc_override_count * sizeof(*src->rc_override), 0);
     alloc_and_copy_or_fail(subtitle_header, src->subtitle_header_size, 1);
-    dest->subtitle_header_size = src->subtitle_header_size;
+    av_assert0(dest->subtitle_header_size == src->subtitle_header_size);
 #undef alloc_and_copy_or_fail
 
     return 0;



More information about the ffmpeg-cvslog mailing list