Wireshark · Wireshark-commits: [Wireshark-commits] master cdd3e9c: Dumpcap: Make sure we set our pipe error status.

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

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=cdd3e9ce2ad472281074542eea30b0fa63ccc8be
Submitter: "Gerald Combs <gerald@xxxxxxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

cdd3e9c by Gerald Combs (gerald@xxxxxxxxxxxxx):

    Dumpcap: Make sure we set our pipe error status.
    
    Make sure cap_pipe_read_data_bytes sets pcap_src->cap_pipe_err if it
    encounters an error or EOF. This fixes a regression introduced in
    ga51b3d1d16. Have it return -1 or the number of bytes read similar to
    read(2). Explicitly treat its return value as a signed integer.
    
    Change-Id: I3de92859eee45e8d4a24a8c8309a816ef1b7924a
    Reviewed-on: https://code.wireshark.org/review/30639
    Petri-Dish: Gerald Combs <gerald@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Gerald Combs <gerald@xxxxxxxxxxxxx>
    

Actions performed:

    from  7059a51   WSDG: update test section with pytest fixtures
     add  cdd3e9c   Dumpcap: Make sure we set our pipe error status.


Summary of changes:
 dumpcap.c | 30 ++++++++++++++++++++----------
 1 file changed, 20 insertions(+), 10 deletions(-)