Wireshark-commits: [Wireshark-commits] master 7518a74: Qt: Fix a merge crash.
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, 17 Nov 2015 09:40:51 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=7518a74dbb32f941509a5dcabb5f74a3af14985d
Submitter: Pascal Quantin (pascal.quantin@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

7518a74 by Gerald Combs (gerald@xxxxxxxxxxxxx):

    Qt: Fix a merge crash.
    
    Update the logic in ui/qt/main_window.cpp:mergeCaptureFile to match
    ui/gtk/capture_file_dlg.c:file_merge_cmd. This ensures that we don't try
    to use a stale (and freed) read filter.
    
    Call cf_set_rfcode in both.
    
    Bug: 11718
    Change-Id: I6da65e428bff39e907f45992bac7337880c02ce9
    Reviewed-on: https://code.wireshark.org/review/11895
    Petri-Dish: Anders Broman <a.broman58@xxxxxxxxx>
    Tested-by: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    Reviewed-by: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    

Actions performed:

    from  622603b   extcap: fix a bug in cfilter
    adds  7518a74   Qt: Fix a merge crash.


Summary of changes:
 ui/gtk/capture_file_dlg.c |    2 +-
 ui/qt/main_window.cpp     |    6 ++----
 2 files changed, 3 insertions(+), 5 deletions(-)