diff options
author | Martin Mathieson <martin.r.mathieson@googlemail.com> | 2011-09-22 15:28:46 +0000 |
---|---|---|
committer | Martin Mathieson <martin.r.mathieson@googlemail.com> | 2011-09-22 15:28:46 +0000 |
commit | 4a3d86e46c0d73b3087560129fcc2bd2f7be9ba7 (patch) | |
tree | c7c4b49d5e3db1f534fdfb9c5025dad34a287b62 /epan | |
parent | 86878844b608a20cde03655475104fa630c31371 (diff) |
Be careful about reporting number of missing SNs when wrapping around SN range.
svn path=/trunk/; revision=39088
Diffstat (limited to 'epan')
-rw-r--r-- | epan/dissectors/packet-rlc-lte.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/packet-rlc-lte.c b/epan/dissectors/packet-rlc-lte.c index 448145546b..b2ee504ba9 100644 --- a/epan/dissectors/packet-rlc-lte.c +++ b/epan/dissectors/packet-rlc-lte.c @@ -927,7 +927,7 @@ static void addChannelSequenceInfo(state_sequence_analysis_report_in_frame *p, p_rlc_lte_info->ueid); proto_item_append_text(seqnum_ti, " - SNs missing (%u to %u)", p->firstSN, p->lastSN); - tap_info->missingSNs = ((p->lastSN - p->firstSN) % 1024) + 1; + tap_info->missingSNs = ((1024 + p->lastSN - p->firstSN) % 1024) + 1; } else { expert_add_info_format(pinfo, ti, PI_SEQUENCE, PI_WARN, @@ -1016,7 +1016,7 @@ static void addChannelSequenceInfo(state_sequence_analysis_report_in_frame *p, p_rlc_lte_info->ueid); proto_item_append_text(seqnum_ti, " - SNs missing (%u to %u)", p->firstSN, p->lastSN); - tap_info->missingSNs = ((p->lastSN - p->firstSN) % snLimit) + 1; + tap_info->missingSNs = ((snLimit + p->lastSN - p->firstSN) % snLimit) + 1; } else { expert_add_info_format(pinfo, ti, PI_SEQUENCE, PI_WARN, |