diff options
| author | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2017-05-25 11:16:12 +0200 |
|---|---|---|
| committer | Michael Mann <mmann78@netscape.net> | 2017-05-25 11:29:51 +0000 |
| commit | fd9f698ed332f94a8a452a420d8b211950f456b7 (patch) | |
| tree | 8347b92a969147f6ced973e6690a0a27e181952d /epan | |
| parent | d962e7ae71a856594ccc8be7ebd187e056d6f1e4 (diff) | |
icq: Fix Dead Store (Dead assignement/Dead increment) Warning found by Clang
Change-Id: Ic009ea5bce2f349ede407e0bed3a8640262433d2
Reviewed-on: https://code.wireshark.org/review/21753
Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan')
| -rw-r--r-- | epan/dissectors/packet-icq.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/epan/dissectors/packet-icq.c b/epan/dissectors/packet-icq.c index bbd321db16..3f3b0f3de3 100644 --- a/epan/dissectors/packet-icq.c +++ b/epan/dissectors/packet-icq.c @@ -892,12 +892,9 @@ icqv5_srv_meta_user(proto_tree *tree, /* Tree to put the data in */ } case META_ABOUT: { - int len; - /* Get the about information */ - proto_tree_add_item_ret_length(sstree, hf_icq_meta_user_about, tvb, offset, - 2, ENC_LITTLE_ENDIAN|ENC_ASCII, &len); - offset += len; + proto_tree_add_item(sstree, hf_icq_meta_user_about, tvb, offset, + 2, ENC_LITTLE_ENDIAN|ENC_ASCII); break; } case META_USER_INFO: |