diff options
author | Martin Kaiser <wireshark@kaiser.cx> | 2015-05-25 15:57:02 +0200 |
---|---|---|
committer | Martin Kaiser <wireshark@kaiser.cx> | 2015-05-25 14:58:29 +0000 |
commit | 16715e01b6603164abf647d7df15e8f9558fbd51 (patch) | |
tree | bd900645f9db11ed531299c2dd44001b7c99c0a4 /epan/dissectors/packet-dvbci.c | |
parent | e37a8050b68a5c21fbb83dd808362cc291af0cf8 (diff) |
DVB-CI: remove NULL-checks for proto_item_append_text()'s first argument
Change-Id: I6648c20a003392a7435ca0461d2b004a1d415d51
Reviewed-on: https://code.wireshark.org/review/8626
Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
Diffstat (limited to 'epan/dissectors/packet-dvbci.c')
-rw-r--r-- | epan/dissectors/packet-dvbci.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/epan/dissectors/packet-dvbci.c b/epan/dissectors/packet-dvbci.c index f415191ad3..df00dd85e9 100644 --- a/epan/dissectors/packet-dvbci.c +++ b/epan/dissectors/packet-dvbci.c @@ -3031,8 +3031,7 @@ dissect_dvbci_payload_dt(guint32 tag, gint len_field, tvb, offset, 1, &resp_intv); if (resp_intv.secs==0) { col_append_sep_fstr(pinfo->cinfo, COL_INFO, NULL, "individual query"); - if (pi) - proto_item_append_text(pi, " (individual query)"); + proto_item_append_text(pi, " (individual query)"); } else { col_append_sep_fstr(pinfo->cinfo, COL_INFO, NULL, @@ -3874,8 +3873,7 @@ dissect_dvbci_payload_lsc(guint32 tag, gint len_field, case COMMS_REP_ID_SEND_ACK: col_append_sep_fstr(pinfo->cinfo, COL_INFO, ": ", "sent #%d", ret_val); - if (pi) - proto_item_append_text(pi, " (sent #%d)", ret_val); + proto_item_append_text(pi, " (sent #%d)", ret_val); break; case COMMS_REP_ID_SET_PARAMS_ACK: ret_val_str = val_to_str_const(ret_val, @@ -3893,8 +3891,7 @@ dissect_dvbci_payload_lsc(guint32 tag, gint len_field, if (ret_val_str) { col_append_sep_fstr(pinfo->cinfo, COL_INFO, ": ", "%s", ret_val_str); - if (pi) - proto_item_append_text(pi, " (%s)", ret_val_str); + proto_item_append_text(pi, " (%s)", ret_val_str); } break; case T_COMMS_SEND_LAST: |