Huge thanks to our Platinum Members Endace and LiveAction,
and our Silver Member Veeam, for supporting the Wireshark Foundation and project.

Wireshark-dev: [Wireshark-dev] Loop in packet-q2931.c

Date: Tue, 22 Aug 2006 11:15:40 +0200
      Hello,


I found a loop in the q2931 dissector, whereas I was dissecting Ranap
Traces with a bad wireshark configuration.
Wireshark did crash, after eating all the memory.

Here is a small patch to solve this issue
<<
svn diff epan/dissectors/packet-q2931.c
Index: epan/dissectors/packet-q2931.c
===================================================================
--- epan/dissectors/packet-q2931.c      (revision 18965)
+++ epan/dissectors/packet-q2931.c      (working copy)
@@ -1556,6 +1556,7 @@
        while (len >= 3) {
                identifier = tvb_get_guint8(tvb, offset);
                value = tvb_get_ntohs(tvb, offset + 1);
+               len -=3;
                switch (identifier) {

                case 0x01:      /* Cumulative transit delay identifier */


>>
(See attached file: packet-q2931.c.diff.gz)

Regards
Florent

Attachment: packet-q2931.c.diff.gz
Description: Binary data