Wireshark-commits: [Wireshark-commits] master-3.0 f282dcd: Handle -k better on platforms that don't
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: Wed, 01 Apr 2020 20:31:51 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=f282dcde191edd006fa958feb851a88a9eae5652
Submitter: "Guy Harris <gharris@xxxxxxxxx>"
Changed: branch: master-3.0
Repository: wireshark

Commits:

f282dcd by Guy Harris (guy@xxxxxxxxxxxx):

    Handle -k better on platforms that don't support it.
    
    Have ws80211_init() return an indication that channel setting isn't
    supported on those platforms.
    
    In dumpcap, try to set up ws80211 before checking the channel argument
    and, if it fails, report the failure, rather than failing because the
    "convert channel name to channel code" routine fails.
    
    See
    
        https://ask.wireshark.org/question/15535/dumpcap-k-is-not-accepting-channel-type-values/
    
    for an example of confusion caused by the previous behavior.
    
    Change-Id: I303f560704700bbcd4f0ecea041f8632744212f3
    Reviewed-on: https://code.wireshark.org/review/36659
    Petri-Dish: Guy Harris <gharris@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Guy Harris <gharris@xxxxxxxxx>
    (cherry picked from commit 0975bf792a5609157349053385badca1f9c29cd7)
    Reviewed-on: https://code.wireshark.org/review/36661
    

Actions performed:

    from  7d71e3e   QT/UI: Fix shortcut
     add  f282dcd   Handle -k better on platforms that don't support it.


Summary of changes:
 caputils/ws80211_utils.c |  8 ++++----
 caputils/ws80211_utils.h |  7 ++++++-
 dumpcap.c                | 19 ++++++++++++-------
 ui/qt/wireless_frame.cpp |  2 +-
 4 files changed, 23 insertions(+), 13 deletions(-)