diff options
| author | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2012-03-01 09:11:08 +0000 |
|---|---|---|
| committer | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2012-03-01 09:11:08 +0000 |
| commit | ddcb79687870a9fc25a4ba15a604966d5b9fe35f (patch) | |
| tree | 1fab433839aaf287f7b3537293c31979528d6d19 /epan/dissectors/packet-cipmotion.c | |
| parent | 23ec82fae9404694b90dc2ce0d1a9a65fe0f6b8c (diff) | |
From Don Westman via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6889
CIP Motion parsing of set axis attribute list request is messing up attr data
parsing a CIP Motion 'Set Axis Attribute List' Request returns correct data for attribute ID, dimension and element size but 'attribute data' field retrieves the same data for each attribute in the list rather than the real data sent in the request.
From me :
Fix the typo error (use the wrong offset variable, need to use local_offset variable)
svn path=/trunk/; revision=41257
Diffstat (limited to 'epan/dissectors/packet-cipmotion.c')
| -rw-r--r-- | epan/dissectors/packet-cipmotion.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-cipmotion.c b/epan/dissectors/packet-cipmotion.c index 0f75b81780..ee547a4b81 100644 --- a/epan/dissectors/packet-cipmotion.c +++ b/epan/dissectors/packet-cipmotion.c @@ -1229,7 +1229,7 @@ dissect_set_axis_attr_list_request (tvbuff_t* tvb, proto_tree* tree, guint32 off } /* Display the value of this attribute */ - proto_tree_add_item(attr_tree, hf_cip_attribute_data, tvb, offset + attribute_start, attribute_size, ENC_NA); + proto_tree_add_item(attr_tree, hf_cip_attribute_data, tvb, local_offset + attribute_start, attribute_size, ENC_NA); /* Round the attribute size up so the next attribute lines up on a 32-bit boundary */ if (attribute_size % 4 != 0) |