On Aug 3, 2014 2:42 PM, "Luca Barbato" <[email protected]> wrote:
>
> The specification says the value is expressed in 10bit including,
> the 4bytes crc.
> ---
>
> Now with the correct value, hopefully.
>
>  libavformat/mpegtsenc.c | 10 +++++++---
>  1 file changed, 7 insertions(+), 3 deletions(-)
>
> diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c
> index 838702e..b297280 100644
> --- a/libavformat/mpegtsenc.c
> +++ b/libavformat/mpegtsenc.c
> @@ -89,6 +89,10 @@ typedef struct MpegTSWrite {
>  #define DEFAULT_PES_HEADER_FREQ 16
>  #define DEFAULT_PES_PAYLOAD_SIZE ((DEFAULT_PES_HEADER_FREQ - 1) * 184 +
170)
>

> +/* the section_lenght is 12bit, first 2 set to 0, the remaining 10bit

s/ht/th/

Also in commit message.

> + * should not exceed 1021. */
> +#define SECTION_LENGTH 1020
> +
>  /* NOTE: 4 bytes must be left at the end for the crc32 */
>  static void mpegts_write_section(MpegTSSection *s, uint8_t *buf, int len)
>  {
> @@ -201,7 +205,7 @@ static void mpegts_write_pat(AVFormatContext *s)
>  {
>      MpegTSWrite *ts = s->priv_data;
>      MpegTSService *service;
> -    uint8_t data[1012], *q;
> +    uint8_t data[SECTION_LENGTH], *q;
>      int i;
>
>      q = data;
> @@ -217,7 +221,7 @@ static void mpegts_write_pat(AVFormatContext *s)
>  static void mpegts_write_pmt(AVFormatContext *s, MpegTSService *service)
>  {
>      MpegTSWrite *ts = s->priv_data;
> -    uint8_t data[1012], *q, *desc_length_ptr, *program_info_length_ptr;
> +    uint8_t data[SECTION_LENGTH], *q, *desc_length_ptr,
*program_info_length_ptr;
>      int val, stream_type, i;
>
>      q = data;
> @@ -372,7 +376,7 @@ static void mpegts_write_sdt(AVFormatContext *s)
>  {
>      MpegTSWrite *ts = s->priv_data;
>      MpegTSService *service;
> -    uint8_t data[1012], *q, *desc_list_len_ptr, *desc_len_ptr;
> +    uint8_t data[SECTION_LENGTH], *q, *desc_list_len_ptr, *desc_len_ptr;
>      int i, running_status, free_ca_mode, val;
>
>      q = data;
> --
> 1.9.0
>
> _______________________________________________
> libav-devel mailing list
> [email protected]
> https://lists.libav.org/mailman/listinfo/libav-devel
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to