Wireshark-commits: [Wireshark-commits] master 1cbcc57: Put a fake "FCS OK" value into the tree if w
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Fri, 01 Mar 2019 03:03:40 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=1cbcc57e0a0b0d4a27458799066e48fb49e2a0b5
Submitter: "Guy Harris <guy@xxxxxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

1cbcc57 by Guy Harris (guy@xxxxxxxxxxxx):

    Put a fake "FCS OK" value into the tree if we don't have FCS information.
    
    That was what was being done before; do it in the main dissector
    routine, as 1) the main dissector routine doesn't call the FCS or TI
    CC24xx dissector if we don't have an FCS or TI CC24xx metadata trailer
    and 2) that means we pull duplicate code out of those dissectors.
    
    Also, those routines are only called if we have the full FCS/metadata
    available, so there's no need for them to check for that.  (Arguably,
    they should be called if the data is present, according to the reported
    length, even if it's not available in the captured data, so we mark the
    frame as having been cut off so the full data isn't available.)
    
    Change-Id: I6be2a1f71a27bc41aea93e3c92743fc12c997c94
    Reviewed-on: https://code.wireshark.org/review/32281
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  74a776d   Pull some common code from the FCS/metadata dissectors.
     add  1cbcc57   Put a fake "FCS OK" value into the tree if we don't have FCS information.


Summary of changes:
 epan/dissectors/packet-ieee802154.c | 27 ++++++++++-----------------
 1 file changed, 10 insertions(+), 17 deletions(-)