diff options
author | Bill Meier <wmeier@newsguy.com> | 2009-09-04 18:18:34 +0000 |
---|---|---|
committer | Bill Meier <wmeier@newsguy.com> | 2009-09-04 18:18:34 +0000 |
commit | 3ab20de02b813c3ed47060b8a0f5e413fdddfc43 (patch) | |
tree | f67fb3d64027874d3d4723e96434c818a8aab185 /epan/dissectors/packet-kingfisher.c | |
parent | 314b69382870fb2651a0c11999039fc24e8d55d7 (diff) |
From Rob Casey: Fix for: Incorrect station identifier parsing in Kingfisher dissector
See https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3946
svn path=/trunk/; revision=29700
Diffstat (limited to 'epan/dissectors/packet-kingfisher.c')
-rw-r--r-- | epan/dissectors/packet-kingfisher.c | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/epan/dissectors/packet-kingfisher.c b/epan/dissectors/packet-kingfisher.c index bc0b7bec2e..9727a8fd43 100644 --- a/epan/dissectors/packet-kingfisher.c +++ b/epan/dissectors/packet-kingfisher.c @@ -259,18 +259,18 @@ dissect_kingfisher(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, gboolean kfp->version = (kfp->function & 0x80)?3:2; - kfp->system = tvb_get_guint8(tvb, 0); - kfp->message = tvb_get_guint8(tvb, 5); - - - if(kfp->version == 2){ - kfp->target = tvb_get_guint8(tvb, 1); - kfp->from = tvb_get_guint8(tvb, 3); - kfp->via = tvb_get_guint8(tvb, 4); - } else { - kfp->target |= (tvb_get_guint8(tvb, 7) << 8); - kfp->from |= (tvb_get_guint8(tvb, 8) << 8); - kfp->via |= (tvb_get_guint8(tvb, 9) << 8); + kfp->system = tvb_get_guint8( tvb, 0 ); + kfp->message = tvb_get_guint8( tvb, 5 ); + + kfp->target = tvb_get_guint8( tvb, 1 ); + kfp->from = tvb_get_guint8( tvb, 3 ); + kfp->via = tvb_get_guint8( tvb, 4 ); + + if( kfp->version == 3 ) + { + kfp->target |= ( tvb_get_guint8( tvb, 7 ) << 8 ); + kfp->from |= ( tvb_get_guint8( tvb, 8 ) << 8 ); + kfp->via |= ( tvb_get_guint8( tvb, 9 ) << 8 ); } |