fix doxygen generation
[obnox/wireshark/wip.git] / capture.c
index 74ee00f8e09e074abd4e798877a0f31b473776e3..e848d29d6d27a2033d1ce92c02641fad223fa716 100644 (file)
--- a/capture.c
+++ b/capture.c
@@ -1,7 +1,7 @@
 /* capture.c
  * Routines for packet capture windows
  *
- * $Id: capture.c,v 1.207 2003/05/15 13:33:53 deniel Exp $
+ * $Id$
  *
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@ethereal.com>
  * 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
@@ -68,7 +72,9 @@
 # include <io.h>
 #endif
 
-#include <gtk/gtk.h>
+#include <pcap.h>
+
+#include <glib.h>
 #include <stdlib.h>
 #include <stdio.h>
 #include <ctype.h>
 #include <signal.h>
 #include <errno.h>
 
-#include <pcap.h>
-
 #ifdef NEED_SNPRINTF_H
 # include "snprintf.h"
 #endif
 #include <process.h>    /* For spawning child process */
 #endif
 
-/*
- * XXX - the various BSDs appear to define BSD in <sys/param.h>; we don't
- * want to include it if it's not present on this platform, however.
- */
-#if defined(__FreeBSD__) || defined(__NetBSD__) || defined(__OpenBSD__) || defined(__bsdi__) || defined(__APPLE__)
-#ifndef BSD
-#define BSD
-#endif /* BSD */
-#endif /* defined(__FreeBSD__) || defined(__NetBSD__) || defined(__OpenBSD__) || defined(__bsdi__) || defined(__APPLE__) */
-
 /*
  * We don't want to do a "select()" on the pcap_t's file descriptor on
  * BSD (because "select()" doesn't work correctly on BPF devices on at
  * least some releases of some flavors of BSD), and we don't want to do
  * it on Windows (because "select()" is something for sockets, not for
- * arbitrary handles).
+ * arbitrary handles).  (Note that "Windows" here includes Cygwin;
+ * even in its pretend-it's-UNIX environment, we're using WinPcap, not
+ * a UNIX libpcap.)
  *
  * We *do* want to do it on other platforms, as, on other platforms (with
  * the possible exception of Ultrix and Digital UNIX), the read timeout
  * doesn't expire if no packets have arrived, so a "pcap_dispatch()" call
  * will block until packets arrive, causing the UI to hang.
+ *
+ * XXX - the various BSDs appear to define BSD in <sys/param.h>; we don't
+ * want to include it if it's not present on this platform, however.
  */
-#if !defined(BSD) && !defined(_WIN32)
+#if !defined(__FreeBSD__) && !defined(__NetBSD__) && !defined(__OpenBSD__) && \
+    !defined(__bsdi__) && !defined(__APPLE__) && !defined(_WIN32) && \
+    !defined(__CYGWIN__)
 # define MUST_DO_SELECT
 #endif
 
 #include <epan/packet.h>
+#include <epan/dfilter/dfilter.h>
 #include "file.h"
 #include "capture.h"
 #include "util.h"
 #include "pcap-util.h"
+#include "alert_box.h"
 #include "simple_dialog.h"
-#include "prefs.h"
+#include <epan/prefs.h>
 #include "globals.h"
 #include "conditions.h"
 #include "capture_stop_conditions.h"
 #include "wiretap/wtap.h"
 #include "wiretap/wtap-capture.h"
 
-#include "packet-atalk.h"
-#include "packet-atm.h"
-#include "packet-clip.h"
-#include "packet-eth.h"
-#include "packet-fddi.h"
-#include "packet-null.h"
-#include "packet-ppp.h"
-#include "packet-raw.h"
-#include "packet-sll.h"
-#include "packet-tr.h"
-#include "packet-ieee80211.h"
-#include "packet-chdlc.h"
-#include "packet-prism.h"
-#include "packet-ipfc.h"
-#include "packet-arcnet.h"
+#include <epan/dissectors/packet-ap1394.h>
+#include <epan/dissectors/packet-atalk.h>
+#include <epan/dissectors/packet-atm.h>
+#include <epan/dissectors/packet-clip.h>
+#include <epan/dissectors/packet-eth.h>
+#include <epan/dissectors/packet-fddi.h>
+#include <epan/dissectors/packet-null.h>
+#include <epan/dissectors/packet-ppp.h>
+#include <epan/dissectors/packet-raw.h>
+#include <epan/dissectors/packet-sll.h>
+#include <epan/dissectors/packet-tr.h>
+#include <epan/dissectors/packet-ieee80211.h>
+#include <epan/dissectors/packet-chdlc.h>
+#include <epan/dissectors/packet-prism.h>
+#include <epan/dissectors/packet-ipfc.h>
+#include <epan/dissectors/packet-arcnet.h>
 
 #ifdef _WIN32
 #include "capture-wpcap.h"
 #endif
+#include "ui_util.h"
 
 /*
  * Capture options.
  */
 capture_options capture_opts;
+gboolean quit_after_cap = FALSE;/* Makes a "capture only mode". Implies -k */
+gboolean capture_child;                /* if this is the child for "-S" */
+
+static int sync_pipe[2];        /* used to sync father */
+enum PIPES { READ, WRITE };     /* Constants 0 and 1 for READ and WRITE */
+static int fork_child = -1;        /* If not -1, in parent, process ID of child */
 
-static int sync_pipe[2]; /* used to sync father */
-enum PIPES { READ, WRITE }; /* Constants 0 and 1 for READ and WRITE */
-int quit_after_cap; /* Makes a "capture only mode". Implies -k */
-gboolean capture_child;        /* if this is the child for "-S" */
-static int fork_child = -1;    /* If not -1, in parent, process ID of child */
-static guint cap_input_id;
+/* Size of buffer to hold decimal representation of
+   signed/unsigned 64-bit int */
+#define SP_DECISIZE 20
 
 /*
  * Indications sent out on the sync pipe.
  */
-#define SP_CAPSTART    ';'     /* capture start message */
+#define SP_CAPSTART    ';'         /* capture start message */
 #define SP_PACKET_COUNT        '*'     /* followed by count of packets captured since last message */
 #define SP_ERROR_MSG   '!'     /* followed by length of error message that follows */
-#define SP_DROPS       '#'     /* followed by count of packets dropped in capture */
-
-#ifdef _WIN32
-static guint cap_timer_id;
-static int cap_timer_cb(gpointer); /* Win32 kludge to check for pipe input */
-#endif
+#define SP_DROPS       '#'         /* followed by count of packets dropped in capture */
 
-static void cap_file_input_cb(gpointer, gint, GdkInputCondition);
-static void wait_for_child(gboolean);
-#ifndef _WIN32
-static char *signame(int);
-#endif
-static void capture_delete_cb(GtkWidget *, GdkEvent *, gpointer);
-static void capture_stop_cb(GtkWidget *, gpointer);
-static void capture_pcap_cb(guchar *, const struct pcap_pkthdr *,
-  const guchar *);
-static void get_capture_file_io_error(char *, int, const char *, int, gboolean);
-static void popup_errmsg(const char *);
-static void send_errmsg_to_parent(const char *);
-static float pct(gint, gint);
-static void stop_capture(int signo);
 
 typedef struct _loop_data {
   gboolean       go;           /* TRUE as long as we're supposed to keep capturing */
@@ -214,94 +203,64 @@ typedef struct _loop_data {
   gint           linktype;
   gint           sync_packets;
   gboolean       pcap_err;     /* TRUE if error from pcap */
-  gboolean       from_pipe;    /* TRUE if we are capturing data from a pipe */
+  gboolean       from_cap_pipe;/* TRUE if we are capturing data from a pipe */
   packet_counts  counts;
   wtap_dumper   *pdh;
 #ifndef _WIN32
   gboolean       modified;     /* TRUE if data in the pipe uses modified pcap headers */
   gboolean       byte_swapped; /* TRUE if data in the pipe is byte swapped */
-  unsigned int   bytes_to_read, bytes_read; /* Used by pipe_dispatch */
+  unsigned int   bytes_to_read, bytes_read; /* Used by cap_pipe_dispatch */
   enum {
          STATE_EXPECT_REC_HDR, STATE_READ_REC_HDR,
          STATE_EXPECT_DATA,     STATE_READ_DATA
-       } pipe_state;
+       } cap_pipe_state;
 
-  enum { PIPOK, PIPEOF, PIPERR, PIPNEXIST } pipe_err;
+  enum { PIPOK, PIPEOF, PIPERR, PIPNEXIST } cap_pipe_err;
 #endif
 } loop_data;
 
-#ifndef _WIN32
-static void adjust_header(loop_data *, struct pcap_hdr *, struct pcaprec_hdr *);
-static int pipe_open_live(char *, struct pcap_hdr *, loop_data *, char *, int);
-static int pipe_dispatch(int, loop_data *, struct pcap_hdr *, \
-               struct pcaprec_modified_hdr *, guchar *, char *, int);
-#endif
-
 /* Win32 needs the O_BINARY flag for open() */
 #ifndef O_BINARY
 #define O_BINARY       0
 #endif
 
-#ifdef _WIN32
-/* Win32 needs a handle to the child capture process */
-int child_process;
+static gboolean sync_pipe_do_capture(gboolean is_tempfile);
+static gboolean sync_pipe_input_cb(gint source, gpointer user_data);
+static void sync_pipe_wait_for_child(gboolean);
+static void sync_pipe_errmsg_to_parent(const char *);
+#ifndef _WIN32
+static char *sync_pipe_signame(int);
 #endif
 
-/* Add a string pointer to a NULL-terminated array of string pointers. */
-static char **
-add_arg(char **args, int *argc, char *arg)
-{
-  /* Grow the array; "*argc" currently contains the number of string
-     pointers, *not* counting the NULL pointer at the end, so we have
-     to add 2 in order to get the new size of the array, including the
-     new pointer and the terminating NULL pointer. */
-  args = g_realloc(args, (*argc + 2) * sizeof (char *));
+static gboolean normal_do_capture(gboolean is_tempfile);
+static void capture_pcap_cb(guchar *, const struct pcap_pkthdr *,
+  const guchar *);
+static void get_capture_file_io_error(char *, int, const char *, int, gboolean);
+static void popup_errmsg(const char *);
+static void stop_capture_signal_handler(int signo);
 
-  /* Stuff the pointer into the penultimate element of the array, which
-     is the one at the index specified by "*argc". */
-  args[*argc] = arg;
+#ifndef _WIN32
+static void cap_pipe_adjust_header(loop_data *, struct pcap_hdr *, struct pcaprec_hdr *);
+static int cap_pipe_open_live(char *, struct pcap_hdr *, loop_data *, char *, int);
+static int cap_pipe_dispatch(int, loop_data *, struct pcap_hdr *, \
+               struct pcaprec_modified_hdr *, guchar *, char *, int);
+#endif
 
-  /* Now bump the count. */
-  (*argc)++;
 
-  /* We overwrite the NULL pointer; put it back right after the
-     element we added. */
-  args[*argc] = NULL;
 
-  return args;
-}
 
-#ifdef _WIN32
-/* Given a string, return a pointer to a quote-encapsulated version of
-   the string, so we can pass it as an argument with "spawnvp" even
-   if it contains blanks. */
-char *
-quote_encapsulate(const char *string)
-{
-  char *encapsulated_string;
 
-  encapsulated_string = g_new(char, strlen(string) + 3);
-  sprintf(encapsulated_string, "\"%s\"", string);
-  return encapsulated_string;
-}
-#endif
 
 /* Open a specified file, or create a temporary file, and start a capture
-   to the file in question. */
-void
+   to the file in question.  Returns TRUE if the capture starts
+   successfully, FALSE otherwise. */
+gboolean
 do_capture(const char *save_file)
 {
   char tmpname[128+1];
   gboolean is_tempfile;
-  guchar c;
-  int i;
-  guint byte_count;
-  char *msg;
-  int err;
-  int capture_succeeded;
-  gboolean stats_known;
-  struct pcap_stat stats;
   gchar *capfile_name;
+  gboolean ret;
 
   if (save_file != NULL) {
     /* If the Sync option is set, we return to the caller while the capture
@@ -309,10 +268,10 @@ do_capture(const char *save_file)
      * case the caller destroys it after we return.
      */
     capfile_name = g_strdup(save_file);
-    if (capture_opts.ringbuffer_on) {
+    if (capture_opts.multi_files_on) {
       /* ringbuffer is enabled */
       cfile.save_file_fd = ringbuf_init(capfile_name,
-                                       capture_opts.ringbuffer_num_files);
+          (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. */
       cfile.save_file_fd = open(capfile_name, O_RDWR|O_BINARY|O_TRUNC|O_CREAT,
@@ -327,32 +286,98 @@ do_capture(const char *save_file)
   }
   if (cfile.save_file_fd == -1) {
     if (is_tempfile) {
-      simple_dialog(ESD_TYPE_CRIT, NULL,
+      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.ringbuffer_on) {
+      if (capture_opts.multi_files_on) {
         ringbuf_error_cleanup();
       }
-      simple_dialog(ESD_TYPE_CRIT, NULL,
-       file_open_error_message(errno, TRUE, WTAP_FILE_PCAP), capfile_name);
+      open_failure_alert_box(capfile_name, errno, TRUE);
     }
     g_free(capfile_name);
-    return;
+    return FALSE;
   }
-  close_cap_file(&cfile);
+  cf_close(&cfile);
   g_assert(cfile.save_file == NULL);
   cfile.save_file = capfile_name;
   /* cfile.save_file is "g_free"ed below, which is equivalent to
      "g_free(capfile_name)". */
+  fork_child = -1;
+
+  if (capture_opts.sync_mode) {        
+    /* sync mode: do the capture in a child process */
+    ret = sync_pipe_do_capture(is_tempfile);
+    /* capture is still running */
+    set_main_window_name("(Live Capture in Progress) - Ethereal");
+  } else {
+    /* normal mode: do the capture synchronously */
+    set_main_window_name("(Live Capture in Progress) - Ethereal");
+    ret = normal_do_capture(is_tempfile);
+    /* capture is finished here */
+  }
+
+  return ret;
+}
+
+
+
+/* Add a string pointer to a NULL-terminated array of string pointers. */
+static char **
+sync_pipe_add_arg(char **args, int *argc, char *arg)
+{
+  /* Grow the array; "*argc" currently contains the number of string
+     pointers, *not* counting the NULL pointer at the end, so we have
+     to add 2 in order to get the new size of the array, including the
+     new pointer and the terminating NULL pointer. */
+  args = g_realloc(args, (*argc + 2) * sizeof (char *));
+
+  /* Stuff the pointer into the penultimate element of the array, which
+     is the one at the index specified by "*argc". */
+  args[*argc] = arg;
+
+  /* Now bump the count. */
+  (*argc)++;
+
+  /* We overwrite the NULL pointer; put it back right after the
+     element we added. */
+  args[*argc] = NULL;
+
+  return args;
+}
+
+#ifdef _WIN32
+/* Given a string, return a pointer to a quote-encapsulated version of
+   the string, so we can pass it as an argument with "spawnvp" even
+   if it contains blanks. */
+char *
+sync_pipe_quote_encapsulate(const char *string)
+{
+  char *encapsulated_string;
+
+  encapsulated_string = g_new(char, strlen(string) + 3);
+  sprintf(encapsulated_string, "\"%s\"", string);
+  return encapsulated_string;
+}
+#endif
+
+
 
-  if (capture_opts.sync_mode) {        /* do the capture in a child process */
+static gboolean
+sync_pipe_do_capture(gboolean is_tempfile) {
+    guint byte_count;
+    int  i;
+    guchar  c;
+    char *msg;
+    int  err;
     char ssnap[24];
     char scount[24];                   /* need a constant for len of numbers */
     char sautostop_filesize[24];       /* need a constant for len of numbers */
     char sautostop_duration[24];       /* need a constant for len of numbers */
     char save_file_fd[24];
+#ifndef _WIN32
     char errmsg[1024+1];
+#endif
     int error;
     int argc;
     char **argv;
@@ -369,44 +394,59 @@ do_capture(const char *save_file)
     *argv = NULL;
 
     /* Now add those arguments used on all platforms. */
-    argv = add_arg(argv, &argc, CHILD_NAME);
+    argv = sync_pipe_add_arg(argv, &argc, CHILD_NAME);
 
-    argv = add_arg(argv, &argc, "-i");
-    argv = add_arg(argv, &argc, cfile.iface);
+    argv = sync_pipe_add_arg(argv, &argc, "-i");
+    argv = sync_pipe_add_arg(argv, &argc, cfile.iface);
 
-    argv = add_arg(argv, &argc, "-w");
-    argv = add_arg(argv, &argc, cfile.save_file);
+    argv = sync_pipe_add_arg(argv, &argc, "-w");
+    argv = sync_pipe_add_arg(argv, &argc, cfile.save_file);
 
-    argv = add_arg(argv, &argc, "-W");
+    argv = sync_pipe_add_arg(argv, &argc, "-W");
     sprintf(save_file_fd,"%d",cfile.save_file_fd);     /* in lieu of itoa */
-    argv = add_arg(argv, &argc, save_file_fd);
+    argv = sync_pipe_add_arg(argv, &argc, save_file_fd);
 
-    if (capture_opts.has_autostop_count) {
-      argv = add_arg(argv, &argc, "-c");
-      sprintf(scount,"%d",capture_opts.autostop_count);
-      argv = add_arg(argv, &argc, scount);
+    if (capture_opts.has_autostop_packets) {
+      argv = sync_pipe_add_arg(argv, &argc, "-c");
+      sprintf(scount,"%d",capture_opts.autostop_packets);
+      argv = sync_pipe_add_arg(argv, &argc, scount);
     }
 
     if (capture_opts.has_snaplen) {
-      argv = add_arg(argv, &argc, "-s");
+      argv = sync_pipe_add_arg(argv, &argc, "-s");
       sprintf(ssnap,"%d",capture_opts.snaplen);
-      argv = add_arg(argv, &argc, ssnap);
+      argv = sync_pipe_add_arg(argv, &argc, ssnap);
+    }
+
+    if (capture_opts.linktype != -1) {
+      argv = sync_pipe_add_arg(argv, &argc, "-y");
+#ifdef HAVE_PCAP_DATALINK_VAL_TO_NAME
+      sprintf(ssnap,"%s",pcap_datalink_val_to_name(capture_opts.linktype));
+#else
+      /* XXX - just treat it as a number */
+      sprintf(ssnap,"%d",capture_opts.linktype);
+#endif
+      argv = sync_pipe_add_arg(argv, &argc, ssnap);
     }
 
     if (capture_opts.has_autostop_filesize) {
-      argv = add_arg(argv, &argc, "-a");
+      argv = sync_pipe_add_arg(argv, &argc, "-a");
       sprintf(sautostop_filesize,"filesize:%d",capture_opts.autostop_filesize);
-      argv = add_arg(argv, &argc, sautostop_filesize);
+      argv = sync_pipe_add_arg(argv, &argc, sautostop_filesize);
     }
 
     if (capture_opts.has_autostop_duration) {
-      argv = add_arg(argv, &argc, "-a");
+      argv = sync_pipe_add_arg(argv, &argc, "-a");
       sprintf(sautostop_duration,"duration:%d",capture_opts.autostop_duration);
-      argv = add_arg(argv, &argc, sautostop_duration);
+      argv = sync_pipe_add_arg(argv, &argc, sautostop_duration);
+    }
+
+    if (!capture_opts.show_info) {
+      argv = sync_pipe_add_arg(argv, &argc, "-H");
     }
 
     if (!capture_opts.promisc_mode)
-      argv = add_arg(argv, &argc, "-p");
+      argv = sync_pipe_add_arg(argv, &argc, "-p");
 
 #ifdef _WIN32
     /* Create a pipe for the child process */
@@ -417,27 +457,27 @@ do_capture(const char *save_file)
       unlink(cfile.save_file);
       g_free(cfile.save_file);
       cfile.save_file = NULL;
-      simple_dialog(ESD_TYPE_CRIT, NULL, "Couldn't create sync pipe: %s",
+      simple_dialog(ESD_TYPE_ERROR, ESD_BTN_OK, "Couldn't create sync pipe: %s",
                         strerror(error));
-      return;
+      return FALSE;
     }
 
     /* Convert font name to a quote-encapsulated string and pass to child */
-    argv = add_arg(argv, &argc, "-m");
-    fontstring = quote_encapsulate(prefs.gui_font_name);
-    argv = add_arg(argv, &argc, fontstring);
+    argv = sync_pipe_add_arg(argv, &argc, "-m");
+    fontstring = sync_pipe_quote_encapsulate(prefs.PREFS_GUI_FONT_NAME);
+    argv = sync_pipe_add_arg(argv, &argc, fontstring);
 
     /* Convert pipe write handle to a string and pass to child */
-    argv = add_arg(argv, &argc, "-Z");
+    argv = sync_pipe_add_arg(argv, &argc, "-Z");
     itoa(sync_pipe[WRITE], sync_pipe_fd, 10);
-    argv = add_arg(argv, &argc, sync_pipe_fd);
+    argv = sync_pipe_add_arg(argv, &argc, sync_pipe_fd);
 
     /* Convert filter string to a quote delimited string and pass to child */
     filterstring = NULL;
     if (cfile.cfilter != NULL && strlen(cfile.cfilter) != 0) {
-      argv = add_arg(argv, &argc, "-f");
-      filterstring = quote_encapsulate(cfile.cfilter);
-      argv = add_arg(argv, &argc, filterstring);
+      argv = sync_pipe_add_arg(argv, &argc, "-f");
+      filterstring = sync_pipe_quote_encapsulate(cfile.cfilter);
+      argv = sync_pipe_add_arg(argv, &argc, filterstring);
     }
 
     /* Spawn process */
@@ -446,8 +486,6 @@ do_capture(const char *save_file)
     if (filterstring) {
       g_free(filterstring);
     }
-    /* Keep a copy for later evaluation by _cwait() */
-    child_process = fork_child;
 #else
     if (pipe(sync_pipe) < 0) {
       /* Couldn't create the pipe between parent and child. */
@@ -455,17 +493,17 @@ do_capture(const char *save_file)
       unlink(cfile.save_file);
       g_free(cfile.save_file);
       cfile.save_file = NULL;
-      simple_dialog(ESD_TYPE_CRIT, NULL, "Couldn't create sync pipe: %s",
+      simple_dialog(ESD_TYPE_ERROR, ESD_BTN_OK, "Couldn't create sync pipe: %s",
                        strerror(error));
-      return;
+      return FALSE;
     }
 
-    argv = add_arg(argv, &argc, "-m");
-    argv = add_arg(argv, &argc, prefs.gui_font_name);
+    argv = sync_pipe_add_arg(argv, &argc, "-m");
+    argv = sync_pipe_add_arg(argv, &argc, prefs.PREFS_GUI_FONT_NAME);
 
     if (cfile.cfilter != NULL && strlen(cfile.cfilter) != 0) {
-      argv = add_arg(argv, &argc, "-f");
-      argv = add_arg(argv, &argc, cfile.cfilter);
+      argv = sync_pipe_add_arg(argv, &argc, "-f");
+      argv = sync_pipe_add_arg(argv, &argc, cfile.cfilter);
     }
 
     if ((fork_child = fork()) == 0) {
@@ -488,7 +526,7 @@ do_capture(const char *save_file)
       execvp(ethereal_path, argv);
       snprintf(errmsg, sizeof errmsg, "Couldn't run %s in child process: %s",
                ethereal_path, strerror(errno));
-      send_errmsg_to_parent(errmsg);
+      sync_pipe_errmsg_to_parent(errmsg);
 
       /* Exit with "_exit()", so that we don't close the connection
          to the X server (and cause stuff buffered up by our parent but
@@ -519,9 +557,9 @@ do_capture(const char *save_file)
       unlink(cfile.save_file);
       g_free(cfile.save_file);
       cfile.save_file = NULL;
-      simple_dialog(ESD_TYPE_CRIT, NULL, "Couldn't create child process: %s",
-                       strerror(error));
-      return;
+      simple_dialog(ESD_TYPE_ERROR, ESD_BTN_OK,
+                       "Couldn't create child process: %s", strerror(error));
+      return FALSE;
     }
 
     /* Read a byte count from "sync_pipe[READ]", terminated with a
@@ -540,8 +578,8 @@ do_capture(const char *save_file)
        unlink(cfile.save_file);
        g_free(cfile.save_file);
        cfile.save_file = NULL;
-       wait_for_child(TRUE);
-       return;
+       sync_pipe_wait_for_child(TRUE);
+       return FALSE;
       }
       if (c == SP_CAPSTART || c == SP_ERROR_MSG)
        break;
@@ -553,71 +591,37 @@ do_capture(const char *save_file)
        unlink(cfile.save_file);
        g_free(cfile.save_file);
        cfile.save_file = NULL;
-       simple_dialog(ESD_TYPE_WARN, NULL,
+       simple_dialog(ESD_TYPE_ERROR, ESD_BTN_OK,
                        "Capture child process sent us a bad message");
-       return;
+       return FALSE;
       }
       byte_count = byte_count*10 + c - '0';
     }
-    if (c == SP_CAPSTART) {
-      /* Success.  Open the capture file, and set up to read it. */
-      err = start_tail_cap_file(cfile.save_file, is_tempfile, &cfile);
-      if (err == 0) {
-       /* We were able to open and set up to read the capture file;
-          arrange that our callback be called whenever it's possible
-          to read from the sync pipe, so that it's called when
-          the child process wants to tell us something. */
-#ifdef _WIN32
-       /* Tricky to use pipes in win9x, as no concept of wait.  NT can
-          do this but that doesn't cover all win32 platforms.  GTK can do
-          this but doesn't seem to work over processes.  Attempt to do
-          something similar here, start a timer and check for data on every
-          timeout. */
-       cap_timer_id = gtk_timeout_add(1000, cap_timer_cb, NULL);
-#else
-       cap_input_id = gtk_input_add_full(sync_pipe[READ],
-                                      GDK_INPUT_READ|GDK_INPUT_EXCEPTION,
-                                      cap_file_input_cb,
-                                      NULL,
-                                      (gpointer) &cfile,
-                                      NULL);
-#endif
-      } else {
-       /* We weren't able to open the capture file; user has been
-          alerted. Close the sync pipe. */
-
-       close(sync_pipe[READ]);
-
-       /* Don't unlink the save file - leave it around, for debugging
-          purposes. */
-       g_free(cfile.save_file);
-       cfile.save_file = NULL;
-      }
-    } else {
+    if (c != SP_CAPSTART) {
       /* Failure - the child process sent us a message indicating
         what the problem was. */
       if (byte_count == 0) {
        /* Zero-length message? */
-       simple_dialog(ESD_TYPE_WARN, NULL,
+       simple_dialog(ESD_TYPE_ERROR, ESD_BTN_OK,
                "Capture child process failed, but its error message was empty.");
       } else {
        msg = g_malloc(byte_count + 1);
        if (msg == NULL) {
-         simple_dialog(ESD_TYPE_WARN, NULL,
+         simple_dialog(ESD_TYPE_ERROR, ESD_BTN_OK,
                "Capture child process failed, but its error message was too big.");
        } else {
          i = read(sync_pipe[READ], msg, byte_count);
          msg[byte_count] = '\0';
          if (i < 0) {
-           simple_dialog(ESD_TYPE_WARN, NULL,
+           simple_dialog(ESD_TYPE_ERROR, ESD_BTN_OK,
                  "Capture child process failed: Error %s reading its error message.",
                  strerror(errno));
          } else if (i == 0) {
-           simple_dialog(ESD_TYPE_WARN, NULL,
+           simple_dialog(ESD_TYPE_ERROR, ESD_BTN_OK,
                  "Capture child process failed: EOF reading its error message.");
-           wait_for_child(FALSE);
+           sync_pipe_wait_for_child(FALSE);
          } else
-           simple_dialog(ESD_TYPE_CRIT, NULL, msg);
+           simple_dialog(ESD_TYPE_ERROR, ESD_BTN_OK, msg);
          g_free(msg);
        }
 
@@ -629,151 +633,68 @@ do_capture(const char *save_file)
        g_free(cfile.save_file);
        cfile.save_file = NULL;
       }
+      return FALSE;
     }
-  } else {
-    /* Not sync mode. */
-    capture_succeeded = capture(&stats_known, &stats);
-    if (quit_after_cap) {
-      /* DON'T unlink the save file.  Presumably someone wants it. */
-      gtk_exit(0);
-    }
-    if (capture_succeeded) {
-      /* Capture succeeded; read in the capture file. */
-      if ((err = open_cap_file(cfile.save_file, is_tempfile, &cfile)) == 0) {
-        /* Set the read filter to NULL. */
-        cfile.rfcode = 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 "read_cap_file()" get those statistics from
-          Wiretap.  We clear the statistics (marking them as "not known")
-          in "open_cap_file()", and "read_cap_file()" will only fetch
-          them and mark them as known if Wiretap supplies them, so if
-          we get the statistics now, after calling "open_cap_file()" but
-          before calling "read_cap_file()", the values we store will
-          be used by "read_cap_file()".
-
-           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 - "read_cap_file()" will get
-          them from the file and use them. */
-        if (stats_known) {
-          cfile.drops_known = 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. */
-          cfile.drops = stats.ps_drop;
-        }
-        switch (read_cap_file(&cfile, &err)) {
-
-        case READ_SUCCESS:
-        case 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 READ_ABORTED:
-          /* Exit by leaving the main loop, so that any quit functions
-             we registered get called. */
-          if (gtk_main_level() > 0)
-            gtk_main_quit();
-          return;
-        }
-      }
-    }
-    /* We're not doing a capture any more, so we don't have a save
-       file. */
-    if (capture_opts.ringbuffer_on) {
-      ringbuf_free();
-    } else {
-      g_free(cfile.save_file);
-    }
-    cfile.save_file = NULL;
-  }
-}
-
-#ifdef _WIN32
-/* The timer has expired, see if there's stuff to read from the pipe,
-   if so call the cap_file_input_cb */
-static gint
-cap_timer_cb(gpointer data)
-{
-  HANDLE handle;
-  DWORD avail = 0;
-  gboolean result, result1;
-  DWORD childstatus;
 
-  /* Oddly enough although Named pipes don't work on win9x,
-     PeekNamedPipe does !!! */
-  handle = (HANDLE) _get_osfhandle (sync_pipe[READ]);
-  result = PeekNamedPipe(handle, NULL, 0, NULL, &avail, NULL);
+    /* The child process started a capture.
+       Attempt to open the capture file and set up to read it. */
+    err = cf_start_tail(cfile.save_file, is_tempfile, &cfile);
+    if (err != 0) {
+      /* We weren't able to open the capture file; user has been
+        alerted. Close the sync pipe. */
 
-  /* Get the child process exit status */
-  result1 = GetExitCodeProcess((HANDLE)child_process, &childstatus);
-
-  /* If the Peek returned an error, or there are bytes to be read
-     or the childwatcher thread has terminated then call the normal
-     callback */
-  if (!result || avail > 0 || childstatus != STILL_ACTIVE) {
-
-    /* avoid reentrancy problems and stack overflow */
-    gtk_timeout_remove(cap_timer_id);
+      close(sync_pipe[READ]);
 
-    /* And call the real handler */
-    cap_file_input_cb((gpointer) &cfile, 0, 0);
+      /* Don't unlink the save file - leave it around, for debugging
+        purposes. */
+      g_free(cfile.save_file);
+      cfile.save_file = NULL;
+      return FALSE;
+    }
+    /* We were able to open and set up to read the capture file;
+       arrange that our callback be called whenever it's possible
+       to read from the sync pipe, so that it's called when
+       the child process wants to tell us something. */
+    pipe_input_set_handler(sync_pipe[READ], (gpointer) &cfile, &fork_child, sync_pipe_input_cb);
 
-    /* Return false so that the timer is not run again */
-    return FALSE;
-  }
-  else {
-    /* No data so let timer run again */
     return TRUE;
-  }
 }
-#endif
+
 
 /* There's stuff to read from the sync pipe, meaning the child has sent
    us a message, or the sync pipe has closed, meaning the child has
    closed it (perhaps because it exited). */
-static void
-cap_file_input_cb(gpointer data, gint source _U_,
-  GdkInputCondition condition _U_)
+static gboolean 
+sync_pipe_input_cb(gint source, gpointer user_data)
 {
-  capture_file *cf = (capture_file *)data;
+  capture_file *cf = (capture_file *)user_data;
 #define BUFSIZE        4096
   char buffer[BUFSIZE+1], *p = buffer, *q = buffer, *msg, *r;
   int  nread, msglen, chars_to_copy;
   int  to_read = 0;
   int  err;
 
-#ifndef _WIN32
-  /* avoid reentrancy problems and stack overflow */
-  gtk_input_remove(cap_input_id);
-#endif
 
-  if ((nread = read(sync_pipe[READ], buffer, BUFSIZE)) <= 0) {
+  if ((nread = read(source, buffer, BUFSIZE)) <= 0) {
     /* The child has closed the sync pipe, meaning it's not going to be
        capturing any more packets.  Pick up its exit status, and
        complain if it did anything other than exit with status 0. */
-    wait_for_child(FALSE);
+    sync_pipe_wait_for_child(FALSE);
 
     /* Read what remains of the capture file, and finish the capture.
        XXX - do something if this fails? */
-    switch (finish_tail_cap_file(cf, &err)) {
+    switch (cf_finish_tail(cf, &err)) {
 
     case READ_SUCCESS:
+        if(cf->count == 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) ? "temporary " : "");
+          cf_close(cf);
+        }
+        break;
     case 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
@@ -783,8 +704,8 @@ cap_file_input_cb(gpointer data, gint source _U_,
     case READ_ABORTED:
       /* Exit by leaving the main loop, so that any quit functions
          we registered get called. */
-      gtk_main_quit();
-      return;
+      main_window_quit();
+      return FALSE;
     }
 
     /* We're not doing a capture any more, so we don't have a save
@@ -792,7 +713,7 @@ cap_file_input_cb(gpointer data, gint source _U_,
     g_free(cf->save_file);
     cf->save_file = NULL;
 
-    return;
+    return FALSE;
   }
 
   buffer[nread] = '\0';
@@ -827,7 +748,7 @@ cap_file_input_cb(gpointer data, gint source _U_,
       while (msglen != 0) {
        if (nread == 0) {
          /* Read more. */
-          if ((nread = read(sync_pipe[READ], buffer, BUFSIZE)) <= 0)
+          if ((nread = read(source, buffer, BUFSIZE)) <= 0)
             break;
           p = buffer;
           q = buffer;
@@ -840,7 +761,7 @@ cap_file_input_cb(gpointer data, gint source _U_,
         msglen -= chars_to_copy;
       }
       *r = '\0';
-      simple_dialog(ESD_TYPE_CRIT, NULL, msg);
+      simple_dialog(ESD_TYPE_ERROR, ESD_BTN_OK, msg);
       g_free(msg);
       break;
     default :
@@ -853,7 +774,7 @@ cap_file_input_cb(gpointer data, gint source _U_,
   /* Read from the capture file the number of records the child told us
      it added.
      XXX - do something if this fails? */
-  switch (continue_tail_cap_file(cf, to_read, &err)) {
+  switch (cf_continue_tail(cf, to_read, &err)) {
 
   case READ_SUCCESS:
   case READ_ERROR:
@@ -867,37 +788,25 @@ cap_file_input_cb(gpointer data, gint source _U_,
   case READ_ABORTED:
     /* Kill the child capture process; the user wants to exit, and we
        shouldn't just leave it running. */
-#ifdef _WIN32
-    /* XXX - kill it. */
-#else
-    kill(fork_child, SIGTERM); /* SIGTERM so it can clean up if necessary */
-#endif
+    kill_capture_child();
     break;
   }
 
-  /* restore pipe handler */
-#ifdef _WIN32
-  cap_timer_id = gtk_timeout_add(1000, cap_timer_cb, NULL);
-#else
-  cap_input_id = gtk_input_add_full (sync_pipe[READ],
-                                    GDK_INPUT_READ|GDK_INPUT_EXCEPTION,
-                                    cap_file_input_cb,
-                                    NULL,
-                                    (gpointer) cf,
-                                    NULL);
-#endif
+  return TRUE;
 }
 
 static void
-wait_for_child(gboolean always_report)
+sync_pipe_wait_for_child(gboolean always_report)
 {
   int  wstatus;
 
 #ifdef _WIN32
-  /* XXX - analyze the wait stuatus and display more information
-     in the dialog box? */
-  if (_cwait(&wstatus, child_process, _WAIT_CHILD) == -1) {
-    simple_dialog(ESD_TYPE_WARN, NULL, "Child capture process stopped unexpectedly");
+  /* XXX - analyze the wait status and display more information
+     in the dialog box?
+     XXX - set "fork_child" to -1 if we find it exited? */
+  if (_cwait(&wstatus, fork_child, _WAIT_CHILD) == -1) {
+    simple_dialog(ESD_TYPE_ERROR, ESD_BTN_OK,
+               "Child capture process stopped unexpectedly");
   }
 #else
   if (wait(&wstatus) != -1) {
@@ -905,25 +814,25 @@ wait_for_child(gboolean always_report)
       /* The child exited; display its exit status, if it's not zero,
          and even if it's zero if "always_report" is true. */
       if (always_report || WEXITSTATUS(wstatus) != 0) {
-        simple_dialog(ESD_TYPE_WARN, NULL,
+        simple_dialog(ESD_TYPE_ERROR, ESD_BTN_OK,
                      "Child capture process exited: exit status %d",
                      WEXITSTATUS(wstatus));
       }
     } else if (WIFSTOPPED(wstatus)) {
       /* It stopped, rather than exiting.  "Should not happen." */
-      simple_dialog(ESD_TYPE_WARN, NULL,
+      simple_dialog(ESD_TYPE_ERROR, ESD_BTN_OK,
                    "Child capture process stopped: %s",
-                   signame(WSTOPSIG(wstatus)));
+                   sync_pipe_signame(WSTOPSIG(wstatus)));
     } else if (WIFSIGNALED(wstatus)) {
       /* It died with a signal. */
-      simple_dialog(ESD_TYPE_WARN, NULL,
+      simple_dialog(ESD_TYPE_ERROR, ESD_BTN_OK,
                    "Child capture process died: %s%s",
-                   signame(WTERMSIG(wstatus)),
+                   sync_pipe_signame(WTERMSIG(wstatus)),
                    WCOREDUMP(wstatus) ? " - core dumped" : "");
     } else {
       /* What?  It had to either have exited, or stopped, or died with
          a signal; what happened here? */
-      simple_dialog(ESD_TYPE_WARN, NULL,
+      simple_dialog(ESD_TYPE_ERROR, ESD_BTN_OK,
                    "Child capture process died: wait status %#o", wstatus);
     }
   }
@@ -933,9 +842,44 @@ wait_for_child(gboolean always_report)
 #endif
 }
 
+static void
+sync_pipe_errmsg_to_parent(const char *errmsg)
+{
+    int msglen = strlen(errmsg);
+    char lenbuf[SP_DECISIZE+1+1];
+
+    sprintf(lenbuf, "%u%c", msglen, SP_ERROR_MSG);
+    write(1, lenbuf, strlen(lenbuf));
+    write(1, errmsg, msglen);
+}
+
+static void
+sync_pipe_drops_to_parent(int drops)
+{
+       char tmp[SP_DECISIZE+1+1];
+       sprintf(tmp, "%d%c", drops, SP_DROPS);
+       write(1, tmp, strlen(tmp));
+}
+
+static void
+sync_pipe_packet_count_to_parent(int packet_count)
+{
+    char tmp[SP_DECISIZE+1+1];
+    sprintf(tmp, "%d%c", packet_count, SP_PACKET_COUNT);
+    write(1, tmp, strlen(tmp));
+}
+
+static void
+sync_pipe_capstart_to_parent(void)
+{
+    static const char capstart_msg = SP_CAPSTART;
+
+    write(1, &capstart_msg, 1);
+}
+
 #ifndef _WIN32
 static char *
-signame(int sig)
+sync_pipe_signame(int sig)
 {
   char *sigmsg;
   static char sigmsg_buf[6+1+3+1];
@@ -1027,6 +971,120 @@ signame(int sig)
 }
 #endif
 
+
+
+
+
+
+static gboolean
+normal_do_capture(gboolean is_tempfile)
+{
+    int capture_succeeded;
+    gboolean stats_known;
+    struct pcap_stat stats;
+    int err;
+
+    /* Not sync mode. */
+    capture_succeeded = capture(&stats_known, &stats);
+    if (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(cfile.save_file);
+      }
+      cfile.save_file = NULL;
+      return FALSE;
+    }
+    /* Capture succeeded; attempt to read in the capture file. */
+    if ((err = cf_open(cfile.save_file, is_tempfile, &cfile)) != 0) {
+      /* 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(cfile.save_file);
+      }
+      cfile.save_file = NULL;
+      return FALSE;
+    }
+
+    /* Set the read filter to NULL. */
+    cfile.rfcode = 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) {
+      cfile.drops_known = 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. */
+      cfile.drops = stats.ps_drop;
+    }
+    switch (cf_read(&cfile)) {
+
+    case READ_SUCCESS:
+    case 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 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(cfile.save_file);
+    }
+    cfile.save_file = NULL;
+
+    /* if we didn't captured even a single packet, close the file again */
+    if(cfile.count == 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(),
+      (cfile.is_tempfile) ? "temporary " : "");
+      cf_close(&cfile);
+    }
+  return TRUE;
+}
+
 /*
  * Timeout, in milliseconds, for reads from the stream of captured packets.
  */
@@ -1036,7 +1094,7 @@ signame(int sig)
 /* Take care of byte order in the libpcap headers read from pipes.
  * (function taken from wiretap/libpcap.c) */
 static void
-adjust_header(loop_data *ld, struct pcap_hdr *hdr, struct pcaprec_hdr *rechdr)
+cap_pipe_adjust_header(loop_data *ld, struct pcap_hdr *hdr, struct pcaprec_hdr *rechdr)
 {
   if (ld->byte_swapped) {
     /* Byte-swap the record header fields. */
@@ -1070,7 +1128,7 @@ adjust_header(loop_data *ld, struct pcap_hdr *hdr, struct pcaprec_hdr *rechdr)
  * N.B. : we can't read the libpcap formats used in RedHat 6.1 or SuSE 6.3
  * because we can't seek on pipes (see wiretap/libpcap.c for details) */
 static int
-pipe_open_live(char *pipename, struct pcap_hdr *hdr, loop_data *ld,
+cap_pipe_open_live(char *pipename, struct pcap_hdr *hdr, loop_data *ld,
                  char *errmsg, int errmsgl)
 {
   struct stat pipe_stat;
@@ -1089,12 +1147,12 @@ pipe_open_live(char *pipename, struct pcap_hdr *hdr, loop_data *ld,
   else {
     if (stat(pipename, &pipe_stat) < 0) {
       if (errno == ENOENT || errno == ENOTDIR)
-        ld->pipe_err = PIPNEXIST;
+        ld->cap_pipe_err = PIPNEXIST;
       else {
         snprintf(errmsg, errmsgl,
           "The capture session could not be initiated "
           "due to error on pipe: %s", strerror(errno));
-        ld->pipe_err = PIPERR;
+        ld->cap_pipe_err = PIPERR;
       }
       return -1;
     }
@@ -1104,12 +1162,12 @@ pipe_open_live(char *pipename, struct pcap_hdr *hdr, loop_data *ld,
          * Assume the user specified an interface on a system where
          * interfaces are in /dev.  Pretend we haven't seen it.
          */
-         ld->pipe_err = PIPNEXIST;
+         ld->cap_pipe_err = PIPNEXIST;
       } else {
         snprintf(errmsg, errmsgl,
             "The capture session could not be initiated because\n"
             "\"%s\" is neither an interface nor a pipe", pipename);
-        ld->pipe_err = PIPERR;
+        ld->cap_pipe_err = PIPERR;
       }
       return -1;
     }
@@ -1118,12 +1176,12 @@ pipe_open_live(char *pipename, struct pcap_hdr *hdr, loop_data *ld,
       snprintf(errmsg, errmsgl,
           "The capture session could not be initiated "
           "due to error on pipe open: %s", strerror(errno));
-      ld->pipe_err = PIPERR;
+      ld->cap_pipe_err = PIPERR;
       return -1;
     }
   }
 
-  ld->from_pipe = TRUE;
+  ld->from_cap_pipe = TRUE;
 
   /* read the pcap header */
   FD_ZERO(&rfds);
@@ -1223,12 +1281,12 @@ pipe_open_live(char *pipename, struct pcap_hdr *hdr, loop_data *ld,
     goto error;
   }
 
-  ld->pipe_state = STATE_EXPECT_REC_HDR;
-  ld->pipe_err = PIPOK;
+  ld->cap_pipe_state = STATE_EXPECT_REC_HDR;
+  ld->cap_pipe_err = PIPOK;
   return fd;
 
 error:
-  ld->pipe_err = PIPERR;
+  ld->cap_pipe_err = PIPERR;
   close(fd);
   return -1;
 
@@ -1238,7 +1296,7 @@ error:
  * header, write the record in the capture file, and update capture statistics. */
 
 static int
-pipe_dispatch(int fd, loop_data *ld, struct pcap_hdr *hdr,
+cap_pipe_dispatch(int fd, loop_data *ld, struct pcap_hdr *hdr,
                struct pcaprec_modified_hdr *rechdr, guchar *data,
                char *errmsg, int errmsgl)
 {
@@ -1247,13 +1305,13 @@ pipe_dispatch(int fd, loop_data *ld, struct pcap_hdr *hdr,
   enum { PD_REC_HDR_READ, PD_DATA_READ, PD_PIPE_EOF, PD_PIPE_ERR,
           PD_ERR } result;
 
-  switch (ld->pipe_state) {
+  switch (ld->cap_pipe_state) {
 
   case STATE_EXPECT_REC_HDR:
     ld->bytes_to_read = ld->modified ?
       sizeof(struct pcaprec_modified_hdr) : sizeof(struct pcaprec_hdr);
     ld->bytes_read = 0;
-    ld->pipe_state = STATE_READ_REC_HDR;
+    ld->cap_pipe_state = STATE_READ_REC_HDR;
     /* Fall through */
 
   case STATE_READ_REC_HDR:
@@ -1273,7 +1331,7 @@ pipe_dispatch(int fd, loop_data *ld, struct pcap_hdr *hdr,
 
   case STATE_EXPECT_DATA:
     ld->bytes_read = 0;
-    ld->pipe_state = STATE_READ_DATA;
+    ld->cap_pipe_state = STATE_READ_DATA;
     /* Fall through */
 
   case STATE_READ_DATA:
@@ -1291,10 +1349,10 @@ pipe_dispatch(int fd, loop_data *ld, struct pcap_hdr *hdr,
     break;
 
   default:
-    snprintf(errmsg, errmsgl, "pipe_dispatch: invalid state");
+    snprintf(errmsg, errmsgl, "cap_pipe_dispatch: invalid state");
     result = PD_ERR;
 
-  } /* switch (ld->pipe_state) */
+  } /* switch (ld->cap_pipe_state) */
 
   /*
    * We've now read as much data as we were expecting, so process it.
@@ -1303,13 +1361,13 @@ pipe_dispatch(int fd, loop_data *ld, struct pcap_hdr *hdr,
 
   case PD_REC_HDR_READ:
     /* We've read the header. Take care of byte order. */
-    adjust_header(ld, hdr, &rechdr->hdr);
+    cap_pipe_adjust_header(ld, hdr, &rechdr->hdr);
     if (rechdr->hdr.incl_len > WTAP_MAX_PACKET_SIZE) {
       snprintf(errmsg, errmsgl, "Frame %u too long (%d bytes)",
         ld->counts.total+1, rechdr->hdr.incl_len);
       break;
     }
-    ld->pipe_state = STATE_EXPECT_DATA;
+    ld->cap_pipe_state = STATE_EXPECT_DATA;
     return 0;
 
   case PD_DATA_READ:
@@ -1321,11 +1379,11 @@ pipe_dispatch(int fd, loop_data *ld, struct pcap_hdr *hdr,
 
     capture_pcap_cb((guchar *)ld, &phdr, data);
 
-    ld->pipe_state = STATE_EXPECT_REC_HDR;
+    ld->cap_pipe_state = STATE_EXPECT_REC_HDR;
     return 1;
 
   case PD_PIPE_EOF:
-    ld->pipe_err = PIPEOF;
+    ld->cap_pipe_err = PIPEOF;
     return -1;
 
   case PD_PIPE_ERR:
@@ -1336,11 +1394,11 @@ pipe_dispatch(int fd, loop_data *ld, struct pcap_hdr *hdr,
     break;
   }
 
-  ld->pipe_err = PIPERR;
+  ld->cap_pipe_err = PIPERR;
   /* Return here rather than inside the switch to prevent GCC warning */
   return -1;
 }
-#endif
+#endif /* not _WIN32 */
 
 /*
  * This needs to be static, so that the SIGUSR1 handler can clear the "go"
@@ -1353,54 +1411,33 @@ static loop_data   ld;
 int
 capture(gboolean *stats_known, struct pcap_stat *stats)
 {
-  GtkWidget  *cap_w, *main_vb, *stop_bt, *counts_tb;
-  GtkWidget  *counts_fr, *running_tb, *running_label, *running_time;
   pcap_t     *pch;
   int         pcap_encap;
   int         file_snaplen;
   gchar       open_err_str[PCAP_ERRBUF_SIZE];
   gchar       lookup_net_err_str[PCAP_ERRBUF_SIZE];
-  gchar       label_str[64];
   bpf_u_int32 netnum, netmask;
   struct bpf_program fcode;
+  const char *set_linktype_err_str;
   time_t      upd_time, cur_time;
   time_t      start_time;
   int         err, inpkts;
-  condition  *cnd_stop_capturesize = NULL;
-  condition  *cnd_stop_timeout = NULL;
-  condition  *cnd_ring_timeout = NULL;
-  unsigned int i;
-  static const char capstart_msg = SP_CAPSTART;
+  condition  *cnd_file_duration = NULL;
+  condition  *cnd_autostop_files = NULL;
+  condition  *cnd_autostop_size = NULL;
+  condition  *cnd_autostop_duration = NULL;
+  guint32     autostop_files = 0;
   char        errmsg[4096+1];
   gboolean    write_ok;
   gboolean    close_ok;
-  fd_set      set1;
-  struct timeval timeout;
-  struct {
-      const gchar *title;
-      gint *value_ptr;
-      GtkWidget *label, *value, *percent;
-  } counts[] = {
-      { "Total", &ld.counts.total, NULL, NULL, NULL },
-      { "SCTP", &ld.counts.sctp, NULL, NULL, NULL },
-      { "TCP", &ld.counts.tcp, NULL, NULL, NULL },
-      { "UDP", &ld.counts.udp, NULL, NULL, NULL },
-      { "ICMP", &ld.counts.icmp, NULL, NULL, NULL },
-      { "ARP", &ld.counts.arp, NULL, NULL, NULL },
-      { "OSPF", &ld.counts.ospf, NULL, NULL, NULL },
-      { "GRE", &ld.counts.gre, NULL, NULL, NULL },
-      { "NetBIOS", &ld.counts.netbios, NULL, NULL, NULL },
-      { "IPX", &ld.counts.ipx, NULL, NULL, NULL },
-      { "VINES", &ld.counts.vines, NULL, NULL, NULL },
-      { "Other", &ld.counts.other, NULL, NULL, NULL }
-  };
-
-#define N_COUNTS (sizeof counts / sizeof counts[0])
+  capture_info   capture_ui;
 
 #ifdef _WIN32
-  WORD wVersionRequested;
-  WSADATA wsaData;
+  WORD        wVersionRequested;
+  WSADATA     wsaData;
 #else
+  fd_set      set1;
+  struct timeval timeout;
   static const char ppamsg[] = "can't find PPA for ";
   char       *libpcap_warn;
   int         sel_ret;
@@ -1412,34 +1449,64 @@ capture(gboolean *stats_known, struct pcap_stat *stats)
 #ifdef MUST_DO_SELECT
   int         pcap_fd = 0;
 #endif
-
-/* Size of buffer to hold decimal representation of
-   signed/unsigned 64-bit int */
-#define DECISIZE 20
+  gboolean    show_info = capture_opts.show_info || !capture_opts.sync_mode;
 
   /* Initialize Windows Socket if we are in a WIN32 OS
      This needs to be done before querying the interface for network/netmask */
 #ifdef _WIN32
-  wVersionRequested = MAKEWORD( 1, 1 );
-  err = WSAStartup( wVersionRequested, &wsaData );
-  if (err!=0) {
-    snprintf(errmsg, sizeof errmsg,
-      "Couldn't initialize Windows Sockets.");
-       pch=NULL;
+  /* XXX - do we really require 1.1 or earlier?
+     Are there any versions that support only 2.0 or higher? */
+  wVersionRequested = MAKEWORD(1, 1);
+  err = WSAStartup(wVersionRequested, &wsaData);
+  if (err != 0) {
+    switch (err) {
+
+    case WSASYSNOTREADY:
+      snprintf(errmsg, sizeof errmsg,
+        "Couldn't initialize Windows Sockets: Network system not ready for network communication");
+      break;
+
+    case WSAVERNOTSUPPORTED:
+      snprintf(errmsg, sizeof errmsg,
+        "Couldn't initialize Windows Sockets: Windows Sockets version %u.%u not supported",
+        LOBYTE(wVersionRequested), HIBYTE(wVersionRequested));
+      break;
+
+    case WSAEINPROGRESS:
+      snprintf(errmsg, sizeof errmsg,
+        "Couldn't initialize Windows Sockets: Blocking operation is in progress");
+      break;
+
+    case WSAEPROCLIM:
+      snprintf(errmsg, sizeof errmsg,
+        "Couldn't initialize Windows Sockets: Limit on the number of tasks supported by this WinSock implementation has been reached");
+      break;
+
+    case WSAEFAULT:
+      snprintf(errmsg, sizeof errmsg,
+        "Couldn't initialize Windows Sockets: Bad pointer passed to WSAStartup");
+      break;
+
+    default:
+      snprintf(errmsg, sizeof errmsg,
+        "Couldn't initialize Windows Sockets: error %d", err);
+      break;
+    }
+    pch = NULL;
     goto error;
   }
 #endif
 
   ld.go             = TRUE;
   ld.counts.total   = 0;
-  if (capture_opts.has_autostop_count)
-    ld.max          = capture_opts.autostop_count;
+  if (capture_opts.has_autostop_packets)
+    ld.max          = capture_opts.autostop_packets;
   else
     ld.max          = 0;       /* no limit */
   ld.err            = 0;       /* no error seen yet */
   ld.linktype       = WTAP_ENCAP_UNKNOWN;
   ld.pcap_err       = FALSE;
-  ld.from_pipe      = FALSE;
+  ld.from_cap_pipe  = FALSE;
   ld.sync_packets   = 0;
   ld.counts.sctp    = 0;
   ld.counts.tcp     = 0;
@@ -1468,7 +1535,32 @@ capture(gboolean *stats_known, struct pcap_stat *stats)
                       capture_opts.promisc_mode, CAP_READ_TIMEOUT,
                       open_err_str);
 
-  if (pch == NULL) {
+  if (pch != NULL) {
+#ifdef _WIN32
+    /* try to set the capture buffer size */
+    if (pcap_setbuff(pch, capture_opts.buffer_size * 1024 * 1024) != 0) {
+        simple_dialog(ESD_TYPE_INFO, ESD_BTN_OK,
+          "%sCouldn't set the capture buffer size!%s\n"
+          "\n"
+          "The capture buffer size of %luMB seems to be too high for your machine,\n"
+          "the default of 1MB will be used.\n"
+          "\n"
+          "Nonetheless, the capture is started.\n",
+          simple_dialog_primary_start(), simple_dialog_primary_end(), capture_opts.buffer_size);
+    }
+#endif
+
+    /* setting the data link type only works on real interfaces */
+    if (capture_opts.linktype != -1) {
+      set_linktype_err_str = set_pcap_linktype(pch, cfile.iface,
+       capture_opts.linktype);
+      if (set_linktype_err_str != NULL) {
+       snprintf(errmsg, sizeof errmsg, "Unable to set data link type (%s).",
+         set_linktype_err_str);
+       goto error;
+      }
+    }
+  } else {
     /* We couldn't open "cfile.iface" as a network device. */
 #ifdef _WIN32
     /* On Windows, we don't support capturing on pipes, so we give up.
@@ -1477,7 +1569,7 @@ capture(gboolean *stats_known, struct pcap_stat *stats)
        capture-progress window, and, since we couldn't start the
        capture, we haven't popped it up. */
     if (!capture_child) {
-      while (gtk_events_pending()) gtk_main_iteration();
+      main_window_update();
     }
 
     /* On Win32 OSes, the capture devices are probably available to all
@@ -1488,13 +1580,15 @@ capture(gboolean *stats_known, struct pcap_stat *stats)
        "The capture session could not be initiated (%s).\n"
        "Please check that you have the proper interface specified.\n"
        "\n"
-       "Note that the driver Ethereal uses for packet capture on Windows doesn't\n"
-       "support capturing on PPP/WAN interfaces in Windows NT/2000/XP/.NET Server.\n",
+       "Note that the WinPcap 2.x version of the driver Ethereal uses for packet\n"
+       "capture on Windows doesn't support capturing on PPP/WAN interfaces in\n"
+       "Windows NT/2000/XP/2003 Server, and that the WinPcap 3.0 and later versions\n"
+       "don't support capturing on PPP/WAN interfaces at all.",
        open_err_str);
     goto error;
 #else
     /* try to open cfile.iface as a pipe */
-    pipe_fd = pipe_open_live(cfile.iface, &hdr, &ld, errmsg, sizeof errmsg);
+    pipe_fd = cap_pipe_open_live(cfile.iface, &hdr, &ld, errmsg, sizeof errmsg);
 
     if (pipe_fd == -1) {
 
@@ -1504,10 +1598,10 @@ capture(gboolean *stats_known, struct pcap_stat *stats)
        * capture, we haven't popped it up.
        */
       if (!capture_child) {
-       while (gtk_events_pending()) gtk_main_iteration();
+          main_window_update();
       }
 
-      if (ld.pipe_err == PIPNEXIST) {
+      if (ld.cap_pipe_err == PIPNEXIST) {
        /* Pipe doesn't exist, so output message for interface */
 
        /* If we got a "can't find PPA for XXX" message, warn the user (who
@@ -1536,19 +1630,19 @@ capture(gboolean *stats_known, struct pcap_stat *stats)
          libpcap_warn);
       }
       /*
-       * Else pipe (or file) does exist and pipe_open_live() has
+       * Else pipe (or file) does exist and cap_pipe_open_live() has
        * filled in errmsg
        */
       goto error;
     } else
-      /* pipe_open_live() succeeded; don't want
+      /* cap_pipe_open_live() succeeded; don't want
          error message from pcap_open_live() */
       open_err_str[0] = '\0';
 #endif
   }
 
   /* capture filters only work on real interfaces */
-  if (cfile.cfilter && !ld.from_pipe) {
+  if (cfile.cfilter && !ld.from_cap_pipe) {
     /* A capture filter was specified; set it up. */
     if (pcap_lookupnet(cfile.iface, &netnum, &netmask, lookup_net_err_str) < 0) {
       /*
@@ -1563,8 +1657,31 @@ capture(gboolean *stats_known, struct pcap_stat *stats)
       netmask = 0;
     }
     if (pcap_compile(pch, &fcode, cfile.cfilter, 1, netmask) < 0) {
-      snprintf(errmsg, sizeof errmsg, "Unable to parse filter string (%s).",
-       pcap_geterr(pch));
+      dfilter_t   *rfcode = NULL;
+      /* filter string invalid, did the user tried a display filter? */
+      if (dfilter_compile(cfile.cfilter, &rfcode) && rfcode != NULL) {
+        snprintf(errmsg, sizeof errmsg,
+          "%sInvalid capture filter: \"%s\"!%s\n"
+          "\n"
+          "That string looks like a valid display filter; however, it is not a valid\n"
+          "capture filter (%s).\n"
+          "\n"
+          "Note that display filters and capture filters don't have the same syntax,\n"
+          "so you can't use most display filter expressions as capture filters.\n"
+          "\n"
+          "See the help for a description of the capture filter syntax.",
+          simple_dialog_primary_start(), cfile.cfilter, simple_dialog_primary_end(),
+          pcap_geterr(pch));
+       dfilter_free(rfcode);
+      } else {
+        snprintf(errmsg, sizeof errmsg,
+          "%sInvalid capture filter: \"%s\"!%s\n"
+          "\n"
+          "That string is not a valid capture filter (%s).\n"
+          "See the help for a description of the capture filter syntax.",
+          simple_dialog_primary_start(), cfile.cfilter, simple_dialog_primary_end(),
+          pcap_geterr(pch));
+      }
       goto error;
     }
     if (pcap_setfilter(pch, &fcode) < 0) {
@@ -1576,7 +1693,7 @@ capture(gboolean *stats_known, struct pcap_stat *stats)
 
   /* Set up to write to the capture file. */
 #ifndef _WIN32
-  if (ld.from_pipe) {
+  if (ld.from_cap_pipe) {
     pcap_encap = hdr.network;
     file_snaplen = hdr.snaplen;
   } else
@@ -1592,7 +1709,7 @@ capture(gboolean *stats_known, struct pcap_stat *stats)
        " that Ethereal doesn't support (data link type %d).", pcap_encap);
     goto error;
   }
-  if (capture_opts.ringbuffer_on) {
+  if (capture_opts.multi_files_on) {
     ld.pdh = ringbuf_init_wtap_dump_fdopen(WTAP_FILE_PCAP, ld.linktype,
       file_snaplen, &err);
   } else {
@@ -1652,92 +1769,20 @@ capture(gboolean *stats_known, struct pcap_stat *stats)
        update its windows to indicate that we have a live capture in
        progress. */
     fflush(wtap_dump_file(ld.pdh));
-    write(1, &capstart_msg, 1);
+    sync_pipe_capstart_to_parent();
   }
 
-  cap_w = gtk_window_new(GTK_WINDOW_TOPLEVEL);
-  gtk_window_set_title(GTK_WINDOW(cap_w), "Ethereal: Capture");
-  gtk_window_set_modal(GTK_WINDOW(cap_w), TRUE);
-
-  /* Container for capture display widgets */
-  main_vb = gtk_vbox_new(FALSE, 1);
-  gtk_container_border_width(GTK_CONTAINER(main_vb), 5);
-  gtk_container_add(GTK_CONTAINER(cap_w), main_vb);
-  gtk_widget_show(main_vb);
-
-  counts_fr = gtk_frame_new("Captured Frames");
-  gtk_box_pack_start(GTK_BOX(main_vb), counts_fr, FALSE, FALSE, 3);
-  gtk_widget_show(counts_fr);
-
-  /* Individual statistic elements */
-  counts_tb = gtk_table_new(N_COUNTS, 3, TRUE);
-  gtk_container_add(GTK_CONTAINER(counts_fr), counts_tb);
-  gtk_container_border_width(GTK_CONTAINER(counts_tb), 5);
-  gtk_widget_show(counts_tb);
-
-  for (i = 0; i < N_COUNTS; i++) {
-      counts[i].label = gtk_label_new(counts[i].title);
-      gtk_misc_set_alignment(GTK_MISC(counts[i].label), 0.0f, 0.0f);
-
-      counts[i].value = gtk_label_new("0");
-      gtk_misc_set_alignment(GTK_MISC(counts[i].value), 0.0f, 0.0f);
-
-      counts[i].percent = gtk_label_new("0.0%");
-      gtk_misc_set_alignment(GTK_MISC(counts[i].percent), 0.0f, 0.0f);
-
-      gtk_table_attach_defaults(GTK_TABLE(counts_tb),
-                                counts[i].label, 0, 1, i, i + 1);
-
-      gtk_table_attach(GTK_TABLE(counts_tb),
-                       counts[i].value,
-                       1, 2, i, i + 1, 0, 0, 5, 0);
-
-      gtk_table_attach_defaults(GTK_TABLE(counts_tb),
-                                counts[i].percent, 2, 3, i, i + 1);
-
-      gtk_widget_show(counts[i].label);
-      gtk_widget_show(counts[i].value);
-      gtk_widget_show(counts[i].percent);
+  /* start capture info dialog */
+  if(show_info) {
+      capture_ui.callback_data  = &ld;
+      capture_ui.counts         = &ld.counts;
+      capture_info_create(&capture_ui, cfile.iface);
   }
 
-  /* Running time */
-  running_tb = gtk_table_new(1, 3, TRUE);
-  gtk_box_pack_start(GTK_BOX(main_vb), running_tb, FALSE, FALSE, 3);
-  gtk_widget_show(running_tb);
-
-  running_label = gtk_label_new("Running");
-  gtk_misc_set_alignment(GTK_MISC(running_label), 0.0f, 0.0f);
-  gtk_widget_show(running_label);
-  gtk_table_attach_defaults(GTK_TABLE(running_tb),
-                                running_label, 0, 1, 0, 1);
-
-  running_time = gtk_label_new("00:00:00");
-  gtk_misc_set_alignment(GTK_MISC(running_time), 0.0f, 0.0f);
-  gtk_widget_show(running_time);
-  gtk_table_attach(GTK_TABLE(running_tb),
-                       running_time,
-                       1, 2, 0, 1, 0, 0, 5, 0);
-
-  /* allow user to either click a stop button, or the close button on
-       the window to stop a capture in progress. */
-  stop_bt = gtk_button_new_with_label ("Stop");
-  gtk_signal_connect(GTK_OBJECT(stop_bt), "clicked",
-    GTK_SIGNAL_FUNC(capture_stop_cb), (gpointer) &ld);
-  gtk_signal_connect(GTK_OBJECT(cap_w), "delete_event",
-       GTK_SIGNAL_FUNC(capture_delete_cb), (gpointer) &ld);
-  gtk_box_pack_start(GTK_BOX(main_vb), stop_bt, FALSE, FALSE, 3);
-  GTK_WIDGET_SET_FLAGS(stop_bt, GTK_CAN_DEFAULT);
-  gtk_widget_grab_default(stop_bt);
-  GTK_WIDGET_SET_FLAGS(stop_bt, GTK_CAN_DEFAULT);
-  gtk_widget_grab_default(stop_bt);
-  gtk_widget_show(stop_bt);
-
-  gtk_widget_show(cap_w);
-
   start_time = time(NULL);
   upd_time = time(NULL);
 #ifdef MUST_DO_SELECT
-  if (!ld.from_pipe) pcap_fd = pcap_fileno(pch);
+  if (!ld.from_cap_pipe) pcap_fd = pcap_fileno(pch);
 #endif
 
 #ifndef _WIN32
@@ -1746,27 +1791,36 @@ capture(gboolean *stats_known, struct pcap_stat *stats)
    * kills us with it due to the user selecting "Capture->Stop".
    */
   if (capture_child)
-    signal(SIGUSR1, stop_capture);
+    signal(SIGUSR1, stop_capture_signal_handler);
 #endif
+
   /* initialize capture stop conditions */
   init_capture_stop_conditions();
   /* create stop conditions */
   if (capture_opts.has_autostop_filesize)
-    cnd_stop_capturesize =
-        cnd_new(CND_CLASS_CAPTURESIZE,(long)capture_opts.autostop_filesize * 1000);
+    cnd_autostop_size =
+        cnd_new(CND_CLASS_CAPTURESIZE,(long)capture_opts.autostop_filesize);
   if (capture_opts.has_autostop_duration)
-    cnd_stop_timeout =
+    cnd_autostop_duration =
         cnd_new(CND_CLASS_TIMEOUT,(gint32)capture_opts.autostop_duration);
 
-  if (capture_opts.ringbuffer_on && capture_opts.has_ring_duration)
-    cnd_ring_timeout =
-       cnd_new(CND_CLASS_TIMEOUT, capture_opts.ringbuffer_duration);
+  if (capture_opts.multi_files_on) {
+      if (capture_opts.has_file_duration)
+        cnd_file_duration =
+           cnd_new(CND_CLASS_TIMEOUT, capture_opts.file_duration);
+
+      if (capture_opts.has_autostop_files)
+        cnd_autostop_files =
+           cnd_new(CND_CLASS_CAPTURESIZE, capture_opts.autostop_files);
+  }
 
+  /* WOW, everything is prepared! */
+  /* please fasten your seat belts, we will enter now the actual capture loop */
   while (ld.go) {
-    while (gtk_events_pending()) gtk_main_iteration();
+    main_window_update();
 
 #ifndef _WIN32
-    if (ld.from_pipe) {
+    if (ld.from_cap_pipe) {
       FD_ZERO(&set1);
       FD_SET(pipe_fd, &set1);
       timeout.tv_sec = 0;
@@ -1784,7 +1838,7 @@ capture(gboolean *stats_known, struct pcap_stat *stats)
        /*
         * "select()" says we can read from the pipe without blocking
         */
-       inpkts = pipe_dispatch(pipe_fd, &ld, &hdr, &rechdr, pcap_data,
+       inpkts = cap_pipe_dispatch(pipe_fd, &ld, &hdr, &rechdr, pcap_data,
           errmsg, sizeof errmsg);
        if (inpkts < 0) {
          ld.go = FALSE;
@@ -1792,7 +1846,7 @@ capture(gboolean *stats_known, struct pcap_stat *stats)
       }
     }
     else
-#endif
+#endif /* _WIN32 */
     {
 #ifdef MUST_DO_SELECT
       /*
@@ -1852,33 +1906,41 @@ capture(gboolean *stats_known, struct pcap_stat *stats)
         ld.pcap_err = TRUE;
         ld.go = FALSE;
       }
-#endif
+#endif /* MUST_DO_SELECT */
     }
 
     if (inpkts > 0) {
       ld.sync_packets += inpkts;
-      /* check capture stop conditons */
-      if (cnd_stop_capturesize != NULL && cnd_eval(cnd_stop_capturesize,
+
+      /* check capture size condition */
+      if (cnd_autostop_size != NULL && cnd_eval(cnd_autostop_size,
                     (guint32)wtap_get_bytes_dumped(ld.pdh))){
-        /* Capture file reached its maximum size. */
-        if (capture_opts.ringbuffer_on) {
+        /* Capture size limit reached, do we have another file? */
+        if (capture_opts.multi_files_on) {
+          if (cnd_autostop_files != NULL && cnd_eval(cnd_autostop_files, ++autostop_files)) {
+            /* no files left: stop here */
+            ld.go = FALSE;
+            continue;
+          }
+
           /* Switch to the next ringbuffer file */
           if (ringbuf_switch_file(&cfile, &ld.pdh, &ld.err)) {
-            /* File switch succeeded: reset the condition */
-            cnd_reset(cnd_stop_capturesize);
-           if (cnd_ring_timeout) {
-             cnd_reset(cnd_ring_timeout);
-           }
+            /* File switch succeeded: reset the conditions */
+            cnd_reset(cnd_autostop_size);
+            if (cnd_file_duration) {
+              cnd_reset(cnd_file_duration);
+            }
           } else {
             /* File switch failed: stop here */
             ld.go = FALSE;
             continue;
           }
         } else {
-          /* no ringbuffer - just stop */
+          /* single file, stop now */
           ld.go = FALSE;
+          continue;
         }
-      }
+      } /* cnd_autostop_size */
     }
 
     /* Only update once a second so as not to overload slow displays */
@@ -1886,67 +1948,80 @@ capture(gboolean *stats_known, struct pcap_stat *stats)
     if (cur_time > upd_time) {
       upd_time = cur_time;
 
+      /*if (pcap_stats(pch, stats) >= 0) {
+        *stats_known = TRUE;
+      }*/
+
+       /* Let the parent process know. */
       /* calculate and display running time */
-      cur_time -= start_time;
-      snprintf(label_str, sizeof(label_str), "%02ld:%02ld:%02ld", 
-               (long)(cur_time/3600), (long)((cur_time%3600)/60),
-               (long)(cur_time%60));
-      gtk_label_set(GTK_LABEL(running_time), label_str);
+      if(show_info) {
+          cur_time -= start_time;
+          capture_ui.running_time   = cur_time;
+          capture_ui.new_packets    = ld.sync_packets;
+          capture_info_update(&capture_ui);
+      }
 
       if (ld.sync_packets) {
-
-        for (i = 0; i < N_COUNTS; i++) {
-            snprintf(label_str, sizeof(label_str), "%d",
-                     *counts[i].value_ptr);
-
-            gtk_label_set(GTK_LABEL(counts[i].value), label_str);
-
-            snprintf(label_str, sizeof(label_str), "(%.1f%%)",
-                     pct(*counts[i].value_ptr, ld.counts.total));
-
-            gtk_label_set(GTK_LABEL(counts[i].percent), label_str);
-        }
-
-        /* do sync here, too */
+        /* do sync here */
         fflush(wtap_dump_file(ld.pdh));
 
         if (capture_child) {
          /* This is the child process for a sync mode capture, so send
             our parent a message saying we've written out "ld.sync_packets"
             packets to the capture file. */
-         char tmp[DECISIZE+1+1];
-         sprintf(tmp, "%d%c", ld.sync_packets, SP_PACKET_COUNT);
-         write(1, tmp, strlen(tmp));
+        sync_pipe_packet_count_to_parent(ld.sync_packets);
         }
 
-       ld.sync_packets = 0;
-
+        ld.sync_packets = 0;
       }
 
-      if (cnd_stop_timeout != NULL && cnd_eval(cnd_stop_timeout)) {
-        /* The specified capture time has elapsed; stop the capture. */
+      /* check capture duration condition */
+      if (cnd_autostop_duration != NULL && cnd_eval(cnd_autostop_duration)) {
+        /* The maximum capture time has elapsed; stop the capture. */
         ld.go = FALSE;
-      } else if (cnd_ring_timeout != NULL && cnd_eval(cnd_ring_timeout)) {
-       /* time elasped for this ring file, swith to the next */
-       if (ringbuf_switch_file(&cfile, &ld.pdh, &ld.err)) {
-         /* File switch succeeded: reset the condition */
-         cnd_reset(cnd_ring_timeout);
-       } else {
-         /* File switch failed: stop here */
-         ld.go = FALSE;
-       }
+        continue;
       }
+      
+      /* check capture file duration condition */
+      if (cnd_file_duration != NULL && cnd_eval(cnd_file_duration)) {
+        /* duration limit reached, do we have another file? */
+        if (capture_opts.multi_files_on) {
+          if (cnd_autostop_files != NULL && cnd_eval(cnd_autostop_files, ++autostop_files)) {
+            /* no files left: stop here */
+            ld.go = FALSE;
+            continue;
+          }
+
+          /* Switch to the next ringbuffer file */
+          if (ringbuf_switch_file(&cfile, &ld.pdh, &ld.err)) {
+            /* file switch succeeded: reset the conditions */
+            cnd_reset(cnd_file_duration);
+            if(cnd_autostop_size)
+              cnd_reset(cnd_autostop_size);
+          } else {
+            /* File switch failed: stop here */
+               ld.go = FALSE;
+            continue;
+          }
+        } else {
+          /* single file, stop now */
+          ld.go = FALSE;
+          continue;
+        }
+      } /* cnd_file_duration */
     }
 
   } /* while (ld.go) */
 
   /* delete stop conditions */
-  if (cnd_stop_capturesize != NULL)
-    cnd_delete(cnd_stop_capturesize);
-  if (cnd_stop_timeout != NULL)
-    cnd_delete(cnd_stop_timeout);
-  if (cnd_ring_timeout != NULL)
-    cnd_delete(cnd_ring_timeout);
+  if (cnd_file_duration != NULL)
+    cnd_delete(cnd_file_duration);
+  if (cnd_autostop_files != NULL)
+    cnd_delete(cnd_autostop_files);
+  if (cnd_autostop_size != NULL)
+    cnd_delete(cnd_autostop_size);
+  if (cnd_autostop_duration != NULL)
+    cnd_delete(cnd_autostop_duration);
 
   if (ld.pcap_err) {
     snprintf(errmsg, sizeof(errmsg), "Error while capturing packets: %s",
@@ -1955,7 +2030,7 @@ capture(gboolean *stats_known, struct pcap_stat *stats)
 #ifdef _WIN32
   }
 #else
-  } else if (ld.from_pipe && ld.pipe_err == PIPERR)
+  } else if (ld.from_cap_pipe && ld.cap_pipe_err == PIPERR)
       popup_errmsg(errmsg);
 #endif
 
@@ -1968,7 +2043,7 @@ capture(gboolean *stats_known, struct pcap_stat *stats)
     write_ok = FALSE;
   }
 
-  if (capture_opts.ringbuffer_on) {
+  if (capture_opts.multi_files_on) {
     close_ok = ringbuf_wtap_dump_close(&cfile, &err);
   } else {
     close_ok = wtap_dump_close(ld.pdh, &err);
@@ -1988,11 +2063,11 @@ capture(gboolean *stats_known, struct pcap_stat *stats)
    * XXX We exhibit different behaviour between normal mode and sync mode
    * when the pipe is stdin and not already at EOF.  If we're a child, the
    * parent's stdin isn't closed, so if the user starts another capture,
-   * pipe_open_live() will very likely not see the expected magic bytes and
+   * cap_pipe_open_live() will very likely not see the expected magic bytes and
    * will say "Unrecognized libpcap format".  On the other hand, in normal
-   * mode, pipe_open_live() will say "End of file on pipe during open".
+   * mode, cap_pipe_open_live() will say "End of file on pipe during open".
    */
-  if (ld.from_pipe && pipe_fd >= 0)
+  if (ld.from_cap_pipe && pipe_fd >= 0)
     close(pipe_fd);
   else
 #endif
@@ -2003,9 +2078,7 @@ capture(gboolean *stats_known, struct pcap_stat *stats)
       *stats_known = TRUE;
       if (capture_child) {
        /* Let the parent process know. */
-       char tmp[DECISIZE+1+1];
-       sprintf(tmp, "%d%c", stats->ps_drop, SP_DROPS);
-       write(1, tmp, strlen(tmp));
+        sync_pipe_drops_to_parent(stats->ps_drop);
       }
     } else {
       snprintf(errmsg, sizeof(errmsg),
@@ -2021,13 +2094,14 @@ capture(gboolean *stats_known, struct pcap_stat *stats)
   WSACleanup();
 #endif
 
-  gtk_grab_remove(GTK_WIDGET(cap_w));
-  gtk_widget_destroy(GTK_WIDGET(cap_w));
+  if(show_info) {
+    capture_info_destroy(&capture_ui);
+  }
 
   return write_ok;
 
 error:
-  if (capture_opts.ringbuffer_on) {
+  if (capture_opts.multi_files_on) {
     /* cleanup ringbuffer */
     ringbuf_error_cleanup();
   } else {
@@ -2044,7 +2118,7 @@ error:
   popup_errmsg(errmsg);
 
 #ifndef _WIN32
-  if (ld.from_pipe) {
+  if (ld.from_cap_pipe) {
     if (pipe_fd >= 0)
       close(pipe_fd);
   } else
@@ -2122,75 +2196,70 @@ popup_errmsg(const char *errmsg)
     /* This is the child process for a sync mode capture.
        Send the error message to our parent, so they can display a
        dialog box containing it. */
-    send_errmsg_to_parent(errmsg);
+    sync_pipe_errmsg_to_parent(errmsg);
   } else {
     /* Display the dialog box ourselves; there's no parent. */
-    simple_dialog(ESD_TYPE_CRIT, NULL, "%s", errmsg);
-  }
-}
-
-static void
-send_errmsg_to_parent(const char *errmsg)
-{
-    int msglen = strlen(errmsg);
-    char lenbuf[DECISIZE+1+1];
-
-    sprintf(lenbuf, "%u%c", msglen, SP_ERROR_MSG);
-    write(1, lenbuf, strlen(lenbuf));
-    write(1, errmsg, msglen);
-}
-
-static float
-pct(gint num, gint denom) {
-  if (denom) {
-    return (float) num * 100.0 / (float) denom;
-  } else {
-    return 0.0;
+    simple_dialog(ESD_TYPE_ERROR, ESD_BTN_OK, "%s", errmsg);
   }
 }
 
 static void
-stop_capture(int signo _U_)
+stop_capture_signal_handler(int signo _U_)
 {
   ld.go = FALSE;
 }
 
-static void
-capture_delete_cb(GtkWidget *w _U_, GdkEvent *event _U_, gpointer data) {
-  capture_stop_cb(NULL, data);
-}
-
-static void
-capture_stop_cb(GtkWidget *w _U_, gpointer data) {
-  loop_data *ld = (loop_data *) data;
-
-  ld->go = FALSE;
-}
 
 void
 capture_stop(void)
 {
-  /*
-   * XXX - find some way of signaling the child in Win32.
-   */
+  if (fork_child != -1) {
 #ifndef _WIN32
-  if (fork_child != -1)
       kill(fork_child, SIGUSR1);
+#else
+      /* XXX: this is not the preferred method of closing a process!
+       * the clean way would be getting the process id of the child process,
+       * then getting window handle hWnd of that process (using EnumChildWindows),
+       * and then do a SendMessage(hWnd, WM_CLOSE, 0, 0) 
+       *
+       * Unfortunately, I don't know how to get the process id from the handle */
+      /* Hint: OpenProcess will get an handle from the id, not vice versa :-(
+       *
+       * Hint: GenerateConsoleCtrlEvent() will only work, if both processes are 
+       * running in the same console, I don't know if that is true for our case.
+       * And this also will require to have the process id
+       */
+      TerminateProcess((HANDLE) fork_child, 0);
 #endif
+  } else {
+      ld.go = FALSE;
+  }
 }
 
 void
 kill_capture_child(void)
 {
-  /*
-   * XXX - find some way of signaling the child in Win32.
-   */
-#ifndef _WIN32
   if (fork_child != -1)
-    kill(fork_child, SIGTERM); /* SIGTERM so it can clean up if necessary */
+#ifndef _WIN32
+      kill(fork_child, SIGTERM);       /* SIGTERM so it can clean up if necessary */
+#else
+      /* XXX: this is not the preferred method of closing a process!
+       * the clean way would be getting the process id of the child process,
+       * then getting window handle hWnd of that process (using EnumChildWindows),
+       * and then do a SendMessage(hWnd, WM_CLOSE, 0, 0) 
+       *
+       * Unfortunately, I don't know how to get the process id from the handle */
+      /* Hint: OpenProcess will get an handle from the id, not vice versa :-(
+       *
+       * Hint: GenerateConsoleCtrlEvent() will only work, if both processes are 
+       * running in the same console, I don't know if that is true for our case.
+       * And this also will require to have the process id
+       */
+      TerminateProcess((HANDLE) fork_child, 0);
 #endif
 }
 
+/* one packet was captured, process it */
 static void
 capture_pcap_cb(guchar *user, const struct pcap_pkthdr *phdr,
   const guchar *pd)
@@ -2200,6 +2269,7 @@ capture_pcap_cb(guchar *user, const struct pcap_pkthdr *phdr,
   loop_data *ld = (loop_data *) user;
   int err;
 
+  /* user told us to stop after x packets, do we have enough? */
   if ((++ld->counts.total >= ld->max) && (ld->max > 0))
   {
      ld->go = FALSE;
@@ -2277,6 +2347,9 @@ capture_pcap_cb(guchar *user, const struct pcap_pkthdr *phdr,
     case WTAP_ENCAP_ARCNET_LINUX:
       capture_arcnet(pd, whdr.caplen, &ld->counts, TRUE, FALSE);
       break;
+    case WTAP_ENCAP_APPLE_IP_OVER_IEEE1394:
+      capture_ap1394(pd, 0, whdr.caplen, &ld->counts);
+      break;
     /* XXX - some ATM drivers on FreeBSD might prepend a 4-byte ATM
        pseudo-header to DLT_ATM_RFC1483, with LLC header following;
        we might have to implement that at some point. */