Wireshark-commits: [Wireshark-commits] master 96c4655: Fix signed vs. unsigned comparison warning.

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, 12 Jun 2018 22:14:35 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=96c4655ae088c5162799df3b799696b6dece814f
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

96c4655 by Guy Harris (guy@xxxxxxxxxxxx):

    Fix signed vs. unsigned comparison warning.
    
    (In retrospect, signed offsets probably were the wrong choice; we
    rarely, if ever, use them to signify offsets from the end of the packet.
    Let's not do so any more in the future.)
    
    Change-Id: I7ace539be8bf927e21148c34b71e9c2b7535581e
    Reviewed-on: https://code.wireshark.org/review/28245
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  871e964   Add some length checks, remove a DISSECTOR_ASSERT().
    adds  96c4655   Fix signed vs. unsigned comparison warning.


Summary of changes:
 epan/crypt/dot11decrypt.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)