diff options
author | Evan Huus <eapache@gmail.com> | 2014-10-25 14:08:57 -0400 |
---|---|---|
committer | Evan Huus <eapache@gmail.com> | 2014-10-25 18:12:46 +0000 |
commit | 41f6923b3049dfb57bef544a4c580f256f807e85 (patch) | |
tree | 37b5dadf2dce81b919c89adfe4363cbf5f74c41c /ui/capture_globals.h | |
parent | 88d1bba17d826ce93dcf5d935385ee0435fb977e (diff) |
ncp2222: initialize buffer in build_expert_data
We do a bunch of conditional logic to fill in the buffer, and then were checking
strlen(buffer) to see if we'd actually filled it in or not, but if we hadn't
then the buffer was garbage and strlen(buffer) was throwing valgrind warnings.
Ensure this works as intended by setting the first byte of the buffer
unconditionally at the beginning, so strlen(buffer) returns 0 as expected in
that case.
Bug: 10628
Change-Id: I6f6c29fbbcb601b6ebccb3a4b312f0fa72c169ae
Reviewed-on: https://code.wireshark.org/review/4924
Reviewed-by: Evan Huus <eapache@gmail.com>
Diffstat (limited to 'ui/capture_globals.h')
0 files changed, 0 insertions, 0 deletions