Wireshark-commits: [Wireshark-commits] master 847f590: MIP6: AUTH mobility option parses Mobility S

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: Sat, 14 Feb 2015 16:36:53 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=847f59096304a7aeebf7e9c498986cb05b960540
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

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

    MIP6: AUTH mobility option parses Mobility SPI and Authentication Data incorrectly
    
    The authentication data in the AUTH option in attached IPv6 packet contains one byte too much. This byte is read beyond the option data.
    In Wireshark, clicking on the AUTH shows that the option ends at 0xCE while clicking on the Authentication Data shows that it ends at 0xCF.
    
    Reported by Boaz
    
    Bug:10626
    Change-Id: I0bcfd6331bc1de30f25d16590487c0e3bf5c002f
    Reviewed-on: https://code.wireshark.org/review/7112
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  91b5042   "Hide" hashether_t structure.
    adds  847f590   MIP6: AUTH mobility option parses Mobility SPI and Authentication Data incorrectly


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