URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=d59b2267fd3636a125678fb2b3ce4383a3a3eff4
Submitter: Pascal Quantin (pascal.quantin@xxxxxxxxx)
Changed: branch: master-2.6
Repository: wireshark
Commits:
d59b226 by Vasil Velichkov (vvvelichkov@xxxxxxxxx):
packet-k12: Fix small memory leak
Use g_strsplit/g_strfreev instead of wmem_strsplit/wmem_free because in
wmem_strutil.h the wmem_strsplit is documented not to be used with a
NULL alocator.
5 bytes in 1 blocks are definitely lost in loss record 63 of 9,354
at 0x4C2EBAB: malloc (vg_replace_malloc.c:299)
by 0x4E8D3C5: g_malloc (gmem.c:99)
by 0x8214317: wmem_alloc (wmem_core.c:37)
by 0x8219227: wmem_strdup (wmem_strutl.c:41)
by 0x8219AC1: wmem_strsplit (wmem_strutl.c:272)
by 0x749E3C9: protos_chk_cb (packet-k12.c:363)
5 bytes in 1 blocks are definitely lost in loss record 64 of 9,354
at 0x4C2EBAB: malloc (vg_replace_malloc.c:299)
by 0x4E8D3C5: g_malloc (gmem.c:99)
by 0x8214317: wmem_alloc (wmem_core.c:37)
by 0x8219227: wmem_strdup (wmem_strutl.c:41)
by 0x8219AC1: wmem_strsplit (wmem_strutl.c:272)
by 0x749E24A: k12_copy_cb (packet-k12.c:327)
Change-Id: I994769d17c87ed1d4f620379a2502452f48d80a5
Reviewed-on: https://code.wireshark.org/review/28779
Reviewed-by: Dario Lombardo <lomato@xxxxxxxxx>
Petri-Dish: Dario Lombardo <lomato@xxxxxxxxx>
Tested-by: Petri Dish Buildbot
Reviewed-by: Pascal Quantin <pascal.quantin@xxxxxxxxx>
Reviewed-on: https://code.wireshark.org/review/28788
Actions performed:
from 35ac0ba Remove code specific to older versions of Visual Studio.
adds d59b226 packet-k12: Fix small memory leak
Summary of changes:
epan/dissectors/packet-k12.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)