[SCM] libav/experimental: In mpegts "reserved_future_use" field must be set to 1 in SDT table

siretart at users.alioth.debian.org siretart at users.alioth.debian.org
Sun Jun 30 17:22:53 UTC 2013


The following commit has been merged in the experimental branch:
commit a7827a17c6b3388322350456d445c94b3a82cd25
Author: Georgi Chorbadzhiyski <gf at unixsol.org>
Date:   Fri Jan 28 18:28:20 2011 +0200

    In mpegts "reserved_future_use" field must be set to 1 in SDT table
    
    According to EN 300 468 section 3.1 (Definitions):
    
       Unless otherwise specified within the present document all
       "reserved_future_use" bits is set to "1".
    
    This was not the case for SDT generation so this patch fixes it.
    
    Signed-off-by: Janne Grunau <janne-ffmpeg at jannau.net>

diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c
index 3c56830..6f35d6f 100644
--- a/libavformat/mpegtsenc.c
+++ b/libavformat/mpegtsenc.c
@@ -128,6 +128,8 @@ static int mpegts_write_section1(MpegTSSection *s, int tid, int id,
 {
     uint8_t section[1024], *q;
     unsigned int tot_len;
+    /* reserved_future_use field must be set to 1 for SDT */
+    unsigned int flags = tid == SDT_TID ? 0xf000 : 0xb000;
 
     tot_len = 3 + 5 + len + 4;
     /* check if not too big */
@@ -136,7 +138,7 @@ static int mpegts_write_section1(MpegTSSection *s, int tid, int id,
 
     q = section;
     *q++ = tid;
-    put16(&q, 0xb000 | (len + 5 + 4)); /* 5 byte header + 4 byte CRC */
+    put16(&q, flags | (len + 5 + 4)); /* 5 byte header + 4 byte CRC */
     put16(&q, id);
     *q++ = 0xc1 | (version << 1); /* current_next_indicator = 1 */
     *q++ = sec_num;

-- 
Libav/FFmpeg packaging



More information about the pkg-multimedia-commits mailing list