avcodec/mpeg4_unpack_bframes_bsf: Copy packet props
authorJan Sebechlebsky <sebechlebskyjan@gmail.com>
Fri, 22 Jul 2016 15:59:29 +0000 (17:59 +0200)
committerMichael Niedermayer <michael@niedermayer.cc>
Fri, 22 Jul 2016 16:55:37 +0000 (18:55 +0200)
mpeg4_unpack_bframes_bsf bitstream filters constructs
resulting packet using av_packet_from_data() function.
This function however modifies only buffer (data) and leaves
other fields untouched, so the content of other fields
of the output packet is undefined.
It is working with old BSF API, since old API filters
just data and the packet fields are copied in
av_apply_bitstream_filters from input packet.

This change fixes the behaviour for the new BSF API.

Signed-off-by: Jan Sebechlebsky <sebechlebskyjan@gmail.com>
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
libavcodec/mpeg4_unpack_bframes_bsf.c

index 0615621..aee8ccb 100644 (file)
@@ -120,6 +120,12 @@ static int mpeg4_unpack_bframes_filter(AVBSFContext *ctx, AVPacket *out)
 
     if (nb_vop == 1 && s->b_frame_buf) {
         /* use frame from BSFContext */
+        ret = av_packet_copy_props(out, in);
+        if (ret < 0) {
+            av_packet_free(&in);
+            return ret;
+        }
+
         av_packet_from_data(out, s->b_frame_buf, s->b_frame_buf_size);
         if (in->size <= MAX_NVOP_SIZE) {
             /* N-VOP */