diff options
| author | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2016-06-07 07:48:58 +0200 |
|---|---|---|
| committer | Anders Broman <a.broman58@gmail.com> | 2016-06-07 08:24:56 +0000 |
| commit | f7cd537e1ad2ab347d36c5b40796aef22629a00c (patch) | |
| tree | cdc4b8226d4804cf178bf165dc0805d9e7867dab /epan/dissectors/packet-pim.c | |
| parent | 4509f2e2a48904bd07d38401249a146c3e6f51dd (diff) | |
PIMv2 Hello packet incorrectly dissect Option 1
Bug: 12493
Change-Id: I646853c6756689f9776b11c585d573c117c68cf2
Reviewed-on: https://code.wireshark.org/review/15762
Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'epan/dissectors/packet-pim.c')
| -rw-r--r-- | epan/dissectors/packet-pim.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/packet-pim.c b/epan/dissectors/packet-pim.c index f2736958a3..ac0c299c37 100644 --- a/epan/dissectors/packet-pim.c +++ b/epan/dissectors/packet-pim.c @@ -949,8 +949,8 @@ dissect_pim(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void* data _U_) switch(hello_opt) { case PIM_HELLO_HOLD_T: /* Hello Hold Time Option */ - ti_hold = proto_tree_add_item_ret_uint(pimopt_tree, hf_pim_holdtime, tvb, - offset, 2, ENC_BIG_ENDIAN, &holdtime); + ti_hold = proto_tree_add_item_ret_uint(opt_tree, hf_pim_holdtime, tvb, + offset+4, 2, ENC_BIG_ENDIAN, &holdtime); switch(holdtime){ case 0: proto_item_append_text(ti_hold, " (goodbye)"); |