Wireshark-commits: [Wireshark-commits] master-2.2 70095e6: Fix signed vs. unsigned comparison warni

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:19:15 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=70095e6fb3ccb850e5d3ee2b2df2ac29d8285e76
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master-2.2
Repository: wireshark

Commits:

70095e6 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>
    (cherry picked from commit 96c4655ae088c5162799df3b799696b6dece814f)
    Reviewed-on: https://code.wireshark.org/review/28248
    

Actions performed:

    from  8d694ba   Add some length checks, remove a DISSECTOR_ASSERT().
    adds  70095e6   Fix signed vs. unsigned comparison warning.


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