[FFmpeg-devel] Fix for KLV in mpegts

[Artyom Lebedev]

This fixes bug which prevents from proper muxing-in KLV stream into mpeg-ts.

mpegtsenc.c:1526

     char *side_data = NULL;
     int stream_id = -1;

     side_data = av_packet_get_side_data(pkt,
AV_PKT_DATA_MPEGTS_STREAM_ID,
                                         &side_data_size);
     if (side_data)
         stream_id = side_data[0];

One-byte stream ID is read from "char *" array to integer making it to 
sign-extend which is not correct. Although it writes it correctly to 
stream, since it is treated as one byte again, but it fails with some 
condition checks, e.g. in mpegtsenc.c:1278:

if (stream_id == 0xbd) /* asynchronous KLV */
     pts = dts = AV_NOPTS_VALUE;
stream_id value in such case is 0xffffffbd.

Fix should be changing side_data type from "char *" to "uint8_t *".


-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Fix-bug-in-mpegts-muxer-which-affects-KLV-async-stre.patch
Type: text/x-patch
Size: 800 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20181207/abc33be1/attachment.bin>