Wireshark-commits: [Wireshark-commits] master-3.2 b59795a: voip_calls: Fix leak of GString from uni

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:15 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=b59795a696f8c05aa9d62652d10bf8dbefbcddca
Submitter: "Guy Harris <gharris@xxxxxxxxx>"
Changed: branch: master-3.2
Repository: wireshark

Commits:

b59795a 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 41a4eaa4465f7f92c07433c8a349b88fed147ad9)
    Reviewed-on: https://code.wireshark.org/review/37076
    

Actions performed:

    from  aa2976d   Some more issues identified by PVS-Studio.
     add  b59795a   voip_calls: Fix leak of GString from unistim handling.


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