diff options
author | Jaap Keuter <jaap.keuter@xs4all.nl> | 2009-01-08 07:36:11 +0000 |
---|---|---|
committer | Jaap Keuter <jaap.keuter@xs4all.nl> | 2009-01-08 07:36:11 +0000 |
commit | 4b6b4b672c77862108295f1cb798147b4ce42e89 (patch) | |
tree | 9dc4553eae797fea5a0c989698acb797ee3ff89c /plugins/wimax/msg_sbc.c | |
parent | 269b45f48ae15bd0132fb5612c931cad228297fe (diff) | |
download | wireshark-4b6b4b672c77862108295f1cb798147b4ce42e89.tar.gz wireshark-4b6b4b672c77862108295f1cb798147b4ce42e89.tar.bz2 wireshark-4b6b4b672c77862108295f1cb798147b4ce42e89.zip |
Fix for bug 2902:
Actually check column status before writing to it.
svn path=/trunk/; revision=27192
Diffstat (limited to 'plugins/wimax/msg_sbc.c')
-rw-r--r-- | plugins/wimax/msg_sbc.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/plugins/wimax/msg_sbc.c b/plugins/wimax/msg_sbc.c index ef47f11148..b2ef888c6d 100644 --- a/plugins/wimax/msg_sbc.c +++ b/plugins/wimax/msg_sbc.c @@ -2248,7 +2248,7 @@ void dissect_mac_mgmt_msg_sbc_req_decoder(tvbuff_t *tvb, packet_info *pinfo, pro tlv_len = get_tlv_length(&tlv_info); if (tlv_type == -1 || tlv_len > MAX_TLV_LEN || tlv_len < 1) { /* invalid tlv info */ - if (pinfo->cinfo) + if (check_col(pinfo->cinfo, COL_INFO)) { col_append_sep_str(pinfo->cinfo, COL_INFO, NULL, "SBC-REQ TLV error"); } @@ -2257,7 +2257,7 @@ void dissect_mac_mgmt_msg_sbc_req_decoder(tvbuff_t *tvb, packet_info *pinfo, pro } if (tlv_type == 0) { /* invalid tlv type */ - if (pinfo->cinfo) + if (check_col(pinfo->cinfo, COL_INFO)) { col_append_sep_str(pinfo->cinfo, COL_INFO, NULL, "Invalid SBC TLV type"); } @@ -2809,7 +2809,7 @@ void dissect_mac_mgmt_msg_sbc_rsp_decoder(tvbuff_t *tvb, packet_info *pinfo, pro tlv_len = get_tlv_length(&tlv_info); if (tlv_type == -1 || tlv_len > MAX_TLV_LEN || tlv_len < 1) { /* invalid tlv info */ - if (pinfo->cinfo) + if (check_col(pinfo->cinfo, COL_INFO)) { col_append_sep_str(pinfo->cinfo, COL_INFO, NULL, "SBC-RSP TLV error"); } @@ -2818,7 +2818,7 @@ void dissect_mac_mgmt_msg_sbc_rsp_decoder(tvbuff_t *tvb, packet_info *pinfo, pro } if (tlv_type == 0) { /* invalid tlv type */ - if (pinfo->cinfo) + if (check_col(pinfo->cinfo, COL_INFO)) { col_append_sep_str(pinfo->cinfo, COL_INFO, NULL, "Invalid SBC TLV type"); } |