diff options
author | Peter Wu <peter@lekensteyn.nl> | 2018-05-05 00:46:20 +0200 |
---|---|---|
committer | Anders Broman <a.broman58@gmail.com> | 2018-05-05 03:24:12 +0000 |
commit | 7c9f333ad0f79d695f530e05cdcceceb7d54d545 (patch) | |
tree | 587d5e81fa7a93dcde7e7650754fe6af3816d7b8 /wiretap/file_wrappers.c | |
parent | 7591ed848e862d6f6f91d7398a1b98c4e5dea0fa (diff) |
nettrace_3gpp_32_423: fix memleak and copy of uninitialized memory
When protocol="map", but the name attribute value is invalid, a memleak
occurs. Observe also that dissector_table_str is 22 bytes (21 characters
plus nul) and rounding up to a multiple of 4 means that 2 bytes of
uninitialized memory could be copied. Avoid that by copying the actual
length. Memory leak was found by Clang Static Analyzer.
Change-Id: I41f5b104449e108191e505611411a8fb18f1f5db
Fixes: v2.1.0rc0-2545-g4b4c7a76c3 ("[Nettrace] Add parsing of some HSS records.")
Reviewed-on: https://code.wireshark.org/review/27350
Petri-Dish: Peter Wu <peter@lekensteyn.nl>
Tested-by: Petri Dish Buildbot
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'wiretap/file_wrappers.c')
0 files changed, 0 insertions, 0 deletions