diff options
author | Kovarththanan Rajaratnam <kovarththanan.rajaratnam@gmail.com> | 2009-09-24 20:21:23 +0000 |
---|---|---|
committer | Kovarththanan Rajaratnam <kovarththanan.rajaratnam@gmail.com> | 2009-09-24 20:21:23 +0000 |
commit | 00810ff19994509f4f8a080286ad43f5156e76c2 (patch) | |
tree | 687e12b760b8702534057b7348f27994c86b617d /epan/dissectors/packet-sctp.c | |
parent | 830dd8e8f05e39238b97857e161821a41ff985df (diff) |
* Prefer col_append_str instead of col_append_fstr for constant strings
* Remove check_col guards
svn path=/trunk/; revision=30127
Diffstat (limited to 'epan/dissectors/packet-sctp.c')
-rw-r--r-- | epan/dissectors/packet-sctp.c | 15 |
1 files changed, 5 insertions, 10 deletions
diff --git a/epan/dissectors/packet-sctp.c b/epan/dissectors/packet-sctp.c index 9bd6f7250a..5b2dbb6478 100644 --- a/epan/dissectors/packet-sctp.c +++ b/epan/dissectors/packet-sctp.c @@ -2197,8 +2197,7 @@ add_fragment(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, guint32 tsn, * frame, so it must be a duplicate fragment. maybe a retransmission? * Mark it as duplicate and return NULL */ - if (check_col(pinfo->cinfo, COL_INFO)) - col_append_str(pinfo->cinfo, COL_INFO, " (Duplicate Message Fragment)"); + col_append_str(pinfo->cinfo, COL_INFO, " (Duplicate Message Fragment)"); proto_tree_add_uint(tree, hf_sctp_duplicate, tvb, 0, 0, fragment->frame_num); return NULL; @@ -2370,8 +2369,7 @@ fragment_reassembly(tvbuff_t *tvb, sctp_fragment* fragment, /* this is not the last fragment, * so let the user know the frame where the reassembly is */ - if (check_col(pinfo->cinfo, COL_INFO)) - col_append_str(pinfo->cinfo, COL_INFO, " (Message Fragment) "); + col_append_str(pinfo->cinfo, COL_INFO, " (Message Fragment) "); proto_tree_add_uint(tree, hf_sctp_reassembled_in, tvb, 0, 0, message->reassembled_in->frame_num); return NULL; @@ -2408,8 +2406,7 @@ fragment_reassembly(tvbuff_t *tvb, sctp_fragment* fragment, * just mark as fragment */ - if (check_col(pinfo->cinfo, COL_INFO)) - col_append_str(pinfo->cinfo, COL_INFO, " (Message Fragment) "); + col_append_str(pinfo->cinfo, COL_INFO, " (Message Fragment) "); return NULL; } @@ -2440,8 +2437,7 @@ fragment_reassembly(tvbuff_t *tvb, sctp_fragment* fragment, */ if ((last_frag->tsn + 1)) { /* there are just fragments missing */ - if (check_col(pinfo->cinfo, COL_INFO)) - col_append_str(pinfo->cinfo, COL_INFO, " (Message Fragment) "); + col_append_str(pinfo->cinfo, COL_INFO, " (Message Fragment) "); return NULL; } @@ -2463,8 +2459,7 @@ fragment_reassembly(tvbuff_t *tvb, sctp_fragment* fragment, if (!frag_i || frag_i != end->fragment || frag_i->tsn != (last_frag->tsn + 1)) { /* we need more fragments. just mark as fragment */ - if (check_col(pinfo->cinfo, COL_INFO)) - col_append_str(pinfo->cinfo, COL_INFO, " (Message Fragment) "); + col_append_str(pinfo->cinfo, COL_INFO, " (Message Fragment) "); return NULL; } |