Wireshark · Wireshark-commits: [Wireshark-commits] master-2.0 9f3bc84: Don't go past the end of a page in a NetScaler file.

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

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=9f3bc84b7e7e435c50b8b68f0fc526d0f5676cbf
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master-2.0
Repository: wireshark

Commits:

9f3bc84 by Guy Harris (guy@xxxxxxxxxxxx):

    Don't go past the end of a page in a NetScaler file.
    
    Records in a properly formatted NetScaler file shouldn't go past the end
    of a page, but nothing guarantees that a NetScaler file will be properly
    formatted.
    
    NetScaler 3.x files allow record bodies to go past the end of a page,
    but 1.x and 2.x files don't, so treat record headers that go past the
    end of a page, and record bodies in 1.x and 2.x files that go past the
    end of a page, as errors.
    
    Clean up some stuff while we're at it.
    
    Bug: 13430
    Change-Id: I3b1d56086e3bb14b246406f306e3d730df337561
    Reviewed-on: https://code.wireshark.org/review/20326
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    Reviewed-on: https://code.wireshark.org/review/20329
    

Actions performed:

    from  284ad58   nstrace: Allow opening of files that are not multiples of 16kB in size.
    adds  9f3bc84   Don't go past the end of a page in a NetScaler file.


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