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

Wireshark-dev: Re: [Wireshark-dev] Patches to update PANA Protocol dissector

From: Jaap Keuter <jaap.keuter@xxxxxxxxx>
Date: Tue, 07 Aug 2007 22:37:56 +0200
Hi,

Two things.

1. We've recently changed the patch submission policy to go through bugzilla. This in order to have patches like this not overlooked.

2. I can't comment on the merits of the patch in relation to the draft referenced, but I've spotted this code error:
        /* Reserved field */
proto_tree_add_item(pana_tree, hf_pana_reserved_type, tvb, offset, 1, FALSE);
-       offset += 1;
+       offset += 2;

This should be:
        /* Reserved field */
- proto_tree_add_item(pana_tree, hf_pana_reserved_type, tvb, offset, 1, FALSE);
-       offset += 1;
+ proto_tree_add_item(pana_tree, hf_pana_reserved_type, tvb, offset, 2, FALSE);
+       offset += 2;

Thanx,
Jaap


Victor Fajardo wrote:
To wireshark-maintainers,

Attached is a patch file for the PANA protocol dissector. The changes updates the packet-pana.c dissector to support draft-ietf-pana-pana-18.txt.

regards,
victor