Wireshark · Wireshark-commits: [Wireshark-commits] master-2.6 4a44fce: Make the TTL field unsigned, but keep warning if the high-or

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

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

Commits:

4a44fce by Guy Harris (guy@xxxxxxxxxxxx):

    Make the TTL field unsigned, but keep warning if the high-order bit is set.
    
    See RFC 2181, section 8 (and RFC 1035 erratum 2130, which notes that
    section 3.2.1 says the TTL is signed but section 4.1.3 says it's
    unsigned); RFC 2181 section 8 says "unsigned, but avoid sending values
    that have the uppermost bit set, and treat values with the uppermost bit
    set as a value of 0".  (STD 13 = RFC 1034, the "concepts and facilities"
    DNS RFC, plus RFC 1035, the "implementation and specification" DNS RFC.)
    
    Change-Id: I9be6ac4f190f62dafbc45d1923a95f8f21306a7d
    Reviewed-on: https://code.wireshark.org/review/35343
    Petri-Dish: Guy Harris <guy@xxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    (cherry picked from commit 8275036fda8a7a5b9d0819c0ea3e3e26ad6e45ec)
    Reviewed-on: https://code.wireshark.org/review/35347
    Reviewed-by: Jaap Keuter <jaap.keuter@xxxxxxxxx>
    Petri-Dish: Jaap Keuter <jaap.keuter@xxxxxxxxx>
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    (cherry picked from commit 7fe976462f411489fea348fda8f44ca722873ba4)
    Reviewed-on: https://code.wireshark.org/review/35362
    

Actions performed:

    from  d362a0c   [Automatic update for 2019-12-08]
     add  4a44fce   Make the TTL field unsigned, but keep warning if the high-order bit is set.


Summary of changes:
 epan/dissectors/packet-dns.c | 50 ++++++++++++++++++++++++++++++++++++++------
 1 file changed, 44 insertions(+), 6 deletions(-)