URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=c6781b6385075ef0d289529b6ec165ec2f22aed2
Submitter: "Guy Harris <guy@xxxxxxxxxxxx>"
Changed: branch: master
Repository: wireshark
Commits:
c6781b6 by Guy Harris (guy@xxxxxxxxxxxx):
Remove FCS length checks from code that's handed a tvbuff without an FCS.
Data handed to dissect_ieee802154_decrypt() should have, and does, have
the FCS stripped off, so we don't need to use the FCS length.
While we're at it:
Update more comments for CC24xx metadata not being an FCS.
Do the header IE processing loop with a "data remaining" counter,
explain why we're doing a "data remaining" check and why it includes the
MIC length, and note that we should use the "data remaining" counter to
do more checks for invalid frames.
Change-Id: I928dbf6142b5876b6a25b954f798936c9e97ac0d
Reviewed-on: https://code.wireshark.org/review/32267
Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
Actions performed:
from 6ce7509 Add IEEE 802.15.4 TAP link type
add c6781b6 Remove FCS length checks from code that's handed a tvbuff without an FCS.
Summary of changes:
epan/dissectors/packet-ieee802154.c | 69 ++++++++++++++++++++++++++-----------
1 file changed, 48 insertions(+), 21 deletions(-)