Wireshark-commits: [Wireshark-commits] master-3.2 c434a2a: voip_calls_dialog/voip_calls: Fix for du
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: Tue, 07 Jan 2020 06:21:32 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=c434a2ab5709604d50144c6f06b01ed26635702c
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master-3.2
Repository: wireshark

Commits:

c434a2a by Jirka Novak (j.novak@xxxxxxxxxxxx):

    voip_calls_dialog/voip_calls: Fix for duplicated entries after retap
    
    Issue seen from 2.6:
    1) Open pcap with any VoIP call
    2) Open Telephony->VoIP calls
    3) Select one or more calls
    4) Press Flow Sequence - sequence is shown
    5) Close Flow Sequence
    6) Press Play Streams - Play dialog is shown
    7) Close Play dialog
    8) Press Flow Sequence - sequence is shown, but all entries are duplicated
    
    If you repeat 6-7 multiple times, all entries are shown multiple times in 8
    
    Patch adds missing clear of graph_analysis before retaping the stream.
    Tested on master, but should be backported to as many stable branches as possible.
    
    Change-Id: I9793f6e874defde2f377732d78689e957df71b33
    Reviewed-on: https://code.wireshark.org/review/35672
    Petri-Dish: Jim Young <jim.young.ws@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    (cherry picked from commit 888137161cd1e4fe48c56cd6681cae5c10deede2)
    Reviewed-on: https://code.wireshark.org/review/35673
    Petri-Dish: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  7cbacb6   NSIS: Don't package .pdb files.
     add  c434a2a   voip_calls_dialog/voip_calls: Fix for duplicated entries after retap


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