Wireshark-bugs: [Wireshark-bugs] [Bug 5724] Add support for new header options in the KDP protoc

Date: Thu, 24 Feb 2011 02:06:31 -0800 (PST)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5724

Alexis La Goutte <alexis.lagoutte@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
                 CC|                            |alexis.lagoutte@xxxxxxxxx
         Resolution|FIXED                       |

--- Comment #3 from Alexis La Goutte <alexis.lagoutte@xxxxxxxxx> 2011-02-24 02:06:29 PST ---
Hi,

There is possibly bug/crash with the this patch..

For option6 and default 
[...]
+      case 6:
+        proto_tree_add_item(kdp_tree, hf_kdp_option6,
+                tvb, offset, option_len - 2, FALSE);
[...]
+      default:
+        proto_tree_add_item(kdp_tree, hf_kdp_option_unknown,
+                tvb, offset, option_len - 2, FALSE);
[...]
The Header field for option6 (hf_kdp_option6) and default
(hf_kdp_option_unknown)
is FT_UINT16 type...

A attach a small patch to fix this issue

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.