Wireshark · Wireshark-commits: [Wireshark-commits] master f86d620: extcap: Read stderr on extcap error without an infinite loop

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

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=f86d6203a93f2b84b771c2d21e8a6dd24adfdca8
Submitter: "Roland Knall <rknall@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

f86d620 by Stig Bjørlykke (stig@xxxxxxxxxxxxx):

    extcap: Read stderr on extcap error without an infinite loop
    
    Check if data is available on stderr before doing a blocking read() to
    avoid an infinite read loop when having less data than STDERR_BUFFER_SIZE.
    
    Append data instead of overwrite when doing multiple read() to fetch
    available data.
    
    This is a regression from g6a949ed155.
    
    Bug: 15205
    Change-Id: I84b232aeafb6123f77f3f5d48bbe89326fe7eb0f
    Reviewed-on: https://code.wireshark.org/review/30209
    Petri-Dish: Stig Bjørlykke <stig@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Roland Knall <rknall@xxxxxxxxx>
    

Actions performed:

    from  674a57a   glib: restore some GLIB_CHECK_VERSION
     add  f86d620   extcap: Read stderr on extcap error without an infinite loop


Summary of changes:
 extcap.c         |  2 +-
 wsutil/ws_pipe.c | 10 +++++++++-
 2 files changed, 10 insertions(+), 2 deletions(-)