Wireshark-commits: [Wireshark-commits] master ce6ac9e: And we need to check Nsts as well.

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: Tue, 9 Feb 2016 02:23:34 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ce6ac9e822c8ef716287f79fd3694c6ae7ca84a0
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

ce6ac9e by Guy Harris (guy@xxxxxxxxxxxx):

    And we need to check Nsts as well.
    
    The check against 0 isn't really necessary, as stbc_streams >= 0 and
    ieee80211_ht_streams[info_n->mcs_index] >= 1, but, what the heck.
    
    Make stbc_streams unsigned, as the value assigned to it is unsigned.
    
    Bug: 12085
    Change-Id: I98e0edb718f62ee07bbe56b9d15859c68b7d2e43
    Reviewed-on: https://code.wireshark.org/review/13844
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  0af5ef3   QCP: Fix retina label calculations.
    adds  ce6ac9e   And we need to check Nsts as well.


Summary of changes:
 epan/dissectors/packet-ieee80211-radio.c |    6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)