Wireshark-commits: [Wireshark-commits] master e7b07ae: IPv6: QuickStart option Nonce is read incorr

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sat, 18 Oct 2014 17:40:39 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=e7b07ae73ec1af97170a5d4dd99f71eef6b4e84f
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

e7b07ae by Alexis La Goutte (alexis.lagoutte@xxxxxxxxx):

    IPv6: QuickStart option Nonce is read incorrectly
    
    In Wireshark, the option's TTL field looks ok, but the [TTL diff] field points to the byte after the TTL field, which is part of the nonce field.
    The nonce field is being read one byte after where it should be read and so the value being read is incorrect.
    Moreover, the option data length says 6 but because of the extra byte being read 7 bytes are being read for this option data and the next option is read one byte later incorrectly so the options after the QuickStart option are being read incorrectly.
    
    Found by boaz
    Bug:10575
    
    Change-Id: Icd836b2a5c2f92cef7b6fea61957a85fca5e3352
    Reviewed-on: https://code.wireshark.org/review/4815
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  51f4962   MIPv6: Mobility Option Mobile Node Link Layer Identifier Link-layer Identifier field is read beyond the option data
    adds  e7b07ae   IPv6: QuickStart option Nonce is read incorrectly


Summary of changes:
 epan/dissectors/packet-ipv6.c |    6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)