URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=c9c1b3ce4b6c9086254e82d0530686e4851cc21a
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master-1.12
Repository: wireshark
Commits:
c9c1b3c by Michael Mann (mmann78@xxxxxxxxxxxx):
Various Mobile IPv6 bugfixes
Most of this was found by Boaz Brickner, but to prevent a few more bug reports, I tried to find any remaining MIPv6 options that used -1 to read until "end of packet" instead of "end of option"
1. Option-Code and Reserved fields should be 1 byte, not 2 in Local Mobility Anchor Address mobility option
2. IPv6 AUTH mobility option parses Mobility SPI
3. IPv6 MESG-ID (should just be a timestamp field)
4. IPv6 Redirect Mobility Option K and N bits are parsed incorrectly
5. The following options had a field that read until the end of the packet instead of the end of the option:
DNS-UPDATE-TYPE
AUTH Authentication Data
Care Of Test
Permanent Home Keygen Token
CGA Parameters
Signature
Experimental Mobility Option
Bug: 10630
Bug: 10629
Bug: 10626
Bug: 10625
Bug: 10624
Bug: 10622
Bug: 10619
Change-Id: I14c984a237efc1d1a09618d9a4a1caef1eaf8df0
Reviewed-on: https://code.wireshark.org/review/4932
Petri-Dish: Michael Mann <mmann78@xxxxxxxxxxxx>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
(cherry picked from commit 803a2f9518b9d207094dabfc01ad462e46588ef3)
Change-Id: I14c984a237efc1d1a09618d9a4a1caef1eaf8df0
Reviewed-on: https://code.wireshark.org/review/4934
Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
Actions performed:
from b726d17 ncp2222: initialize buffer in build_expert_data
adds c9c1b3c Various Mobile IPv6 bugfixes
Summary of changes:
epan/dissectors/packet-mip6.c | 70 ++++++++++++++++++++++++-----------------
1 file changed, 42 insertions(+), 28 deletions(-)