Wireshark · Wireshark-commits: [Wireshark-commits] master-2.4 c26e2af: Qt: use toLocalFile() instead of path() to check for existen

[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=c26e2af2a23cb7b451ea8db234c80caf301c1aa1
Submitter: "Guy Harris <guy@xxxxxxxxxxxx>"
Changed: branch: master-2.4
Repository: wireshark

Commits:

c26e2af by Dario Lombardo (lomato@xxxxxxxxx):

    Qt: use toLocalFile() instead of path() to check for existence.
    
    path() incorrectly gives paths as /C:/Program Files/Wireshark/udpdump.html
    under windows. The leading slash gives a wrong test on the file. Instead
    toLocalFile() handles it correctly. isLocalFile() has been used to get if
    we have a local file or a network URL. The reported bug occurred under Windows
    only, but the change is compatible with Linux paths as well.
    
    Accidentally when the test on the file was successful, nothing got called.
    The routine has been reworked to open an existing local file.
    
    Bug: 15592
    Change-Id: Id6e3a91dfb4c9d20ae8cb0735eabab64caeff47f
    Reviewed-on: https://code.wireshark.org/review/32772
    Petri-Dish: Dario Lombardo <lomato@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    (cherry picked from commit 4dda4bac178debe04eac081aa23a951e807f6d31)
    Reviewed-on: https://code.wireshark.org/review/32807
    

Actions performed:

    from  b6a0ec9   2.4.14 → 2.4.15.
     add  c26e2af   Qt: use toLocalFile() instead of path() to check for existence.


Summary of changes:
 ui/qt/extcap_options_dialog.cpp | 22 +++++++++-------------
 1 file changed, 9 insertions(+), 13 deletions(-)