Wireshark-commits: [Wireshark-commits] master f3986c2: steam-ihs: fix memleak on exception
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Wed, 10 Oct 2018 04:01:35 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=f3986c24728f03a346a10388fd6c15ea9ae16d41
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

f3986c2 by Peter Wu (peter@xxxxxxxxxxxxx):

    steam-ihs: fix memleak on exception
    
    When protobuf_dissect_unknown_field throws an exception,
    steamdiscover_dissect_body_status will leak memory as
    wmem_destroy_allocator is not called. Capture fuzz-2018-10-06-3104.pcap
    from the linked bug leaks 64kiB memory in each frame 14 and 36.
    
    Bug: 15171
    Change-Id: I930d0738fde61799ab4ef2310f8ff11c1bcb032b
    Fixes: v2.5.1rc0-130-g7ae954c7ac ("steam-ihs: Add dissector for the Steam IHS Discovery Protocol")
    Reviewed-on: https://code.wireshark.org/review/30098
    Petri-Dish: Peter Wu <peter@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  0d812e0   xdmcp: fix indent
     add  f3986c2   steam-ihs: fix memleak on exception


Summary of changes:
 epan/dissectors/packet-steam-ihs-discovery.c | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)