osmith
/
wireshark
forked from osmocom/wireshark
1
0
Fork 0
Code Pull Requests Releases Wiki Activity

5co: Fix failed assertion 

Check length before calling tvb_get_raw_bytes_as_string()
Related to #17890
This commit is contained in:
Uli Heilmeier 2022-01-24 21:59:58 +01:00
parent 5f29a00814
commit fb7f1823da
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
epan/dissectors/packet-5co-legacy.c
View File

@ -693,9 +693,11 @@ dissect_FiveCoLegacy(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *
" (%d bytes to load into flash at offset %d)", unSize, unOffset);
break;
case FLASH_AREA_ANSWER:
string_buf = wmem_alloc(wmem_packet_scope(), header_data_length);
tvb_get_raw_bytes_as_string(tvb, tcp_data_offset, string_buf, header_data_length - 1);
proto_item_append_text(fiveco_data_item, " (%s)", string_buf);
if ( header_data_length > 1 ) {
string_buf = wmem_alloc(wmem_packet_scope(), header_data_length);
tvb_get_raw_bytes_as_string(tvb, tcp_data_offset, string_buf, header_data_length - 1);
proto_item_append_text(fiveco_data_item, " (%s)", string_buf);
}
break;
case WRITE_REGISTER_ANSWER: