Wireshark-commits: [Wireshark-commits] master 6d4c0ed: Don't go past the end of the "extra data".

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, 6 Dec 2014 04:54:34 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=6d4c0edc06ec95986a3ed26c9f9b14e8828d224d
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

6d4c0ed by Guy Harris (guy@xxxxxxxxxxxx):

    Don't go past the end of the "extra data".
    
    The "extra data", according to the RF5 API manual I have, is "hardware
    parameters", so change some names.
    
    Report an error if the variable parts of the configuration event record
    run past the record length.
    
    Report an error if the hardware parameters are too short.
    
    For the purportedly DS0 hardware parameters, don't assume they'll be
    long enough for the mask; they might not be.
    
    Change-Id: Ib63d042e4ede32216fb474c4ecdba84db1387abc
    Reviewed-on: https://code.wireshark.org/review/5638
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  8c190d6   RPKI-RTR: Missing break in switch (CID 1233099)
    adds  6d4c0ed   Don't go past the end of the "extra data".


Summary of changes:
 wiretap/k12.c |  138 ++++++++++++++++++++++++++++-----------------------------
 1 file changed, 69 insertions(+), 69 deletions(-)