Submitter: Guy Harris (g...@alum.mit.edu)
Changed: branch: master
Repository: wireshark


8556f65 by Guy Harris (g...@alum.mit.edu):

    pcapng_read_block() should just return a Boolean.
    It either returns "OK" or "fail", so we might as well make it a Boolean.
    While we're at it, in pcapng_open(), handle EOF/short read and "invalid
    file" errors when trying to read the first block differently; for the
    first of those, we don't need to free *err_info, and this may be a bit
    safer in case *err_info didn't happen to be set to NULL somewhere along
    the line.
    Change-Id: If8135624e3efb7838dceeb28e30e5c8c4b064786
    Reviewed-on: https://code.wireshark.org/review/25608
    Reviewed-by: Guy Harris <g...@alum.mit.edu>

Actions performed:

    from  f53750a   pcapng_read_section_header_block() should just return a 
    adds  8556f65   pcapng_read_block() should just return a Boolean.

Summary of changes:
 wiretap/pcapng.c | 77 +++++++++++++++++++++++++++-----------------------------
 1 file changed, 37 insertions(+), 40 deletions(-)
Sent via:    Wireshark-commits mailing list <wireshark-commits@wireshark.org>
Archives:    https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-commits

Reply via email to