diff options
author | wmeier <wmeier@f5534014-38df-0310-8fa8-9805f1628bb7> | 2009-03-17 19:30:58 +0000 |
---|---|---|
committer | wmeier <wmeier@f5534014-38df-0310-8fa8-9805f1628bb7> | 2009-03-17 19:30:58 +0000 |
commit | 57f3f119732dd86ef98412d73a306e990d365a51 (patch) | |
tree | 183f176718551fa9b22092c06137fd694282f0f8 /epan/dissectors/packet-kdp.c | |
parent | 4e8e17ecf4b73d3c80c3102c67ed26f95ecf7609 (diff) |
(Minor): Remove some unneccessary code; Fix a typo;
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@27759 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'epan/dissectors/packet-kdp.c')
-rw-r--r-- | epan/dissectors/packet-kdp.c | 18 |
1 files changed, 6 insertions, 12 deletions
diff --git a/epan/dissectors/packet-kdp.c b/epan/dissectors/packet-kdp.c index 872df15cbd..073924e2de 100644 --- a/epan/dissectors/packet-kdp.c +++ b/epan/dissectors/packet-kdp.c @@ -207,25 +207,19 @@ static void dissect_kdp(tvbuff_t *tvb, char src_flowid_string[BUFFER_SIZE]; if (packet_flags & KDP_ACK_FLAG) { - int numchars = g_snprintf(ack_string, sizeof(ack_string), - "ACK=%x ", ack_number); - ack_string[numchars] = 0; + g_snprintf(ack_string, sizeof(ack_string), "ACK=%x ", ack_number); } else { - ack_string[0] = 0; + ack_string[0] = '\0'; } if (header_len > 4) { - int numchars = g_snprintf(seq_num_string, sizeof(seq_num_string), - "SEQ=%x ", sequence_number); - seq_num_string[numchars] = 0; + g_snprintf(seq_num_string, sizeof(seq_num_string), "SEQ=%x ", sequence_number); } else { - seq_num_string[0] = 0; + seq_num_string[0] = '\0'; } if (packet_flags & (KDP_SYN_FLAG | KDP_BCST_FLAG)) { - int numchars = g_snprintf(src_flowid_string, sizeof(ack_string), - "SRC_FLOWID=%x ", src_flowid); - src_flowid_string[numchars] = 0; + g_snprintf(src_flowid_string, sizeof(src_flowid_string), "SRC_FLOWID=%x ", src_flowid); } else { - src_flowid_string[0] = 0; + src_flowid_string[0] = '\0'; } col_add_fstr(pinfo->cinfo, COL_INFO, "%s%s%s%s%s%s%s%serrors=%d", ((packet_flags & KDP_DROP_FLAG) ? "DROP " : ""), |