More warining fixes: char -> const char
[obnox/wireshark/wip.git] / capture.c
index 26bf74829a23b9d837b88a5fbe29f7efd19341b1..12f9cbe3f3e4b44a87a51a906ec0961be20ccbf5 100644 (file)
--- a/capture.c
+++ b/capture.c
@@ -1,5 +1,5 @@
 /* capture.c
- * Routines for packet capture windows
+ * Routines for packet capture
  *
  * $Id$
  *
  * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
  */
 
-/* With MSVC and a libethereal.dll this file needs to import some variables 
-   in a special way. Therefore _NEED_VAR_IMPORT_ is defined. */
-#define _NEED_VAR_IMPORT_
-
 #ifdef HAVE_CONFIG_H
 # include "config.h"
 #endif
 
 #ifdef HAVE_LIBPCAP
 
+#ifdef HAVE_UNISTD_H
+#include <unistd.h>
+#endif
+
 #include <stdlib.h>
 #include <string.h>
 #include <ctype.h>
 #include "capture-wpcap.h"
 #endif
 #include "ui_util.h"
+#include "log.h"
 
 
-/* Win32 needs the O_BINARY flag for open() */
-#ifndef O_BINARY
-#define O_BINARY       0
-#endif
 
-static gboolean normal_do_capture(capture_options *capture_opts, gboolean is_tempfile);
-static void stop_capture_signal_handler(int signo);
+/** 
+ * Start a capture.
+ *
+ * @return TRUE if the capture starts successfully, FALSE otherwise.
+ */
+gboolean
+capture_start(capture_options *capture_opts)
+{
+  gboolean ret;
 
 
-void
-capture_opts_init(capture_options *capture_opts, void *cfile)
-{
-  capture_opts->cf                      = cfile;
-  capture_opts->cfilter                            = g_strdup("");
-  capture_opts->iface                   = NULL;
-#ifdef _WIN32
-  capture_opts->buffer_size             = 1;                /* 1 MB */
-#endif
-  capture_opts->has_snaplen             = FALSE;
-  capture_opts->snaplen                 = MIN_PACKET_SIZE;
-  capture_opts->promisc_mode            = TRUE;
-  capture_opts->linktype                = -1;               /* the default linktype */
-  capture_opts->capture_child           = FALSE;
-  capture_opts->save_file               = NULL;
-  capture_opts->save_file_fd            = -1;
-  capture_opts->sync_mode               = TRUE;
-  capture_opts->show_info               = TRUE;
-  capture_opts->quit_after_cap          = FALSE;
-
-  capture_opts->multi_files_on          = FALSE;
-  capture_opts->has_file_duration       = FALSE;
-  capture_opts->file_duration           = 60;               /* 1 min */
-  capture_opts->has_ring_num_files      = TRUE;
-  capture_opts->ring_num_files          = 2;
-
-  capture_opts->has_autostop_files      = FALSE;
-  capture_opts->autostop_files          = 1;
-  capture_opts->has_autostop_packets    = FALSE;
-  capture_opts->autostop_packets        = 1;
-  capture_opts->has_autostop_filesize   = FALSE;
-  capture_opts->autostop_filesize       = 1024 * 1024;      /* 1 MB */
-  capture_opts->has_autostop_duration   = FALSE;
-  capture_opts->autostop_duration       = 60;               /* 1 min */
+  /* close the currently loaded capture file */
+  cf_close(capture_opts->cf);
+
+  g_assert(capture_opts->state == CAPTURE_STOPPED);
+  capture_opts->state = CAPTURE_PREPARING;
+
+  g_log(LOG_DOMAIN_CAPTURE, G_LOG_LEVEL_MESSAGE, "Capture Start ...");
+
+  /* try to start the capture child process */
+  ret = sync_pipe_start(capture_opts);
+  if(!ret) {
+      if(capture_opts->save_file != NULL) {
+          g_free(capture_opts->save_file);
+          capture_opts->save_file = NULL;
+      }
+
+      capture_opts->state = CAPTURE_STOPPED;
+  } else {
+      /* the capture child might not respond shortly after bringing it up */
+      /* (especially it will block, if no input coming from an input capture pipe (e.g. mkfifo) is coming in) */
 
+      /* to prevent problems, bring the main GUI into "capture mode" right after successfully */
+      /* spawn/exec the capture child, without waiting for any response from it */
+      cf_callback_invoke(cf_cb_live_capture_prepared, capture_opts);
+  }
+
+  return ret;
 }
 
-static int
-get_natural_int(const char *string, const char *name)
+
+void
+capture_stop(capture_options *capture_opts)
 {
-  long number;
-  char *p;
-
-  number = strtol(string, &p, 10);
-  if (p == string || *p != '\0') {
-    fprintf(stderr, "ethereal: The specified %s \"%s\" isn't a decimal number\n",
-           name, string);
-    exit(1);
-  }
-  if (number < 0) {
-    fprintf(stderr, "ethereal: The specified %s \"%s\" is a negative number\n",
-           name, string);
-    exit(1);
-  }
-  if (number > INT_MAX) {
-    fprintf(stderr, "ethereal: The specified %s \"%s\" is too large (greater than %d)\n",
-           name, string, INT_MAX);
-    exit(1);
-  }
-  return number;
+  g_log(LOG_DOMAIN_CAPTURE, G_LOG_LEVEL_MESSAGE, "Capture Stop ...");
+
+  /* stop the capture child gracefully */
+  sync_pipe_stop(capture_opts);
 }
 
 
-static int
-get_positive_int(const char *string, const char *name)
+void
+capture_restart(capture_options *capture_opts)
 {
-  long number;
+    g_log(LOG_DOMAIN_CAPTURE, G_LOG_LEVEL_MESSAGE, "Capture Restart");
 
-  number = get_natural_int(string, name);
+    capture_opts->restart = TRUE;
+    capture_stop(capture_opts);
+}
 
-  if (number == 0) {
-    fprintf(stderr, "ethereal: The specified %s is zero\n",
-           name);
-    exit(1);
-  }
 
-  return number;
+void
+capture_kill_child(capture_options *capture_opts)
+{
+  g_log(LOG_DOMAIN_CAPTURE, G_LOG_LEVEL_INFO, "Capture Kill");
+
+  /* kill the capture child */
+  sync_pipe_kill(capture_opts);
 }
 
 
-/*
- * Given a string of the form "<autostop criterion>:<value>", as might appear
- * as an argument to a "-a" option, parse it and set the criterion in
- * question.  Return an indication of whether it succeeded or failed
- * in some fashion.
- */
+
+/* We've succeeded a (non real-time) capture, try to read it into a new capture file */
 static gboolean
-set_autostop_criterion(capture_options *capture_opts, const char *autostoparg)
+capture_input_read_all(capture_options *capture_opts, gboolean is_tempfile, gboolean drops_known,
+guint32 drops)
 {
-  gchar *p, *colonp;
+  int err;
 
-  colonp = strchr(autostoparg, ':');
-  if (colonp == NULL)
-    return FALSE;
 
-  p = colonp;
-  *p++ = '\0';
-
-  /*
-   * Skip over any white space (there probably won't be any, but
-   * as we allow it in the preferences file, we might as well
-   * allow it here).
-   */
-  while (isspace((guchar)*p))
-    p++;
-  if (*p == '\0') {
-    /*
-     * Put the colon back, so if our caller uses, in an
-     * error message, the string they passed us, the message
-     * looks correct.
-     */
-    *colonp = ':';
-    return FALSE;
-  }
-  if (strcmp(autostoparg,"duration") == 0) {
-    capture_opts->has_autostop_duration = TRUE;
-    capture_opts->autostop_duration = get_positive_int(p,"autostop duration");
-  } else if (strcmp(autostoparg,"filesize") == 0) {
-    capture_opts->has_autostop_filesize = TRUE;
-    capture_opts->autostop_filesize = get_positive_int(p,"autostop filesize");
-  } else {
+  /* Capture succeeded; attempt to open the capture file. */
+  if (cf_open(capture_opts->cf, capture_opts->save_file, is_tempfile, &err) != CF_OK) {
+    /* We're not doing a capture any more, so we don't have a save
+       file. */
     return FALSE;
   }
-  *colonp = ':'; /* put the colon back */
-  return TRUE;
-}
 
-/*
- * Given a string of the form "<ring buffer file>:<duration>", as might appear
- * as an argument to a "-b" option, parse it and set the arguments in
- * question.  Return an indication of whether it succeeded or failed
- * in some fashion.
- */
-static gboolean
-get_ring_arguments(capture_options *capture_opts, const char *arg)
-{
-  gchar *p = NULL, *colonp;
+  /* Set the read filter to NULL. */
+  /* XXX - this is odd here, try to put it somewhere, where it fits better */
+  cf_set_rfcode(capture_opts->cf, NULL);
+
+  /* Get the packet-drop statistics.
+
+     XXX - there are currently no packet-drop statistics stored
+     in libpcap captures, and that's what we're reading.
+
+     At some point, we will add support in Wiretap to return
+     packet-drop statistics for capture file formats that store it,
+     and will make "cf_read()" get those statistics from Wiretap.
+     We clear the statistics (marking them as "not known") in
+     "cf_open()", and "cf_read()" will only fetch them and mark
+     them as known if Wiretap supplies them, so if we get the
+     statistics now, after calling "cf_open()" but before calling
+     "cf_read()", the values we store will be used by "cf_read()".
+
+     If a future libpcap capture file format stores the statistics,
+     we'll put them into the capture file that we write, and will
+     thus not have to set them here - "cf_read()" will get them from
+     the file and use them. */
+  if (drops_known) {
+    cf_set_drops_known(capture_opts->cf, TRUE);
+
+    /* XXX - on some systems, libpcap doesn't bother filling in
+       "ps_ifdrop" - it doesn't even set it to zero - so we don't
+       bother looking at it.
+
+       Ideally, libpcap would have an interface that gave us
+       several statistics - perhaps including various interface
+       error statistics - and would tell us which of them it
+       supplies, allowing us to display only the ones it does. */
+    cf_set_drops(capture_opts->cf, drops);
+  }
 
-  colonp = strchr(arg, ':');
+  /* read in the packet data */
+  switch (cf_read(capture_opts->cf)) {
 
-  if (colonp != NULL) {
-    p = colonp;
-    *p++ = '\0';
-  }
+  case CF_READ_OK:
+  case CF_READ_ERROR:
+    /* Just because we got an error, that doesn't mean we were unable
+       to read any of the file; we handle what we could get from the
+       file. */
+    break;
 
-  capture_opts->ring_num_files = 
-    get_natural_int(arg, "number of ring buffer files");
-
-  if (colonp == NULL)
-    return TRUE;
-
-  /*
-   * Skip over any white space (there probably won't be any, but
-   * as we allow it in the preferences file, we might as well
-   * allow it here).
-   */
-  while (isspace((guchar)*p))
-    p++;
-  if (*p == '\0') {
-    /*
-     * Put the colon back, so if our caller uses, in an
-     * error message, the string they passed us, the message
-     * looks correct.
-     */
-    *colonp = ':';
+  case CF_READ_ABORTED:
+    /* User wants to quit program. Exit by leaving the main loop, 
+       so that any quit functions we registered get called. */
+    main_window_nested_quit();
     return FALSE;
   }
 
-  capture_opts->has_file_duration = TRUE;
-  capture_opts->file_duration = get_positive_int(p,
-                                                     "ring buffer duration");
-
-  *colonp = ':';       /* put the colon back */
+  /* if we didn't captured even a single packet, close the file again */
+  if(cf_packet_count(capture_opts->cf) == 0 && !capture_opts->restart) {
+    simple_dialog(ESD_TYPE_INFO, ESD_BTN_OK, 
+    "%sNo packets captured!%s\n\n"
+    "As no data was captured, closing the %scapture file!",
+    simple_dialog_primary_start(), simple_dialog_primary_end(),
+    (cf_is_tempfile(capture_opts->cf)) ? "temporary " : "");
+    cf_close(capture_opts->cf);
+  }
   return TRUE;
 }
 
 
-void
-capture_opt_add(capture_options *capture_opts, int opt, const char *optarg, gboolean *start_capture)
+/* capture child tells us, we have a new (or the first) capture file */
+gboolean
+capture_input_new_file(capture_options *capture_opts, gchar *new_file)
 {
-#ifdef _WIN32
-    int i;
-#endif
+  gboolean is_tempfile;
+  int  err;
 
-    switch(opt) {
-    case 'a':        /* autostop criteria */
-        if (set_autostop_criterion(capture_opts, optarg) == FALSE) {
-          fprintf(stderr, "ethereal: Invalid or unknown -a flag \"%s\"\n", optarg);
-          exit(1);
-        }
-        break;
-    case 'b':        /* Ringbuffer option */
-        capture_opts->multi_files_on = TRUE;
-        capture_opts->has_ring_num_files = TRUE;
-        if (get_ring_arguments(capture_opts, optarg) == FALSE) {
-          fprintf(stderr, "ethereal: Invalid or unknown -b arg \"%s\"\n", optarg);
-          exit(1);
-        }
-        break;
-    case 'c':        /* Capture xxx packets */
-        capture_opts->has_autostop_packets = TRUE;
-        capture_opts->autostop_packets = get_positive_int(optarg, "packet count");
-        break;
-    case 'f':        /* capture filter */
-        if (capture_opts->cfilter)
-            g_free(capture_opts->cfilter);
-        capture_opts->cfilter = g_strdup(optarg);
-        break;
-    case 'H':        /* Hide capture info dialog box */
-        capture_opts->show_info = FALSE;
-        break;
-    case 'i':        /* Use interface xxx */
-        capture_opts->iface = g_strdup(optarg);
-        break;
-    case 'k':        /* Start capture immediately */
-        *start_capture = TRUE;
-        break;
-    /*case 'l':*/    /* Automatic scrolling in live capture mode */
-    case 'p':        /* Don't capture in promiscuous mode */
-        capture_opts->promisc_mode = FALSE;
-        break;
-    case 'Q':        /* Quit after capture (just capture to file) */
-        capture_opts->quit_after_cap  = TRUE;
-        *start_capture   = TRUE;  /*** -Q implies -k !! ***/
-        break;
-    case 's':        /* Set the snapshot (capture) length */
-        capture_opts->has_snaplen = TRUE;
-        capture_opts->snaplen = get_positive_int(optarg, "snapshot length");
-        break;
-    case 'S':        /* "Sync" mode: used for following file ala tail -f */
-        capture_opts->sync_mode = TRUE;
-        break;
-    case 'w':        /* Write to capture file xxx */
-        capture_opts->save_file = g_strdup(optarg);
-           break;
-    case 'W':        /* Write to capture file FD xxx */
-        capture_opts->save_file_fd = atoi(optarg);
-        break;
-    case 'y':        /* Set the pcap data link type */
-#ifdef HAVE_PCAP_DATALINK_NAME_TO_VAL
-        capture_opts->linktype = pcap_datalink_name_to_val(optarg);
-        if (capture_opts->linktype == -1) {
-          fprintf(stderr, "ethereal: The specified data link type \"%s\" isn't valid\n",
-                  optarg);
-          exit(1);
-        }
-#else /* HAVE_PCAP_DATALINK_NAME_TO_VAL */
-        /* XXX - just treat it as a number */
-        capture_opts->linktype = get_natural_int(optarg, "data link type");
-#endif /* HAVE_PCAP_DATALINK_NAME_TO_VAL */
-        break;
-#ifdef _WIN32
-      /* Hidden option supporting Sync mode */
-    case 'Z':        /* Write to pipe FD XXX */
-        /* associate stdout with pipe */
-        i = atoi(optarg);
-        if (dup2(i, 1) < 0) {
-          fprintf(stderr, "Unable to dup pipe handle\n");
-          exit(1);
-        }
-        break;
-#endif /* _WIN32 */
-    default:
-        /* the caller is responsible to send us only the right opt's */
-        g_assert_not_reached();
-    }
-}
 
-/* open the output file (temporary/specified name/ringbuffer) and close the old one */
-/* Returns TRUE if the file opened successfully, FALSE otherwise. */
-static gboolean
-capture_open_output(capture_options *capture_opts, gboolean *is_tempfile) {
-  char tmpname[128+1];
-  gchar *capfile_name;
-
-
-  if (capture_opts->save_file != NULL) {
-    /* If the Sync option is set, we return to the caller while the capture
-     * is in progress.  Therefore we need to take a copy of save_file in
-     * case the caller destroys it after we return.
-     */
-    capfile_name = g_strdup(capture_opts->save_file);
-    if (capture_opts->multi_files_on) {
-      /* ringbuffer is enabled */
-      capture_opts->save_file_fd = ringbuf_init(capfile_name,
-          (capture_opts->has_ring_num_files) ? capture_opts->ring_num_files : 0);
-    } else {
-      /* Try to open/create the specified file for use as a capture buffer. */
-      capture_opts->save_file_fd = open(capfile_name, O_RDWR|O_BINARY|O_TRUNC|O_CREAT,
-                               0600);
-    }
-    *is_tempfile = FALSE;
-  } else {
-    /* Choose a random name for the temporary capture buffer */
-    capture_opts->save_file_fd = create_tempfile(tmpname, sizeof tmpname, "ether");
-    capfile_name = g_strdup(tmpname);
-    *is_tempfile = TRUE;
+  if(capture_opts->state == CAPTURE_PREPARING) {
+    g_log(LOG_DOMAIN_CAPTURE, G_LOG_LEVEL_MESSAGE, "Capture started!");
   }
+  g_log(LOG_DOMAIN_CAPTURE, G_LOG_LEVEL_MESSAGE, "File: \"%s\"", new_file);
 
-  /* did we fail to open the output file? */
-  if (capture_opts->save_file_fd == -1) {
-    if (is_tempfile) {
-      simple_dialog(ESD_TYPE_ERROR, ESD_BTN_OK,
-       "The temporary file to which the capture would be saved (\"%s\")"
-       "could not be opened: %s.", capfile_name, strerror(errno));
-    } else {
-      if (capture_opts->multi_files_on) {
-        ringbuf_error_cleanup();
-      }
-      open_failure_alert_box(capfile_name, errno, TRUE);
-    }
-    g_free(capfile_name);
-    return FALSE;
-  }
+  g_assert(capture_opts->state == CAPTURE_PREPARING || capture_opts->state == CAPTURE_RUNNING);
 
-  /* close the old file */
-  cf_close(capture_opts->cf);
+  /* free the old filename */
   if(capture_opts->save_file != NULL) {
+    /* we start a new capture file, close the old one (if we had one before) */
+    if( ((capture_file *) capture_opts->cf)->state != FILE_CLOSED) {
+        cf_callback_invoke(cf_cb_live_capture_update_finished, capture_opts->cf);
+        cf_finish_tail(capture_opts->cf, &err);
+        cf_close(capture_opts->cf);
+    }
     g_free(capture_opts->save_file);
+    is_tempfile = FALSE;
+    cf_set_tempfile(capture_opts->cf, FALSE);
+  } else {
+    /* we didn't had a save_file before, must be a tempfile */
+    is_tempfile = TRUE;
+    cf_set_tempfile(capture_opts->cf, TRUE);
   }
-  capture_opts->save_file = capfile_name;
-  /* capture_opts.save_file is "g_free"ed later, which is equivalent to
-     "g_free(capfile_name)". */
 
-  return TRUE;
-}
-
-
-/* Open a specified file, or create a temporary file, and start a capture
-   to the file in question.  */
-/* Returns TRUE if the capture starts successfully, FALSE otherwise. */
-gboolean
-do_capture(capture_options *capture_opts)
-{
-  gboolean is_tempfile;
-  gboolean ret;
+  /* save the new filename */
+  capture_opts->save_file = g_strdup(new_file);
 
-  /* open the output file (temporary/specified name/ringbuffer) and close the old one */
-  if(!capture_open_output(capture_opts, &is_tempfile)) {
-    return FALSE;
-  }
+  /* if we are in real-time mode, open the new file now */
+  if(capture_opts->real_time_mode) {
+    /* Attempt to open the capture file and set up to read from it. */
+       switch(cf_start_tail(capture_opts->cf, capture_opts->save_file, is_tempfile, &err)) {
+    case CF_OK:
+      break;
+    case CF_ERROR:
+      /* Don't unlink (delete) the save file - leave it around, 
+         for debugging purposes. */
+      g_free(capture_opts->save_file);
+      capture_opts->save_file = NULL;
+      return FALSE;
+      break;
+    }
 
-  if (capture_opts->sync_mode) {       
-    /* sync mode: do the capture in a child process */
-    ret = sync_pipe_do_capture(capture_opts, is_tempfile);
-    /* capture is still running */
-    cf_callback_invoke(cf_cb_live_capture_prepare, capture_opts);
+    cf_callback_invoke(cf_cb_live_capture_update_started, capture_opts);
   } else {
-    /* normal mode: do the capture synchronously */
-    cf_callback_invoke(cf_cb_live_capture_prepare, capture_opts);
-    ret = normal_do_capture(capture_opts, is_tempfile);
-    /* capture is finished here */
+    cf_callback_invoke(cf_cb_live_capture_fixed_started, capture_opts);
   }
 
-  return ret;
-}
+  capture_opts->state = CAPTURE_RUNNING;
 
+  return TRUE;
+}
 
-/* start a normal capture session */
-static gboolean
-normal_do_capture(capture_options *capture_opts, gboolean is_tempfile)
+    
+/* capture child tells us, we have new packets to read */
+void
+capture_input_new_packets(capture_options *capture_opts, int to_read)
 {
-    int capture_succeeded;
-    gboolean stats_known;
-    struct pcap_stat stats;
-    int err;
-
-    /* Not sync mode. */
-    capture_succeeded = capture_start(capture_opts, &stats_known, &stats);
-    if (capture_opts->quit_after_cap) {
-      /* DON'T unlink the save file.  Presumably someone wants it. */
-        main_window_exit();
-    }
-    if (!capture_succeeded) {
-      /* We didn't succeed in doing the capture, so we don't have a save
-        file. */
-      if (capture_opts->multi_files_on) {
-       ringbuf_free();
-      } else {
-       g_free(capture_opts->save_file);
-      }
-      capture_opts->save_file = NULL;
-      return FALSE;
-    }
-    /* Capture succeeded; attempt to read in the capture file. */
-    if (cf_open(capture_opts->cf, capture_opts->save_file, is_tempfile, &err) != CF_OK) {
-      /* We're not doing a capture any more, so we don't have a save
-        file. */
-      if (capture_opts->multi_files_on) {
-       ringbuf_free();
-      } else {
-       g_free(capture_opts->save_file);
-      }
-      capture_opts->save_file = NULL;
-      return FALSE;
-    }
+  int  err;
 
-    /* Set the read filter to NULL. */
-    cf_set_rfcode(capture_opts->cf, NULL);
-
-    /* Get the packet-drop statistics.
-
-       XXX - there are currently no packet-drop statistics stored
-       in libpcap captures, and that's what we're reading.
-
-       At some point, we will add support in Wiretap to return
-       packet-drop statistics for capture file formats that store it,
-       and will make "cf_read()" get those statistics from Wiretap.
-       We clear the statistics (marking them as "not known") in
-       "cf_open()", and "cf_read()" will only fetch them and mark
-       them as known if Wiretap supplies them, so if we get the
-       statistics now, after calling "cf_open()" but before calling
-       "cf_read()", the values we store will be used by "cf_read()".
-
-       If a future libpcap capture file format stores the statistics,
-       we'll put them into the capture file that we write, and will
-       thus not have to set them here - "cf_read()" will get them from
-       the file and use them. */
-    if (stats_known) {
-      cf_set_drops_known(capture_opts->cf, TRUE);
-
-      /* XXX - on some systems, libpcap doesn't bother filling in
-         "ps_ifdrop" - it doesn't even set it to zero - so we don't
-         bother looking at it.
-
-         Ideally, libpcap would have an interface that gave us
-         several statistics - perhaps including various interface
-         error statistics - and would tell us which of them it
-         supplies, allowing us to display only the ones it does. */
-      cf_set_drops(capture_opts->cf, stats.ps_drop);
-    }
-    switch (cf_read(capture_opts->cf)) {
+
+  g_assert(capture_opts->save_file);
+
+  if(capture_opts->real_time_mode) {
+    /* Read from the capture file the number of records the child told us
+       it added.
+       XXX - do something if this fails? */
+    switch (cf_continue_tail(capture_opts->cf, to_read, &err)) {
 
     case CF_READ_OK:
     case CF_READ_ERROR:
       /* Just because we got an error, that doesn't mean we were unable
          to read any of the file; we handle what we could get from the
-         file. */
+         file.
+
+         XXX - abort on a read error? */
+         cf_callback_invoke(cf_cb_live_capture_update_continue, capture_opts->cf);
+         main_window_update();
       break;
 
     case CF_READ_ABORTED:
-      /* Exit by leaving the main loop, so that any quit functions
-         we registered get called. */
-      main_window_nested_quit();
-      return FALSE;
-    }
-
-    /* We're not doing a capture any more, so we don't have a save
-       file. */
-    if (capture_opts->multi_files_on) {
-      ringbuf_free();
-    } else {
-      g_free(capture_opts->save_file);
-    }
-    capture_opts->save_file = NULL;
-
-    /* if we didn't captured even a single packet, close the file again */
-    if(cf_packet_count(capture_opts->cf) == 0) {
-      simple_dialog(ESD_TYPE_INFO, ESD_BTN_OK, 
-      "%sNo packets captured!%s\n\n"
-      "As no data was captured, closing the %scapture file!",
-      simple_dialog_primary_start(), simple_dialog_primary_end(),
-      (cf_is_tempfile(capture_opts->cf)) ? "temporary " : "");
-      cf_close(capture_opts->cf);
+      /* Kill the child capture process; the user wants to exit, and we
+         shouldn't just leave it running. */
+      capture_kill_child(capture_opts);
+      break;
     }
-  return TRUE;
+  }
 }
 
 
-static void
-stop_capture_signal_handler(int signo _U_)
+/* capture child closed it's side ot the pipe, do the required cleanup */
+void
+capture_input_closed(capture_options *capture_opts)
 {
-  capture_loop_stop();
-}
+    int  err;
 
 
-int  
-capture_start(capture_options *capture_opts, gboolean *stats_known, struct pcap_stat *stats)
-{
-#ifndef _WIN32
-  /*
-   * Catch SIGUSR1, so that we exit cleanly if the parent process
-   * kills us with it due to the user selecting "Capture->Stop".
-   */
-  if (capture_opts->capture_child)
-    signal(SIGUSR1, stop_capture_signal_handler);
-#endif
+    g_log(LOG_DOMAIN_CAPTURE, G_LOG_LEVEL_MESSAGE, "Capture stopped!");
+    g_assert(capture_opts->state == CAPTURE_PREPARING || capture_opts->state == CAPTURE_RUNNING);
 
-  return capture_loop_start(capture_opts, stats_known, stats);
-}
+    /* if we didn't started the capture (happens if an error occured), do a fake start */
+    if(capture_opts->state == CAPTURE_PREPARING) {
+        if(capture_opts->real_time_mode) {
+            cf_callback_invoke(cf_cb_live_capture_update_started, capture_opts);
+        } else {
+            cf_callback_invoke(cf_cb_live_capture_fixed_started, capture_opts);
+        }
+    }
 
-void
-capture_stop(capture_options *capture_opts)
-{
+    if(capture_opts->real_time_mode) {
+        /* first of all, we are not doing a capture any more */
+        cf_callback_invoke(cf_cb_live_capture_update_finished, capture_opts->cf);
+
+        /* Read what remains of the capture file, and finish the capture.
+           XXX - do something if this fails? */
+        switch (cf_finish_tail(capture_opts->cf, &err)) {
+
+        case CF_READ_OK:
+            if(cf_packet_count(capture_opts->cf) == 0 && !capture_opts->restart) {
+              simple_dialog(ESD_TYPE_INFO, ESD_BTN_OK, 
+              "%sNo packets captured!%s\n\n"
+              "As no data was captured, closing the %scapture file!",
+              simple_dialog_primary_start(), simple_dialog_primary_end(),
+              cf_is_tempfile(capture_opts->cf) ? "temporary " : "");
+              cf_close(capture_opts->cf);
+            }
+            break;
+        case CF_READ_ERROR:
+          /* Just because we got an error, that doesn't mean we were unable
+             to read any of the file; we handle what we could get from the
+             file. */
+          break;
+
+        case CF_READ_ABORTED:
+          /* Exit by leaving the main loop, so that any quit functions
+             we registered get called. */
+          main_window_quit();
+        }
 
-  if (capture_opts->sync_mode) {       
-    sync_pipe_stop(capture_opts);
-  }
-    
-  capture_loop_stop();
-}
+    } else {
+        /* first of all, we are not doing a capture any more */
+        cf_callback_invoke(cf_cb_live_capture_fixed_finished, capture_opts->cf);
 
-void
-capture_kill_child(capture_options *capture_opts)
-{
-  if (capture_opts->sync_mode) {       
-    sync_pipe_kill(capture_opts);
-  }
+        /* this is a normal mode capture and if no error happened, read in the capture file data */
+        if(capture_opts->save_file != NULL) {
+            capture_input_read_all(capture_opts, cf_is_tempfile(capture_opts->cf), 
+                cf_get_drops_known(capture_opts->cf), cf_get_drops(capture_opts->cf));
+        }
+    }
+
+    capture_opts->state = CAPTURE_STOPPED;
+
+    /* if we couldn't open a capture file, there's nothing more for us to do */
+    if(capture_opts->save_file == NULL) {
+        cf_close(capture_opts->cf);
+        return;
+    }
+
+    /* does the user wants to restart the current capture? */
+    if(capture_opts->restart) {
+        capture_opts->restart = FALSE;
+
+        unlink(capture_opts->save_file);
+
+        /* if it was a tempfile, throw away the old filename (so it will become a tempfile again) */
+        if(cf_is_tempfile(capture_opts->cf)) {
+            g_free(capture_opts->save_file);
+            capture_opts->save_file = NULL;
+        }
+
+        /* ... and start the capture again */
+        capture_start(capture_opts);
+    } else {
+        /* We're not doing a capture any more, so we don't have a save file. */
+        g_free(capture_opts->save_file);
+        capture_opts->save_file = NULL;
+    }
 }