Wireshark · Wireshark-commits: [Wireshark-commits] master c3c9386: extcap: do not leak "version" field

[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=c3c93869fc736e36649f1cd6e33f00b49cb3122f
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

c3c9386 by Peter Wu (peter@xxxxxxxxxxxxx):

    extcap: do not leak "version" field
    
    Caught by LeakSanitizer:
    
        Direct leak of 18 byte(s) in 3 object(s) allocated from:
            #0 0x55ec8c5ffec8 in __interceptor_malloc (run/wireshark+0x145dec8)
            #1 0x7f4d021e4328 in g_malloc /build/src/glib-2.46.2/glib/gmem.c:94
            #2 0x7f4d021fd0de in g_strdup /build/src/glib-2.46.2/glib/gstrfuncs.c:363
            #3 0x55ec8c6ce514 in extcap_parse_interface_sentence extcap_parser.c:670:26
            #4 0x55ec8c6ce7ad in extcap_parse_interfaces extcap_parser.c:683:13
            #5 0x55ec8c6b6781 in interfaces_cb extcap.c:313:5
            #6 0x55ec8c6b4ce6 in extcap_foreach extcap.c:206:26
            #7 0x55ec8c6b62a6 in extcap_interface_list extcap.c:415:5
            #8 0x55ec8c6b7fab in extcap_register_preferences extcap.c:437:9
            #9 0x55ec8c63104a in main wireshark-qt.cpp:847:5
            #10 0x7f4ce8f4460f in __libc_start_main (/usr/lib/libc.so.6+0x2060f)
            #11 0x55ec8c569ed8 in _start (run/wireshark+0x13c7ed8)
    
    Change-Id: I0ef89e647b2cc9aab495a80f6c638e9b67cf3ad1
    Reviewed-on: https://code.wireshark.org/review/13692
    Reviewed-by: Dario Lombardo <lomato@xxxxxxxxx>
    Petri-Dish: Dario Lombardo <lomato@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  7f3f11f   Add Userlog dissector
    adds  c3c9386   extcap: do not leak "version" field


Summary of changes:
 extcap_parser.c |    9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)