Wireshark-commits: [Wireshark-commits] master-2.6 a19ed2f: voip_calls: Fix leak of GString from uni
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: Sun, 03 May 2020 19:09:07 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=a19ed2f860e0cf61af13f5392c92f62404e228d6
Submitter: "Guy Harris <gharris@xxxxxxxxx>"
Changed: branch: master-2.6
Repository: wireshark

Commits:

a19ed2f 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>
    (cherry picked from commit 0b1e27ec774b65c157dd2f5de387f73a25ae306e)
    Reviewed-on: https://code.wireshark.org/review/37078
    

Actions performed:

    from  78488ed   Some more issues identified by PVS-Studio.
     add  a19ed2f   voip_calls: Fix leak of GString from unistim handling.


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