Huge thanks to our Platinum Members Endace and LiveAction,
and our Silver Member Veeam, for supporting the Wireshark Foundation and project.

Wireshark-commits: [Wireshark-commits] master c0aad88: capture_loop_write_pcapng_cb() shouldn't be

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sat, 10 Nov 2018 03:03:49 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=c0aad886bad7374a4990db32cfc2cda78a77bf75
Submitter: "Guy Harris <guy@xxxxxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

c0aad88 by Guy Harris (guy@xxxxxxxxxxxx):

    capture_loop_write_pcapng_cb() shouldn't be called if use_pcapng is false.
    
    If it *is* called when global_capture_opts.use_pcapng is false, don't
    just silently drop the packet on the floor, abort.
    
    Change-Id: Idb8f8e4c4ba231cfe674a81da34bf46e00f8247c
    Reviewed-on: https://code.wireshark.org/review/30562
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  3b68fc9   Dumpcap: Move the "just wrote one packet" logic to one place.
     add  c0aad88   capture_loop_write_pcapng_cb() shouldn't be called if use_pcapng is false.


Summary of changes:
 dumpcap.c | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)