[bugzilla-daemon@xxxxxxxxxxxxx]

Comment # 1 on bug 12409 from Guy Harris

Running the top-of-the-1.12-branch version of TShark on that file gives

    WARNING **: Dissector bug, protocol WBXML, in packet 1: More than 1000000
items in the tree -- possible infinite loop

2.0 doesn't have a problem.

The changes in 2.0 that aren't in 1.12 are:

commit c462f1b54ed4459c0c93dabbf5fe30f224e796a6
Author: Michael Mann <mmann78@netscape.net>
Date:   Sun Aug 9 18:33:30 2015 -0400

    Eliminate proto_tree_add_text from packet-wbxml.c

    While I was at it, some (mostly) duplicate functionality allowed
rearranging of functions to eliminate most forward declarations.

    Change-Id: I2d7027d336c391d81dfe81c7a1ebf0d56c0826b2
    Reviewed-on: https://code.wireshark.org/review/9951
    Reviewed-by: Michael Mann <mmann78@netscape.net>
    Petri-Dish: Michael Mann <mmann78@netscape.net>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
    Reviewed-by: Anders Broman <a.broman58@gmail.com>

commit adb81f335d17a151361f00001e125480eeaf7bc5
Author: Martin Mathieson <martin.r.mathieson@googlemail.com>
Date:   Thu Dec 25 22:34:06 2014 +0000

    Deleting unnecessary #includes from dissectors.
    Fifth batch (packet-rtp.c -> end).

    Will look at cleaning up and committing script afterwards.

    Change-Id: I8ed61dc941d98d3f7259a9d1f74e214eb7b4bfa2
    Reviewed-on: https://code.wireshark.org/review/6052
    Reviewed-by: Martin Mathieson <martin.r.mathieson@googlemail.com>

commit b5d7b7ab6e6b0e595d3250d7fa6553d0da79c000
Author: Bill Meier <wmeier@newsguy.com>
Date:   Sat Dec 20 16:23:59 2014 -0500

    Cleanup use of #includes in non-generated epan/dissector/*.c

    Specifically:
    - Set packet.h to be the first wireshark #include after
       config.h and "system" #includes.
       packet.h added as an #include in some cases when missing.
    - Remove some #includes included (directly/indirectly) in
       packet.h. E.g., glib.h.
       (Done only for those files including packet.h).
    - As needed, move "system" #includes to be after config.h and
       before wireshark #includes.
    - Rework various #include file specifications for consistency.
    - Misc.

    Change-Id: Ifaa1a14b50b69fbad38ea4838a49dfe595c54c95
    Reviewed-on: https://code.wireshark.org/review/5923
    Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com>
    Reviewed-by: Bill Meier <wmeier@newsguy.com>

commit 24d50bc2904c265f9f89b3df25c9e41f309a29bb
Author: Bill Meier <wmeier@newsguy.com>
Date:   Mon Oct 6 12:55:18 2014 -0400

    Adjust indentation to match editor modelines; Do other minor whitespace
changes.

    Change-Id: Ic020b2c92db5d14a2be9dc4d35aef4514b8b0353
    Reviewed-on: https://code.wireshark.org/review/4502
    Reviewed-by: Bill Meier <wmeier@newsguy.com>

commit c71e88e4a9ad00600627b27ec6bd68cd3a929c58
Author: Michael Mann <mmann78@netscape.net>
Date:   Sun Jul 6 14:21:40 2014 -0400

    convert to proto_tree_add_subtree[_format]

    Change-Id: I3efa2a81ab2685cde6eae0a00b24520478a545ce
    Reviewed-on: https://code.wireshark.org/review/2900
    Reviewed-by: Michael Mann <mmann78@netscape.net>

---

      You are receiving this mail because:

• You are watching all bug changes.