Wireshark-commits: [Wireshark-commits] master 0b1e27e: voip_calls: Fix leak of GString from unistim

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sun, 03 May 2020 19:08:01 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=0b1e27ec774b65c157dd2f5de387f73a25ae306e
Submitter: "Guy Harris <gharris@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

0b1e27e by Martin Mathieson (martin.mathieson@xxxxxxxxxxxx):

    voip_calls: Fix leak of GString from unistim handling.
    
    /opt/SourceCode/wireshark/ui/voip_calls.c	3786	err	V773 The function was exited without releasing the 'g_tmp' pointer. A memory leak is possible.
    
    Tested with unistim_call.pcap from SampleCaptures.
    
    Bug: 16335
    Change-Id: I8518a3e277e8acec15e09ca5f36672b5bdd181fe
    Reviewed-on: https://code.wireshark.org/review/37028
    Petri-Dish: Martin Mathieson <martin.r.mathieson@xxxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Guy Harris <gharris@xxxxxxxxx>
    

Actions performed:

    from  6ba8bd0   Some more issues identified by PVS-Studio.
     add  0b1e27e   voip_calls: Fix leak of GString from unistim handling.


Summary of changes:
 ui/voip_calls.c | 1 +
 1 file changed, 1 insertion(+)