Be careful about reporting number of missing SNs when wrapping around SN range.
authormartinm <martinm@f5534014-38df-0310-8fa8-9805f1628bb7>
Thu, 22 Sep 2011 15:28:46 +0000 (15:28 +0000)
committermartinm <martinm@f5534014-38df-0310-8fa8-9805f1628bb7>
Thu, 22 Sep 2011 15:28:46 +0000 (15:28 +0000)
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@39088 f5534014-38df-0310-8fa8-9805f1628bb7

epan/dissectors/packet-rlc-lte.c

index 448145546b6ed91d802da8cd5cd4602e0398c3ab..b2ee504ba9826250a404a6cbc56b5640d2dc4b2b 100644 (file)
@@ -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,