From Stephen Fisher

Attached is a patch to fix bug #1170: "Wireshark interpretation of WBXML
does not comply with Spec."  This has been verified with the sample
capture the user provided.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@19681 f5534014-38df-0310-8fa8-9805f1628bb7

diff --git a/epan/dissectors/packet-wbxml.c b/epan/dissectors/packet-wbxml.c
index a8b1e5097ec6008ae4070f25f1a0a469984b81b8..d164dace7ec2b37e53a2c7904b03356433da1905 100644 (file)
--- a/epan/dissectors/packet-wbxml.c
+++ b/epan/dissectors/packet-wbxml.c
@@ -5676,7 +5676,7 @@ parse_wbxml_tag_defined (proto_tree *tree, tvbuff_t *tvb, guint32 offset,
                                                                off, str_tbl, *level, codepage_attr, map);
                                                /* Check that there is still room in packet */
                                                off += len;
-                                               if (off >= tvb_len) {
+                                               if (off > tvb_len) {
                                                        DebugLog(("STAG: level = %u, ThrowException: len = %u (short frame)\n", *level, off - offset));
                                                        /*
                                                         * TODO - Do we need to free g_malloc()ed memory?