Wireshark-dev: [Wireshark-dev] [Patch] Fix for bug #1170
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Stephen Fisher <stephentfisher@xxxxxxxxx>
Date: Tue, 24 Oct 2006 21:58:26 -0700
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.


Steve


Index: epan/dissectors/packet-wbxml.c
===================================================================
--- epan/dissectors/packet-wbxml.c	(revision 19678)
+++ epan/dissectors/packet-wbxml.c	(working copy)
@@ -5676,7 +5676,7 @@
 								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?