URL: 
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=8556f65e0740af647482a6fca86371bbc498b68e
Submitter: Guy Harris (g...@alum.mit.edu)
Changed: branch: master
Repository: wireshark

Commits:

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 
Boolean.
    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
             mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

Reply via email to