diff options
author | Kovarththanan Rajaratnam <kovarththanan.rajaratnam@gmail.com> | 2009-08-09 07:59:51 +0000 |
---|---|---|
committer | Kovarththanan Rajaratnam <kovarththanan.rajaratnam@gmail.com> | 2009-08-09 07:59:51 +0000 |
commit | 6110a96f689d51c92c5b00b4dfda3a9cd6916a25 (patch) | |
tree | ef21c459b918013ddfaded20b267565537e9bf64 /epan/dissectors/packet-icq.c | |
parent | 0a5b014cecf14ec1bea5c87b0ffb5c2f7fa366a1 (diff) |
Don't guard col_set_str (COL_INFO/COL_PROTOCOL) with col_check
svn path=/trunk/; revision=29345
Diffstat (limited to 'epan/dissectors/packet-icq.c')
-rw-r--r-- | epan/dissectors/packet-icq.c | 28 |
1 files changed, 7 insertions, 21 deletions
diff --git a/epan/dissectors/packet-icq.c b/epan/dissectors/packet-icq.c index f45cf05346..db6f76b850 100644 --- a/epan/dissectors/packet-icq.c +++ b/epan/dissectors/packet-icq.c @@ -451,12 +451,8 @@ dissect_icqv4(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) proto_item *ti = NULL; /* Not really implemented yet */ - if (check_col(pinfo->cinfo, COL_PROTOCOL)) { - col_set_str(pinfo->cinfo, COL_PROTOCOL, "ICQv4 (UDP)"); - } - if (check_col(pinfo->cinfo, COL_INFO)) { - col_set_str(pinfo->cinfo, COL_INFO, "ICQ Version 4 protocol"); - } + col_set_str(pinfo->cinfo, COL_PROTOCOL, "ICQv4 (UDP)"); + col_set_str(pinfo->cinfo, COL_INFO, "ICQ Version 4 protocol"); if (tree) { ti = proto_tree_add_protocol_format(tree, proto_icq, tvb, 0, -1, "ICQv4"); @@ -474,12 +470,8 @@ dissect_icqv3(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) proto_item *ti = NULL; /* Not really implemented yet */ - if (check_col(pinfo->cinfo, COL_PROTOCOL)) { - col_set_str(pinfo->cinfo, COL_PROTOCOL, "ICQv3 (UDP)"); - } - if (check_col(pinfo->cinfo, COL_INFO)) { - col_set_str(pinfo->cinfo, COL_INFO, "ICQ Version 3 protocol"); - } + col_set_str(pinfo->cinfo, COL_PROTOCOL, "ICQv3 (UDP)"); + col_set_str(pinfo->cinfo, COL_INFO, "ICQ Version 3 protocol"); if (tree) { ti = proto_tree_add_protocol_format(tree, proto_icq, tvb, 0, -1, "ICQv3"); @@ -497,12 +489,8 @@ dissect_icqv2(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) proto_item *ti = NULL; /* Not really implemented yet */ - if (check_col(pinfo->cinfo, COL_PROTOCOL)) { - col_set_str(pinfo->cinfo, COL_PROTOCOL, "ICQv2 (UDP)"); - } - if (check_col(pinfo->cinfo, COL_INFO)) { - col_set_str(pinfo->cinfo, COL_INFO, "ICQ Version 2 protocol"); - } + col_set_str(pinfo->cinfo, COL_PROTOCOL, "ICQv2 (UDP)"); + col_set_str(pinfo->cinfo, COL_INFO, "ICQ Version 2 protocol"); if (tree) { ti = proto_tree_add_protocol_format(tree, proto_icq, tvb, 0, -1, "ICQv2"); @@ -1740,9 +1728,7 @@ dissect_icq(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) if (version < 2 || version > 5) return 0; /* This is not a (recognized) ICQ packet */ - if (check_col(pinfo->cinfo, COL_PROTOCOL)) { - col_set_str(pinfo->cinfo, COL_PROTOCOL, "ICQ"); - } + col_set_str(pinfo->cinfo, COL_PROTOCOL, "ICQ"); if (check_col(pinfo->cinfo, COL_INFO)) { col_clear(pinfo->cinfo, COL_INFO); } |