diff options
| author | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2017-05-14 17:35:07 +0200 |
|---|---|---|
| committer | Guy Harris <guy@alum.mit.edu> | 2017-05-14 19:16:13 +0000 |
| commit | 504998d3f028033eee7370c5a6c28eae870afa12 (patch) | |
| tree | 65e5a4e03152d9c4838ed41965621f3b2f1cba13 /epan/dissectors/packet-smb.c | |
| parent | 33fbc25977d1d2c6b044b5014416e42dfaca7056 (diff) | |
smb: don't use tvb_get_* on proto_tree_add_uint
Change-Id: I38db11b444a1d4b5be7980c262e9f84ae48c7427
Reviewed-on: https://code.wireshark.org/review/21649
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Petri-Dish: Anders Broman <a.broman58@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to 'epan/dissectors/packet-smb.c')
| -rw-r--r-- | epan/dissectors/packet-smb.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/epan/dissectors/packet-smb.c b/epan/dissectors/packet-smb.c index 5651a367fd..3c0423f6fc 100644 --- a/epan/dissectors/packet-smb.c +++ b/epan/dissectors/packet-smb.c @@ -13915,23 +13915,23 @@ dissect_transaction_request(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, /* it is not a secondary request */ /* total param count , only a 16 bit integer here*/ - proto_tree_add_uint(tree, hf_smb_total_param_count, tvb, offset, 2, tvb_get_letohs(tvb, offset)); + proto_tree_add_item(tree, hf_smb_total_param_count, tvb, offset, 2, ENC_LITTLE_ENDIAN); offset += 2; /* total data count , only 16bit integer here*/ - proto_tree_add_uint(tree, hf_smb_total_data_count, tvb, offset, 2, tvb_get_letohs(tvb, offset)); + proto_tree_add_item(tree, hf_smb_total_data_count, tvb, offset, 2, ENC_LITTLE_ENDIAN); offset += 2; /* max param count , only 16bit integer here*/ - proto_tree_add_uint(tree, hf_smb_max_param_count, tvb, offset, 2, tvb_get_letohs(tvb, offset)); + proto_tree_add_item(tree, hf_smb_max_param_count, tvb, offset, 2, ENC_LITTLE_ENDIAN); offset += 2; /* max data count, only 16bit integer here*/ - proto_tree_add_uint(tree, hf_smb_max_data_count, tvb, offset, 2, tvb_get_letohs(tvb, offset)); + proto_tree_add_item(tree, hf_smb_max_data_count, tvb, offset, 2, ENC_LITTLE_ENDIAN); offset += 2; /* max setup count, only 16bit integer here*/ - proto_tree_add_uint(tree, hf_smb_max_setup_count, tvb, offset, 1, tvb_get_guint8(tvb, offset)); + proto_tree_add_item(tree, hf_smb_max_setup_count, tvb, offset, 1, ENC_NA); offset += 1; /* reserved byte */ @@ -15739,7 +15739,7 @@ dissect_qfsi_vals(tvbuff_t * tvb, packet_info * pinfo, proto_tree * tree, /* volume label length, only one byte here */ CHECK_BYTE_COUNT_TRANS_SUBR(1); - proto_tree_add_uint(tree, hf_smb_volume_label_len, tvb, offset, 1, tvb_get_guint8(tvb, offset)); + proto_tree_add_item(tree, hf_smb_volume_label_len, tvb, offset, 1, ENC_NA); COUNT_BYTES_TRANS_SUBR(1); /* label - not aligned! */ |