Wireshark-dev: Re: [Wireshark-dev] [PATCH] Further improvements to the MPEG decoder
From: "Shaun Jackman" <sjackman@xxxxxxxxx>
Date: Sun, 19 Aug 2007 11:06:32 -0700
Please apply or provide comments.
Thanks,
Shaun
On 8/9/07, Shaun Jackman <sjackman@xxxxxxxxx> wrote:
> This patch adds SCR, PTS, and DTS decoding to the MPEG PES decoder.
>
> Cheers,
> Shaun
>
> 2007-08-09 Shaun Jackman <sjackman@xxxxxxxxx>
>
> * asn1/mpeg/packet-mpeg-pes-template.c: Improved decoding of PES
> extension header and Pack header. Decode SCR, PTS, and DTS.
> * asn1/mpeg/mpeg-pes.asn (Pack): Remove.
> * epan/dissectors/packet-mpeg-pes.c: Regenerate.
> * wiretap/mpeg.c (mpeg_read): Decode the SCR using integer
> arithmetic instead of double float arithmetic to prevent rounding
> error.
> * wiretap/wtap-int.h (mpeg_t) <t0>: Use time_t instead of double.
>
> Index: asn1/mpeg/packet-mpeg-pes-template.c
> ===================================================================
> --- asn1/mpeg/packet-mpeg-pes-template.c (revision 22479)
> +++ asn1/mpeg/packet-mpeg-pes-template.c (working copy)
> @@ -43,12 +43,30 @@
>
> static int proto_mpeg = -1;
> static int proto_mpeg_pes = -1;
> +
> +static int ett_mpeg_pes_pack_header = -1;
> +static int ett_mpeg_pes_header_data = -1;
> +
> static int hf_mpeg_pes_pack_header = -1;
> +static int hf_mpeg_pes_scr = -1;
> +static int hf_mpeg_pes_program_mux_rate = -1;
> +static int hf_mpeg_pes_stuffing_length = -1;
> static int hf_mpeg_pes_stuffing = -1;
> static int hf_mpeg_pes_extension = -1;
> static int hf_mpeg_pes_header_data = -1;
> +static int hf_mpeg_pes_pts = -1;
> +static int hf_mpeg_pes_dts = -1;
> +static int hf_mpeg_pes_escr = -1;
> +static int hf_mpeg_pes_es_rate = -1;
> +static int hf_mpeg_pes_copy_info = -1;
> +static int hf_mpeg_pes_crc = -1;
> +static int hf_mpeg_pes_extension_flags = -1;
> +static int hf_mpeg_pes_private_data = -1;
> +static int hf_mpeg_pes_pack_length = -1;
> +static int hf_mpeg_pes_sequence = -1;
> +static int hf_mpeg_pes_pstd_buffer = -1;
> +static int hf_mpeg_pes_extension2 = -1;
> static int hf_mpeg_pes_padding = -1;
> -
> static int hf_mpeg_pes_data = -1;
>
> static int hf_mpeg_video_sequence_header = -1;
> @@ -59,6 +77,7 @@
> static int hf_mpeg_video_data = -1;
>
> enum { PES_PREFIX = 1 };
> +
> enum {
> STREAM_PICTURE = 0x00,
> STREAM_SEQUENCE = 0xb3,
> @@ -75,6 +94,204 @@
> STREAM_VIDEO = 0xe0
> };
>
> +enum {
> + PTS_FLAG = 0x80,
> + DTS_FLAG = 0x40,
> + ESCR_FLAG = 0x20,
> + ES_RATE_FLAG = 0x10,
> + DSM_TRICK_MODE_FLAG = 0x08,
> + COPY_INFO_FLAG = 0x04,
> + CRC_FLAG = 0x02,
> + EXTENSION_FLAG = 0x01,
> +};
> +
> +enum {
> + PRIVATE_DATA_FLAG = 0x80,
> + PACK_LENGTH_FLAG = 0x40,
> + SEQUENCE_FLAG = 0x20,
> + PSTD_BUFFER_FLAG = 0x10,
> + MUST_BE_ONES = 0x07,
> + EXTENSION_FLAG2 = 0x01,
> +};
> +
> +static guint64 tvb_get_ntoh40(tvbuff_t *tvb, unsigned offset)
> +{
> + return (guint64)tvb_get_guint8(tvb, offset) << 32
> + | tvb_get_ntohl(tvb, offset + 1);
> +}
> +
> +static guint64 tvb_get_ntoh48(tvbuff_t *tvb, unsigned offset)
> +{
> + return (guint64)tvb_get_ntohs(tvb, offset) << 32
> + | tvb_get_ntohl(tvb, offset + 2);
> +}
> +
> +#define TSHZ 90000
> +
> +static guint64 decode_time_stamp(tvbuff_t *tvb, unsigned offset, nstime_t *nst)
> +{
> + guint64 bytes = tvb_get_ntoh40(tvb, offset);
> + guint64 ts =
> + (bytes >> 33 & 0x0007) << 30 |
> + (bytes >> 17 & 0x7fff) << 15 |
> + (bytes >> 1 & 0x7fff) << 0;
> + unsigned rem = ts % TSHZ;
> + nst->secs = ts / TSHZ;
> + nst->nsecs = 1000000000LL * rem / TSHZ;
> + return ts;
> +}
> +
> +#define SCRHZ 27000000
> +
> +static guint64 decode_clock_reference(tvbuff_t *tvb, unsigned offset,
> + nstime_t *nst)
> +{
> + guint64 bytes = tvb_get_ntoh48(tvb, offset);
> + guint64 ts =
> + (bytes >> 43 & 0x0007) << 30 |
> + (bytes >> 27 & 0x7fff) << 15 |
> + (bytes >> 11 & 0x7fff) << 0;
> + unsigned ext = bytes >> 1 & 0x1ff;
> + guint64 cr = 300 * ts + ext;
> + unsigned rem = cr % SCRHZ;
> + nst->secs = cr / SCRHZ;
> + nst->nsecs = 1000000000LL * rem / SCRHZ;
> + return cr;
> +}
> +
> +static void
> +dissect_mpeg_pes_header_data(tvbuff_t *tvb, packet_info *pinfo,
> + proto_tree *root, unsigned flags)
> +{
> + proto_item *item = proto_tree_add_item(root, hf_mpeg_pes_header_data, tvb,
> + 0, -1, FALSE);
> + proto_tree *tree = proto_item_add_subtree(item, ett_mpeg_pes_header_data);
> +
> + unsigned offset = 0;
> + if (flags & PTS_FLAG) {
> + nstime_t nst;
> + decode_time_stamp(tvb, offset, &nst);
> + proto_tree_add_time(tree, hf_mpeg_pes_pts, tvb,
> + offset, 5, &nst);
> + offset += 5;
> +
> + if (check_col(pinfo->cinfo, COL_DEF_DST)) {
> + SET_ADDRESS(&pinfo->dst, AT_NONE, 0, NULL);
> + col_add_fstr(pinfo->cinfo, COL_DEF_DST,
> + "PTS %u.%09u", nst.secs, nst.nsecs);
> + }
> + }
> + if (flags & DTS_FLAG) {
> + nstime_t nst;
> + decode_time_stamp(tvb, offset, &nst);
> + proto_tree_add_time(tree, hf_mpeg_pes_dts, tvb,
> + offset, 5, &nst);
> + offset += 5;
> +
> + if (check_col(pinfo->cinfo, COL_DEF_SRC)) {
> + SET_ADDRESS(&pinfo->src, AT_NONE, 0, NULL);
> + col_add_fstr(pinfo->cinfo, COL_DEF_SRC,
> + "DTS %u.%09u", nst.secs, nst.nsecs);
> + }
> + }
> + if (flags & ESCR_FLAG) {
> + nstime_t nst;
> + decode_clock_reference(tvb, offset, &nst);
> + proto_tree_add_time(tree, hf_mpeg_pes_escr, tvb,
> + offset, 6, &nst);
> + offset += 6;
> + }
> + if (flags & ES_RATE_FLAG) {
> + unsigned es_rate = (tvb_get_ntohs(tvb, offset) >> 1 & 0x3fff) * 50;
> + proto_tree_add_uint(tree, hf_mpeg_pes_es_rate, tvb,
> + offset, 3, es_rate);
> + offset += 3;
> + }
> + if (flags & COPY_INFO_FLAG) {
> + proto_tree_add_item(tree, hf_mpeg_pes_copy_info, tvb,
> + offset, 1, FALSE);
> + offset++;
> + }
> + if (flags & CRC_FLAG) {
> + proto_tree_add_item(tree, hf_mpeg_pes_crc, tvb,
> + offset, 2, FALSE);
> + offset += 2;
> + }
> +
> + if (flags & EXTENSION_FLAG) {
> + int flags2 = tvb_get_guint8(tvb, offset);
> + proto_tree_add_item(tree, hf_mpeg_pes_extension_flags, tvb,
> + offset, 1, FALSE);
> + offset++;
> +
> + if (flags2 & PRIVATE_DATA_FLAG) {
> + proto_tree_add_item(tree, hf_mpeg_pes_private_data, tvb,
> + offset, 2, FALSE);
> + offset += 2;
> + }
> + if (flags2 & PACK_LENGTH_FLAG) {
> + proto_tree_add_item(tree, hf_mpeg_pes_pack_length, tvb,
> + offset, 1, FALSE);
> + offset++;
> + }
> + if (flags2 & SEQUENCE_FLAG) {
> + proto_tree_add_item(tree, hf_mpeg_pes_sequence, tvb,
> + offset, 2, FALSE);
> + offset += 2;
> + }
> + if (flags2 & PSTD_BUFFER_FLAG) {
> + unsigned pstd = tvb_get_ntohs(tvb, offset);
> + proto_tree_add_uint(tree, hf_mpeg_pes_pstd_buffer, tvb,
> + offset, 2, (pstd & 0x2000 ? 1024 : 128) * (pstd & 0x1ff));
> + offset += 2;
> + }
> + if (flags2 & EXTENSION_FLAG2) {
> + proto_tree_add_item(tree, hf_mpeg_pes_extension2, tvb,
> + offset, 2, FALSE);
> + offset += 2;
> + }
> + }
> +}
> +
> +static unsigned
> +dissect_mpeg_pes_pack_header(tvbuff_t *tvb, unsigned offset,
> + packet_info *pinfo, proto_tree *root)
> +{
> + unsigned program_mux_rate, stuffing_length;
> +
> + proto_item *item = proto_tree_add_item(root, hf_mpeg_pes_pack_header, tvb,
> + offset / 8, 10, FALSE);
> + proto_tree *tree = proto_item_add_subtree(item, ett_mpeg_pes_pack_header);
> +
> + nstime_t nst;
> + decode_clock_reference(tvb, offset / 8, &nst);
> + proto_tree_add_time(tree, hf_mpeg_pes_scr, tvb, offset / 8, 6, &nst);
> + offset += 6 * 8;
> +
> + program_mux_rate = (tvb_get_ntoh24(tvb, offset / 8) >> 2) * 50;
> + proto_tree_add_uint(tree, hf_mpeg_pes_program_mux_rate, tvb, offset / 8, 3,
> + program_mux_rate);
> + offset += 3 * 8;
> +
> + if (check_col(pinfo->cinfo, COL_DEF_SRC)) {
> + SET_ADDRESS(&pinfo->src, AT_NONE, 0, NULL);
> + col_add_fstr(pinfo->cinfo, COL_DEF_SRC, "%u B/s", program_mux_rate);
> + }
> +
> + stuffing_length = tvb_get_guint8(tvb, offset / 8) & 0x07;
> + proto_tree_add_item(tree, hf_mpeg_pes_stuffing_length, tvb,
> + offset / 8, 1, FALSE);
> + offset += 1 * 8;
> +
> + if (stuffing_length > 0) {
> + proto_tree_add_item(tree, hf_mpeg_pes_stuffing, tvb,
> + offset / 8, stuffing_length, FALSE);
> + offset += stuffing_length * 8;
> + }
> +
> + return offset;
> +}
> +
> void
> dissect_mpeg(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree);
>
> @@ -84,7 +301,7 @@
> int prefix;
> int stream;
> asn1_ctx_t asn1_ctx;
> - int offset = 0;
> + unsigned offset = 0;
>
> if (!tvb_bytes_exist(tvb, 0, 3))
> return FALSE; /* not enough bytes for a PES prefix */
> @@ -156,31 +373,23 @@
> es = tvb_new_subset(tvb, offset / 8, -1, -1);
> dissect_mpeg_pes(es, pinfo, tree);
> } else if (stream == STREAM_PACK) {
> - int length;
> - switch (tvb_get_guint8(tvb, 4) >> 6) {
> - case 1:
> - length = tvb_get_guint8(tvb, 13) & 0x07;
> - offset = dissect_mpeg_pes_Pack(tvb, offset, &asn1_ctx,
> - tree, hf_mpeg_pes_pack_header);
> - if (length > 0)
> - proto_tree_add_item(tree, hf_mpeg_pes_stuffing, tvb,
> - offset / 8, length, FALSE);
> - break;
> - default:
> - length = 8;
> - proto_tree_add_item(tree, hf_mpeg_pes_data, tvb,
> - offset / 8, length, FALSE);
> + if (tvb_get_guint8(tvb, offset / 8) >> 6 == 1) {
> + offset = dissect_mpeg_pes_pack_header(tvb, offset, pinfo, tree);
> + } else {
> + proto_tree_add_item(tree, hf_mpeg_pes_data, tvb,
> + offset / 8, 8, FALSE);
> + offset += 8 * 8;
> }
> - offset += length * 8;
> - } else if (stream == STREAM_SYSTEM) {
> - offset = dissect_mpeg_pes_Stream(tvb, offset, &asn1_ctx,
> - tree, hf_mpeg_pes_extension);
> + } else if (stream == STREAM_SYSTEM || stream == STREAM_PRIVATE2) {
> + unsigned data_length = tvb_get_ntohs(tvb, offset / 8);
> + proto_tree_add_item(tree, hf_mpeg_pes_length, tvb,
> + offset / 8, 2, FALSE);
> + offset += 2 * 8;
> +
> proto_tree_add_item(tree, hf_mpeg_pes_data, tvb,
> - offset / 8, -1, FALSE);
> + offset / 8, data_length, FALSE);
> } else if (stream == STREAM_PADDING) {
> - int padding_length;
> -
> - padding_length = tvb_get_ntohs(tvb, 4);
> + unsigned padding_length = tvb_get_ntohs(tvb, offset / 8);
> proto_tree_add_item(tree, hf_mpeg_pes_length, tvb,
> offset / 8, 2, FALSE);
> offset += 2 * 8;
> @@ -189,32 +398,43 @@
> offset / 8, padding_length, FALSE);
> } else if (stream == STREAM_PRIVATE1
> || stream >= STREAM_AUDIO) {
> - int length;
> - int header_length;
> - tvbuff_t *es;
> + int length = tvb_get_ntohs(tvb, 4);
>
> - length = tvb_get_ntohs(tvb, 4);
> + if ((tvb_get_guint8(tvb, 6) & 0xc0) == 0x80) {
> + int header_length;
> + tvbuff_t *es;
>
> - offset = dissect_mpeg_pes_Stream(tvb, offset, &asn1_ctx,
> - tree, hf_mpeg_pes_extension);
> - length -= 5 * 8;
> + offset = dissect_mpeg_pes_Stream(tvb, offset, &asn1_ctx,
> + tree, hf_mpeg_pes_extension);
> + length -= 5 * 8;
>
> - header_length = tvb_get_guint8(tvb, 8);
> - if (header_length > 0) {
> - proto_tree_add_item(tree, hf_mpeg_pes_header_data, tvb,
> - offset / 8, header_length, FALSE);
> - offset += header_length * 8;
> - length -= header_length * 8;
> + header_length = tvb_get_guint8(tvb, 8);
> + if (header_length > 0) {
> + int flags = tvb_get_guint8(tvb, 7);
> + tvbuff_t *header_data = tvb_new_subset(tvb, offset / 8,
> + header_length, header_length);
> + dissect_mpeg_pes_header_data(header_data, pinfo, tree, flags);
> + offset += header_length * 8;
> + length -= header_length * 8;
> + }
> +
> + es = tvb_new_subset(tvb, offset / 8, -1, length / 8);
> + if (tvb_get_ntoh24(es, 0) == PES_PREFIX)
> + dissect_mpeg_pes(es, pinfo, tree);
> + else if (tvb_get_guint8(es, 0) == 0xff)
> + dissect_mpeg(es, pinfo, tree);
> + else
> + proto_tree_add_item(tree, hf_mpeg_pes_data, es,
> + 0, -1, FALSE);
> + } else {
> + unsigned data_length = tvb_get_ntohs(tvb, offset / 8);
> + proto_tree_add_item(tree, hf_mpeg_pes_length, tvb,
> + offset / 8, 2, FALSE);
> + offset += 2 * 8;
> +
> + proto_tree_add_item(tree, hf_mpeg_pes_data, tvb,
> + offset / 8, data_length, FALSE);
> }
> -
> - es = tvb_new_subset(tvb, offset / 8, -1, length / 8);
> - if (tvb_get_ntoh24(es, 0) == PES_PREFIX)
> - dissect_mpeg_pes(es, pinfo, tree);
> - else if (tvb_get_guint8(es, 0) == 0xff)
> - dissect_mpeg(es, pinfo, tree);
> - else
> - proto_tree_add_item(tree, hf_mpeg_pes_data, es,
> - 0, -1, FALSE);
> } else {
> proto_tree_add_item(tree, hf_mpeg_pes_data, tvb,
> offset / 8, -1, FALSE);
> @@ -245,6 +465,15 @@
> { &hf_mpeg_pes_pack_header,
> { "Pack header", "mpeg-pes.pack",
> FT_NONE, BASE_NONE, NULL, 0, NULL, HFILL }},
> + { &hf_mpeg_pes_scr,
> + { "system clock reference (SCR)", "mpeg-pes.scr",
> + FT_RELATIVE_TIME, BASE_NONE, NULL, 0, NULL, HFILL }},
> + { &hf_mpeg_pes_program_mux_rate,
> + { "PES program mux rate", "mpeg-pes.program-mux-rate",
> + FT_UINT24, BASE_DEC, NULL, 0, NULL, HFILL }},
> + { &hf_mpeg_pes_stuffing_length,
> + { "PES stuffing length", "mpeg-pes.stuffing-length",
> + FT_UINT8, BASE_DEC, NULL, 0x07, NULL, HFILL }},
> { &hf_mpeg_pes_stuffing,
> { "PES stuffing bytes", "mpeg-pes.stuffing",
> FT_BYTES, BASE_NONE, NULL, 0, NULL, HFILL }},
> @@ -254,6 +483,42 @@
> { &hf_mpeg_pes_header_data,
> { "PES header data", "mpeg-pes.header-data",
> FT_BYTES, BASE_NONE, NULL, 0, NULL, HFILL }},
> + { &hf_mpeg_pes_pts,
> + { "presentation time stamp (PTS)", "mpeg-pes.pts",
> + FT_RELATIVE_TIME, BASE_NONE, NULL, 0, NULL, HFILL }},
> + { &hf_mpeg_pes_dts,
> + { "decode time stamp (DTS)", "mpeg-pes.dts",
> + FT_RELATIVE_TIME, BASE_NONE, NULL, 0, NULL, HFILL }},
> + { &hf_mpeg_pes_escr,
> + { "elementary stream clock reference (ESCR)", "mpeg-pes.escr",
> + FT_RELATIVE_TIME, BASE_NONE, NULL, 0, NULL, HFILL }},
> + { &hf_mpeg_pes_es_rate,
> + { "elementary stream rate", "mpeg-pes.es-rate",
> + FT_UINT24, BASE_DEC, NULL, 0x7ffe, NULL, HFILL }},
> + { &hf_mpeg_pes_copy_info,
> + { "copy info", "mpeg-pes.copy-info",
> + FT_UINT8, BASE_DEC, NULL, 0x7f, NULL, HFILL }},
> + { &hf_mpeg_pes_crc,
> + { "CRC", "mpeg-pes.crc",
> + FT_UINT16, BASE_DEC, NULL, 0, NULL, HFILL }},
> + { &hf_mpeg_pes_extension_flags,
> + { "extension flags", "mpeg-pes.extension-flags",
> + FT_UINT8, BASE_HEX, NULL, 0, NULL, HFILL }},
> + { &hf_mpeg_pes_private_data,
> + { "private data", "mpeg-pes.private-data",
> + FT_UINT16, BASE_HEX, NULL, 0, NULL, HFILL }},
> + { &hf_mpeg_pes_pack_length,
> + { "pack length", "mpeg-pes.pack-length",
> + FT_UINT8, BASE_DEC, NULL, 0, NULL, HFILL }},
> + { &hf_mpeg_pes_sequence,
> + { "sequence", "mpeg-pes.sequence",
> + FT_UINT16, BASE_HEX, NULL, 0, NULL, HFILL }},
> + { &hf_mpeg_pes_pstd_buffer,
> + { "P-STD buffer size", "mpeg-pes.pstd-buffer",
> + FT_UINT16, BASE_DEC, NULL, 0, NULL, HFILL }},
> + { &hf_mpeg_pes_extension2,
> + { "extension2", "mpeg-pes.extension2",
> + FT_UINT16, BASE_HEX, NULL, 0, NULL, HFILL }},
> { &hf_mpeg_pes_padding,
> { "PES padding", "mpeg-pes.padding",
> FT_BYTES, BASE_NONE, NULL, 0, NULL, HFILL }},
> @@ -282,6 +547,8 @@
>
> static gint *ett[] = {
> #include "packet-mpeg-pes-ettarr.c"
> + &ett_mpeg_pes_pack_header,
> + &ett_mpeg_pes_header_data,
> };
>
> proto_mpeg = proto_register_protocol(
> Index: asn1/mpeg/mpeg-pes.asn
> ===================================================================
> --- asn1/mpeg/mpeg-pes.asn (revision 22479)
> +++ asn1/mpeg/mpeg-pes.asn (working copy)
> @@ -26,24 +26,6 @@
> } (0..255)
> }
>
> -Pack ::= SEQUENCE {
> - must-be-zero BOOLEAN,
> - must-be-one0 BOOLEAN,
> - scr30 BIT STRING (SIZE (3)),
> - must-be-one1 BOOLEAN,
> - scr15 BIT STRING (SIZE (15)),
> - must-be-one2 BOOLEAN,
> - scr0 BIT STRING (SIZE (15)),
> - must-be-one3 BOOLEAN,
> - scr-ext BIT STRING (SIZE (9)),
> - must-be-one4 BOOLEAN,
> - program-mux-rate BIT STRING (SIZE (22)),
> - must-be-one5 BOOLEAN,
> - must-be-one6 BOOLEAN,
> - reserved BIT STRING (SIZE (5)),
> - stuffing-length INTEGER (0..7)
> -}
> -
> Stream ::= SEQUENCE {
> length INTEGER (0..65535),
> must-be-one BOOLEAN,
> Index: wiretap/wtap-int.h
> ===================================================================
> --- wiretap/wtap-int.h (revision 22479)
> +++ wiretap/wtap-int.h (working copy)
> @@ -151,7 +151,7 @@
>
> typedef struct {
> struct wtap_nstime now;
> - double t0;
> + time_t t0;
> } mpeg_t;
>
> typedef gboolean (*subtype_read_func)(struct wtap*, int*, char**, gint64*);
> Index: wiretap/mpeg.c
> ===================================================================
> --- wiretap/mpeg.c (revision 22479)
> +++ wiretap/mpeg.c (working copy)
> @@ -41,7 +41,6 @@
> #include "buffer.h"
> #include "file_wrappers.h"
> #include <errno.h>
> -#include <math.h>
> #include <stdlib.h>
> #include <string.h>
> #include <time.h>
> @@ -106,6 +105,8 @@
> return TRUE;
> }
>
> +#define SCRHZ 27000000
> +
> static gboolean
> mpeg_read(wtap *wth, int *err, gchar **err_info _U_,
> gint64 *data_offset)
> @@ -138,10 +139,6 @@
> guint32 pack0;
> guint64 pack;
> guint8 stuffing;
> - guint32 scr;
> - guint16 scr_ext;
> - double t;
> - double secs;
>
> bytes_read = file_read(&pack1, 1, sizeof pack1, wth->fh);
> if (bytes_read != sizeof pack1) {
> @@ -172,15 +169,20 @@
> stuffing &= 0x07;
> packet_size = 14 + stuffing;
>
> - scr = (guint32)
> - ((pack >> 59 & 0x0007) << 30 |
> - (pack >> 43 & 0x7fff) << 15 |
> - (pack >> 27 & 0x7fff) << 0);
> - scr_ext = (guint16)(pack >> 17 & 0x1ff);
> - t = wth->capture.mpeg->t0 + scr / 90e3 + scr_ext / 27e6;
> -
> - wth->capture.mpeg->now.nsecs = (int)(modf(t, &secs) * 1e9);
> - wth->capture.mpeg->now.secs = (time_t)secs;
> + {
> + guint64 bytes = pack >> 16;
> + guint64 ts =
> + (bytes >> 43 & 0x0007) << 30 |
> + (bytes >> 27 & 0x7fff) << 15 |
> + (bytes >> 11 & 0x7fff) << 0;
> + unsigned ext = bytes >> 1 & 0x1ff;
> + guint64 cr = 300 * ts + ext;
> + unsigned rem = cr % SCRHZ;
> + wth->capture.mpeg->now.secs
> + = wth->capture.mpeg->t0 + cr / SCRHZ;
> + wth->capture.mpeg->now.nsecs
> + = 1000000000LL * rem / SCRHZ;
> + }
> ts = wth->capture.mpeg->now;
> break;
> default:
> @@ -297,7 +299,7 @@
> wth->capture.mpeg = g_malloc(sizeof(mpeg_t));
> wth->capture.mpeg->now.secs = time(NULL);
> wth->capture.mpeg->now.nsecs = 0;
> - wth->capture.mpeg->t0 = (double) wth->capture.mpeg->now.secs;
> + wth->capture.mpeg->t0 = wth->capture.mpeg->now.secs;
>
> return 1;
> }
>
>
- Follow-Ups:
- Re: [Wireshark-dev] [PATCH] Further improvements to the MPEG decoder
- From: Jaap Keuter
- Re: [Wireshark-dev] [PATCH] Further improvements to the MPEG decoder
- References:
- [Wireshark-dev] [PATCH] Further improvements to the MPEG decoder
- From: Shaun Jackman
- [Wireshark-dev] [PATCH] Further improvements to the MPEG decoder
- Prev by Date: [Wireshark-dev] Help Required in Identificaiton of new packet Layer
- Next by Date: Re: [Wireshark-dev] [Wireshark-commits] rev 22556: /trunk/ /trunk/epan/: oids.c oids.h /trunk/asn1/snmp/: packet-snmp-template.c snmp.asn snmp.cnf
- Previous by thread: [Wireshark-dev] [PATCH] Further improvements to the MPEG decoder
- Next by thread: Re: [Wireshark-dev] [PATCH] Further improvements to the MPEG decoder
- Index(es):