Wireshark · Wireshark-commits: [Wireshark-commits] master c6781b6: Remove FCS length checks from code that's handed a tvbuff withou

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

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(-)