Wireshark-commits: [Wireshark-commits] master f746d5e: Fix ASAN error due to invalid type
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=f746d5ec1daaa58802989dae8cbb731582c44d19
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark
Commits:
f746d5e by Peter Wu (peter@xxxxxxxxxxxxx):
Fix ASAN error due to invalid type
ei_array is supposed to be an array of expert_entry items. However, it
was initialized of an array of expert_info_t items which is much larger.
This caused an ASAN error when running `tshark -z expert` because
expert_stat_packet wants to read past the stack.
Fix this by correcting the type. While at it, reduce the size of
expert_entry for 64-bit systems (reduces initial memory usage by 8
kilobytes) and avoid a redundant g_array_index call.
Change-Id: I2e08676a5e242743ed502dd2836806604ea75cc0
Reviewed-on: https://code.wireshark.org/review/1275
Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
Actions performed:
from 2d774c6 Minor cleanup
adds f746d5e Fix ASAN error due to invalid type
Summary of changes:
ui/cli/tap-expert.c | 12 +++++-------
1 file changed, 5 insertions(+), 7 deletions(-)