diff options
| author | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2021-09-05 14:57:57 +0200 |
|---|---|---|
| committer | Wireshark GitLab Utility <gerald+gitlab-utility@wireshark.org> | 2021-09-06 04:21:14 +0000 |
| commit | 203e082a50580b242a2288554b5958b5d4f77620 (patch) | |
| tree | 289966e47a58d0ca6ef54c1e406ef87177a920e6 /wiretap/nettrace_3gpp_32_423.c | |
| parent | d940352da7ebc8c8e0d9399bbd3515e145f54c39 (diff) | |
nettrace_3gpp (wiretap): Fix Dead Store found by Clang Analyzer
nettrace_3gpp_32_423.c:256:2: warning: Value stored to 'prev_pos' is never read [deadcode.DeadStores]
nettrace_3gpp_32_423.c:295:2: warning: Value stored to 'next_msg_pos' is never read [deadcode.DeadStores]
nettrace_3gpp_32_423.c:487:4: warning: Value stored to 'port_type_defined' is never read [deadcode.DeadStores]
Diffstat (limited to 'wiretap/nettrace_3gpp_32_423.c')
| -rw-r--r-- | wiretap/nettrace_3gpp_32_423.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/wiretap/nettrace_3gpp_32_423.c b/wiretap/nettrace_3gpp_32_423.c index a68a1611a4..74c21cc9eb 100644 --- a/wiretap/nettrace_3gpp_32_423.c +++ b/wiretap/nettrace_3gpp_32_423.c @@ -253,7 +253,7 @@ nettrace_msg_to_packet(nettrace_3gpp_32_423_file_info_t *file_info, wtap_rec *re return FALSE; } - prev_pos = curr_pos = input + CLEN(c_s_msg); + curr_pos = input + CLEN(c_s_msg); rec->rec_type = REC_TYPE_PACKET; rec->block = wtap_block_create(WTAP_BLOCK_PACKET); @@ -292,7 +292,6 @@ nettrace_msg_to_packet(nettrace_3gpp_32_423_file_info_t *file_info, wtap_rec *re status = FALSE; goto end; } - next_msg_pos += CLEN(c_e_msg); /* Check if we have a time stamp "changeTime" * expressed in number of seconds and milliseconds (nbsec.ms). @@ -484,7 +483,6 @@ nettrace_msg_to_packet(nettrace_3gpp_32_423_file_info_t *file_info, wtap_rec *re if (exported_pdu_info.presence_flags & EXP_PDU_TAG_DST_PORT_BIT) { if (!port_type_defined) { exp_pdu_tags_len += 4 + EXP_PDU_TAG_PORT_TYPE_LEN; - port_type_defined = TRUE; } exp_pdu_tags_len += 4 + EXP_PDU_TAG_PORT_LEN; } |