Wireshark-commits: [Wireshark-commits] master 41c8cd6: Fix setting of STATS_LEN - it's what's in th

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, 15 Mar 2014 09:46:05 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=41c8cd62bf9df07e981de4e46f59a5452baae472
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

41c8cd6 by Guy Harris (guy@xxxxxxxxxxxx):

    Fix setting of STATS_LEN - it's what's in the file, not what we *generate*.
    
    vwr->STATS_LEN is the length of the statistics trailer at the end of the
    packet, and it's the same for S2 and S3 versions of the WLAN card.  It
    should *not* be set to the length of the metadata headers that we
    *generate and put in front of the packet data that we hand to our
    callers*.
    
    Get rid of a debugging message while we're at it.
    
    Change-Id: I465b5ba4dedb88f1f401d34439b44b16a4bb01cc
    Reviewed-on: https://code.wireshark.org/review/671
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  0d6cdb5   Clean up naming and get rid of unused and duplicate definitions.
    adds  41c8cd6   Fix setting of STATS_LEN - it's what's in the file, not what we *generate*.


Summary of changes:
 wiretap/vwr.c |    3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)