diff options
| author | Bill Meier <wmeier@newsguy.com> | 2012-08-13 18:54:58 +0000 |
|---|---|---|
| committer | Bill Meier <wmeier@newsguy.com> | 2012-08-13 18:54:58 +0000 |
| commit | a9424a928477ec64727bc9e747bcfdcb0836251e (patch) | |
| tree | 4cc038b1714cafb183ee6c95be5eb172daf666e4 /epan | |
| parent | 46800b6c317696b53544f8dc5b105b99c9164fbc (diff) | |
From Tobias Rutz; Minor updates.
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7518
svn path=/trunk/; revision=44475
Diffstat (limited to 'epan')
| -rw-r--r-- | epan/dissectors/packet-dvb-s2-bb.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/epan/dissectors/packet-dvb-s2-bb.c b/epan/dissectors/packet-dvb-s2-bb.c index 4a534a9558..1713b0f48c 100644 --- a/epan/dissectors/packet-dvb-s2-bb.c +++ b/epan/dissectors/packet-dvb-s2-bb.c @@ -597,7 +597,6 @@ static int dissect_dvb_s2_gse(tvbuff_t *tvb, int cur_off, proto_tree *tree, pack return new_off; } else { if (BIT_IS_CLEAR(gse_hdr, DVB_S2_GSE_HDR_START_POS) || BIT_IS_CLEAR(gse_hdr, DVB_S2_GSE_HDR_STOP_POS)) { - /* TODO: needs to be tested */ proto_tree_add_item(dvb_s2_gse_tree, hf_dvb_s2_gse_fragid, tvb, cur_off + new_off, 1, ENC_BIG_ENDIAN); @@ -702,7 +701,7 @@ static gboolean test_dvb_s2_crc(tvbuff_t *tvb, guint offset) { guint8 input8; /* only check BB Header and return */ - if (tvb_length(tvb) < (offset + DVB_S2_BB_HEADER_LEN)) /* XXX: I think 'offset + ...' is about right */ + if (tvb_length(tvb) < (offset + DVB_S2_BB_HEADER_LEN)) return FALSE; input8 = tvb_get_guint8(tvb, offset + DVB_S2_BB_OFFS_CRC); |