Be a little more explicit in our description of tvb_get_ptr.
[metze/wireshark/wip.git] / tshark.c
index bbaf15ceef8428c4a0396fa2d58b286a96cccf3c..f6f069b88b084d2f5306cbcf95cafe071ab9b907 100644 (file)
--- a/tshark.c
+++ b/tshark.c
@@ -1,4 +1,7 @@
 /* tshark.c
+ *
+ * Text-mode variant of Wireshark, along the lines of tcpdump and snoop,
+ * by Gilbert Ramirez <gram@alumni.rice.edu> and Guy Harris <guy@alum.mit.edu>.
  *
  * $Id$
  *
@@ -6,9 +9,6 @@
  * By Gerald Combs <gerald@wireshark.org>
  * Copyright 1998 Gerald Combs
  *
- * Text-mode variant, by Gilbert Ramirez <gram@alumni.rice.edu>
- * and Guy Harris <guy@alum.mit.edu>.
- *
  * This program is free software; you can redistribute it and/or
  * modify it under the terms of the GNU General Public License
  * as published by the Free Software Foundation; either version 2
@@ -62,8 +62,7 @@
 #include <glib.h>
 #include <epan/epan.h>
 #include <epan/filesystem.h>
-#include <epan/privileges.h>
-#include <wiretap/file_util.h>
+#include <wsutil/privileges.h>
 
 #include "globals.h"
 #include <epan/timestamp.h>
@@ -78,7 +77,6 @@
 #include "clopts_common.h"
 #include "cmdarg_err.h"
 #include "version_info.h"
-#include <epan/conversation.h>
 #include <epan/plugins.h>
 #include "register.h"
 #include <epan/epan_dissect.h>
@@ -97,7 +95,6 @@
 #endif /* _WIN32 */
 #include "capture_sync.h"
 #endif /* HAVE_LIBPCAP */
-#include "epan/emem.h"
 #include "log.h"
 #include <epan/funnel.h>
 
@@ -147,7 +144,7 @@ static const char please_report[] =
 static gboolean print_packet_counts;
 
 
-static capture_options capture_opts;
+static capture_options global_capture_opts;
 
 #ifdef SIGINFO
 static gboolean infodelay;     /* if TRUE, don't print capture info in SIGINFO handler */
@@ -155,8 +152,6 @@ static gboolean infoprint;  /* if TRUE, print capture info after clearing infodel
 #endif /* SIGINFO */
 
 static int capture(void);
-static void capture_pcap_cb(u_char *, const struct pcap_pkthdr *,
-  const u_char *);
 static void report_counts(void);
 #ifdef _WIN32
 static BOOL WINAPI capture_cleanup(DWORD);
@@ -171,7 +166,7 @@ static void report_counts_siginfo(int);
 static int load_cap_file(capture_file *, char *, int, int, gint64);
 static gboolean process_packet(capture_file *cf, gint64 offset,
     const struct wtap_pkthdr *whdr, union wtap_pseudo_header *pseudo_header,
-    const guchar *pd);
+    const guchar *pd, gboolean filtering_tap_listeners, guint tap_flags);
 static void show_capture_file_io_error(const char *, int, gboolean);
 static void show_print_file_io_error(int err);
 static gboolean write_preamble(capture_file *cf);
@@ -184,9 +179,38 @@ static void open_failure_message(const char *filename, int err,
     gboolean for_writing);
 static void failure_message(const char *msg_format, va_list ap);
 static void read_failure_message(const char *filename, int err);
+static void write_failure_message(const char *filename, int err);
 
 capture_file cfile;
 
+/*
+ * Mark a particular frame.
+ * Copied from file.c
+ */
+void
+cf_mark_frame(capture_file *cf, frame_data *frame)
+{
+  if (! frame->flags.marked) {
+    frame->flags.marked = TRUE;
+    if (cf->count > cf->marked_count)
+      cf->marked_count++;
+  }
+}
+
+/*
+ * Unmark a particular frame.
+ * Copied from file.c
+ */
+void
+cf_unmark_frame(capture_file *cf, frame_data *frame)
+{
+  if (frame->flags.marked) {
+    frame->flags.marked = FALSE;
+    if (cf->marked_count > 0)
+      cf->marked_count--;
+  }
+}
+
 
 static void list_capture_types(void) {
     int i;
@@ -260,7 +284,9 @@ print_usage(gboolean print_ver)
 
   /*fprintf(output, "\n");*/
   fprintf(output, "Output:\n");
-  fprintf(output, "  -w <outfile|->           set the output filename (or '-' for stdout)\n");
+  fprintf(output, "  -w <outfile|->           write packets to a pcap-format file named \"outfile\"\n");
+  fprintf(output, "                           (or to the standard output for \"-\")\n");
+  fprintf(output, "  -C <config profile>      start with specified configuration profile\n");
   fprintf(output, "  -F <output file type>    set the output file type, default is libpcap\n");
   fprintf(output, "                           an empty \"-F\" option will list the file types\n");
   fprintf(output, "  -V                       add output of packet tree        (Packet Details)\n");
@@ -275,7 +301,7 @@ print_usage(gboolean print_ver)
   fprintf(output, "     separator=/t|/s|<char> select tab, space, printable character as separator\n");
   fprintf(output, "     quote=d|s|n           select double, single, no quotes for values\n");
   fprintf(output, "  -t ad|a|r|d|dd|e         output format of time stamps (def: r: rel. to first)\n");
-  fprintf(output, "  -l                       flush output after each packet\n");
+  fprintf(output, "  -l                       flush standard output after each packet\n");
   fprintf(output, "  -q                       be more quiet on stdout (e.g. when using statistics)\n");
   fprintf(output, "  -X <key>:<value>         eXtension options, see the man page for details\n");
   fprintf(output, "  -z <statistics>          various statistics, see the man page for details\n");
@@ -285,6 +311,7 @@ print_usage(gboolean print_ver)
   fprintf(output, "  -h                       display this help and exit\n");
   fprintf(output, "  -v                       display version info and exit\n");
   fprintf(output, "  -o <name>:<value> ...    override preference setting\n");
+  fprintf(output, "  -K <keytab>              keytab file to use for kerberos decryption\n");
 }
 
 /*
@@ -422,12 +449,11 @@ add_decode_as(const gchar *cl_param)
   ftenum_t                      dissector_table_selector_type;
   struct protocol_name_search   user_protocol_name;
 
-/* The following code will allocate and copy the command-line options in a string pointed by decoded_param */
+  /* The following code will allocate and copy the command-line options in a string pointed by decoded_param */
 
   g_assert(cl_param);
-  decoded_param = g_malloc( sizeof(gchar) * (strlen(cl_param) + 1) ); /* Allocate enough space to have a working copy of the command-line parameter */
+  decoded_param = g_strdup(cl_param);
   g_assert(decoded_param);
-  strcpy(decoded_param, cl_param);
 
 
   /* The lines below will parse this string (modifying it) to extract all
@@ -525,6 +551,7 @@ add_decode_as(const gchar *cl_param)
 
   case FT_STRING:
   case FT_STRINGZ:
+  case FT_EBCDIC:
     /* The selector for this table is a string. */
     break;
 
@@ -621,6 +648,7 @@ add_decode_as(const gchar *cl_param)
 
   case FT_STRING:
   case FT_STRINGZ:
+  case FT_EBCDIC:
     /* The selector for this table is a string. */
     dissector_change_string(table_name, selector_str, dissector_matching);
     break;
@@ -635,9 +663,29 @@ add_decode_as(const gchar *cl_param)
 }
 
 static void
-log_func_ignore (const gchar *log_domain _U_, GLogLevelFlags log_level _U_,
-    const gchar *message _U_, gpointer user_data _U_)
+tshark_log_handler (const gchar *log_domain, GLogLevelFlags log_level,
+    const gchar *message, gpointer user_data)
 {
+  /* ignore log message, if log_level isn't interesting based
+     upon the console log preferences.
+     If the preferences haven't been loaded loaded yet, display the 
+     message anyway.
+
+     The default console_log_level preference value is such that only 
+       ERROR, CRITICAL and WARNING level messages are processed;
+       MESSAGE, INFO and DEBUG level messages are ignored.
+       
+     XXX: Aug 07, 2009: Prior tshark g_log code was hardwired to process only
+           ERROR and CRITICAL level messages so the current code is a behavioral
+           change.  The current behavior is the same as in Wireshark.
+  */
+  if((log_level & G_LOG_LEVEL_MASK & prefs.console_log_level) == 0 &&
+     prefs.console_log_level != 0) {
+    return;
+  }
+
+  g_log_default_handler(log_domain, log_level, message, user_data);
+
 }
 
 static char *
@@ -657,7 +705,7 @@ output_file_description(const char *fname)
 }
 
 static void
-print_current_user() {
+print_current_user(void) {
   gchar *cur_user, *cur_group;
   if (started_with_special_privs()) {
     cur_user = get_cur_username();
@@ -674,8 +722,9 @@ print_current_user() {
 }
 
 static void
-check_npf_sys() {
+check_capture_privs(void) {
 #ifdef _WIN32
+  load_wpcap();
   /* Warn the user if npf.sys isn't loaded. */
   if (!npf_sys_is_running() && get_os_major_version() >= 6) {
     fprintf(stderr, "The NPF driver isn't running.  You may have trouble "
@@ -724,7 +773,7 @@ main(int argc, char *argv[])
   GLogLevelFlags       log_flags;
   int                  optind_initial;
 
-#define OPTSTRING_INIT "a:b:c:d:De:E:f:F:G:hi:lLnN:o:pqr:R:s:St:T:vVw:xX:y:z:"
+#define OPTSTRING_INIT "a:b:c:C:d:De:E:f:F:G:hi:K:lLnN:o:pqr:R:s:St:T:vVw:xX:y:z:"
 #ifdef HAVE_LIBPCAP
 #ifdef _WIN32
 #define OPTSTRING_WIN32 "B:"
@@ -745,7 +794,7 @@ main(int argc, char *argv[])
   /*
    * Attempt to get the pathname of the executable file.
    */
-  init_progfile_dir_error = init_progfile_dir(argv[0]);
+  init_progfile_dir_error = init_progfile_dir(argv[0], main);
   if (init_progfile_dir_error != NULL) {
     fprintf(stderr, "tshark: Can't get pathname of tshark program: %s.\n",
             init_progfile_dir_error);
@@ -760,6 +809,14 @@ main(int argc, char *argv[])
 
   while ((opt = getopt(argc, argv, optstring)) != -1) {
          switch (opt) {
+                 case 'C':        /* Configuration Profile */
+                         if (profile_exists (optarg)) {
+                                 set_profile_name (optarg);
+                         } else {
+                                 cmdarg_err("Configuration Profile \"%s\" does not exist", optarg);
+                                 exit(1);
+                         }
+                         break;
                  case 'X':
                          ex_opt_add(optarg);
                          break;
@@ -771,24 +828,39 @@ main(int argc, char *argv[])
   optind = optind_initial;
   opterr = 1;
 
-  /* nothing more than the standard GLib handler, but without a warning */
+
+
+/** Send All g_log messages to our own handler **/
+
   log_flags =
+                   G_LOG_LEVEL_ERROR|
+                   G_LOG_LEVEL_CRITICAL|
                    G_LOG_LEVEL_WARNING|
                    G_LOG_LEVEL_MESSAGE|
                    G_LOG_LEVEL_INFO|
-                   G_LOG_LEVEL_DEBUG;
+                   G_LOG_LEVEL_DEBUG|
+                   G_LOG_FLAG_FATAL|G_LOG_FLAG_RECURSION;
 
   g_log_set_handler(NULL,
                    log_flags,
-                   log_func_ignore, NULL /* user_data */);
+                   tshark_log_handler, NULL /* user_data */);
+  g_log_set_handler(LOG_DOMAIN_MAIN,
+                   log_flags,
+                   tshark_log_handler, NULL /* user_data */);
+
+#ifdef HAVE_LIBPCAP
   g_log_set_handler(LOG_DOMAIN_CAPTURE,
                    log_flags,
-                   log_func_ignore, NULL /* user_data */);
+                   tshark_log_handler, NULL /* user_data */);
+  g_log_set_handler(LOG_DOMAIN_CAPTURE_CHILD,
+                   log_flags,
+                   tshark_log_handler, NULL /* user_data */);
+#endif
 
   initialize_funnel_ops();
 
 #ifdef HAVE_LIBPCAP
-  capture_opts_init(&capture_opts, NULL /* cfile */);
+  capture_opts_init(&global_capture_opts, &cfile);
 #endif
 
   timestamp_set_type(TS_RELATIVE);
@@ -799,7 +871,8 @@ main(int argc, char *argv[])
      dissectors, and we must do it before we read the preferences, in
      case any dissectors register preferences. */
   epan_init(register_all_protocols, register_all_protocol_handoffs, NULL, NULL,
-            failure_message, open_failure_message, read_failure_message);
+            failure_message, open_failure_message, read_failure_message,
+            write_failure_message);
 
   /* Register all tap listeners; we do this before we parse the arguments,
      as the "-z" argument can specify a registered tap. */
@@ -825,6 +898,8 @@ main(int argc, char *argv[])
      If none of our build or other processes uses "-G" with no arguments,
      we can just process it with the other arguments. */
   if (argc >= 2 && strcmp(argv[1], "-G") == 0) {
+         proto_initialize_all_prefixes();
+
     if (argc == 2)
       proto_registrar_dump_fields(1);
     else {
@@ -913,6 +988,7 @@ main(int argc, char *argv[])
     g_free(dp_path);
   }
 
+  check_capture_privs();
 
   init_cap_file(&cfile);
 
@@ -945,7 +1021,7 @@ main(int argc, char *argv[])
       case 'B':        /* Buffer size */
 #endif /* _WIN32 */
 #ifdef HAVE_LIBPCAP
-        status = capture_opts_add_opt(&capture_opts, opt, optarg, &start_capture);
+        status = capture_opts_add_opt(&global_capture_opts, opt, optarg, &start_capture);
         if(status != 0) {
             exit(status);
         }
@@ -954,13 +1030,20 @@ main(int argc, char *argv[])
         arg_error = TRUE;
 #endif
         break;
+      case 'C':
+        /* Configuration profile settings were already processed just ignore them this time*/
+       break;
       case 'd':        /* Decode as rule */
         if (!add_decode_as(optarg))
           exit(1);
         break;
+#if defined(HAVE_HEIMDAL_KERBEROS) || defined(HAVE_MIT_KERBEROS)
+      case 'K':        /* Kerberos keytab file */
+        read_keytab_file(optarg);
+        break;
+#endif
       case 'D':        /* Print a list of capture devices and exit */
 #ifdef HAVE_LIBPCAP
-        check_npf_sys();
         status = capture_opts_list_interfaces(FALSE);
         exit(status);
 #else
@@ -1010,7 +1093,6 @@ main(int argc, char *argv[])
       case 'L':        /* Print list of link-layer types and exit */
 #ifdef HAVE_LIBPCAP
         list_link_layer_types = TRUE;
-        break;
 #else
         capture_option_specified = TRUE;
         arg_error = TRUE;
@@ -1024,8 +1106,9 @@ main(int argc, char *argv[])
           g_resolv_flags = RESOLV_NONE;
         badopt = string_to_name_resolve(optarg, &g_resolv_flags);
         if (badopt != '\0') {
-          cmdarg_err("-N specifies unknown resolving option '%c'; valid options are 'm', 'n', and 't'",
+          cmdarg_err("-N specifies unknown resolving option '%c';",
                      badopt);
+          cmdarg_err_cont( "           Valid options are 'm', 'n', 't', and 'C'");
           exit(1);
         }
         break;
@@ -1175,13 +1258,13 @@ main(int argc, char *argv[])
       rfilter = get_args_as_string(argc, argv, optind);
     } else {
 #ifdef HAVE_LIBPCAP
-      if (capture_opts.has_cfilter) {
+      if (global_capture_opts.has_cfilter) {
         cmdarg_err("Capture filters were specified both with \"-f\""
             " and with additional command-line arguments");
         exit(1);
       }
-      capture_opts.has_cfilter = TRUE;
-      capture_opts.cfilter = get_args_as_string(argc, argv, optind);
+      global_capture_opts.has_cfilter = TRUE;
+      global_capture_opts.cfilter = get_args_as_string(argc, argv, optind);
 #else
       capture_option_specified = TRUE;
 #endif
@@ -1189,7 +1272,7 @@ main(int argc, char *argv[])
   }
 
 #ifdef HAVE_LIBPCAP
-  if (!capture_opts.saving_to_file) {
+  if (!global_capture_opts.saving_to_file) {
     /* We're not saving the capture to a file; if "-q" wasn't specified,
        we should print packet information */
     if (!quiet)
@@ -1200,7 +1283,7 @@ main(int argc, char *argv[])
        output, reject the request.  At best, we could redirect that
        to the standard error; we *can't* write both to the standard
        output and have either of them be useful. */
-    if (strcmp(capture_opts.save_file, "-") == 0 && print_packet_info) {
+    if (strcmp(global_capture_opts.save_file, "-") == 0 && print_packet_info) {
       cmdarg_err("You can't write both raw packet data and dissected packets"
           " to the standard output.");
       exit(1);
@@ -1227,7 +1310,7 @@ main(int argc, char *argv[])
      support in capture files we read). */
 #ifdef HAVE_LIBPCAP
   if (cf_name != NULL) {
-    if (capture_opts.has_cfilter) {
+    if (global_capture_opts.has_cfilter) {
       cmdarg_err("Only read filters, not capture filters, "
           "can be specified when reading a capture file.");
       exit(1);
@@ -1252,7 +1335,7 @@ main(int argc, char *argv[])
       exit(1);
     }
     /* No - did they specify a ring buffer option? */
-    if (capture_opts.multi_files_on) {
+    if (global_capture_opts.multi_files_on) {
       cmdarg_err("Ring buffer requested, but a capture isn't being done.");
       exit(1);
     }
@@ -1262,22 +1345,22 @@ main(int argc, char *argv[])
        * "-r" was specified, so we're reading a capture file.
        * Capture options don't apply here.
        */
-      if (capture_opts.multi_files_on) {
+      if (global_capture_opts.multi_files_on) {
         cmdarg_err("Multiple capture files requested, but "
                    "a capture isn't being done.");
         exit(1);
       }
-      if (capture_opts.has_file_duration) {
+      if (global_capture_opts.has_file_duration) {
         cmdarg_err("Switching capture files after a time interval was specified, but "
                    "a capture isn't being done.");
         exit(1);
       }
-      if (capture_opts.has_ring_num_files) {
+      if (global_capture_opts.has_ring_num_files) {
         cmdarg_err("A ring buffer of capture files was specified, but "
           "a capture isn't being done.");
         exit(1);
       }
-      if (capture_opts.has_autostop_files) {
+      if (global_capture_opts.has_autostop_files) {
         cmdarg_err("A maximum number of capture files was specified, but "
           "a capture isn't being done.");
         exit(1);
@@ -1287,7 +1370,7 @@ main(int argc, char *argv[])
        * and byte count as well as a write file. Other autostop options remain valid
        * only for a write file.
        */
-      if (capture_opts.has_autostop_duration) {
+      if (global_capture_opts.has_autostop_duration) {
         cmdarg_err("A maximum capture time was specified, but "
           "a capture isn't being done.");
         exit(1);
@@ -1296,7 +1379,7 @@ main(int argc, char *argv[])
       /*
        * "-r" wasn't specified, so we're doing a live capture.
        */
-      if (capture_opts.saving_to_file) {
+      if (global_capture_opts.saving_to_file) {
         /* They specified a "-w" flag, so we'll be saving to a capture file. */
 
         /* When capturing, we only support writing libpcap format. */
@@ -1304,23 +1387,23 @@ main(int argc, char *argv[])
           cmdarg_err("Live captures can only be saved in libpcap format.");
           exit(1);
         }
-        if (capture_opts.multi_files_on) {
+        if (global_capture_opts.multi_files_on) {
           /* Multiple-file mode doesn't work under certain conditions:
              a) it doesn't work if you're writing to the standard output;
              b) it doesn't work if you're writing to a pipe;
          */
-          if (strcmp(capture_opts.save_file, "-") == 0) {
+          if (strcmp(global_capture_opts.save_file, "-") == 0) {
             cmdarg_err("Multiple capture files requested, but "
               "the capture is being written to the standard output.");
             exit(1);
           }
-          if (capture_opts.output_to_pipe) {
+          if (global_capture_opts.output_to_pipe) {
             cmdarg_err("Multiple capture files requested, but "
               "the capture file is a pipe.");
             exit(1);
           }
-          if (!capture_opts.has_autostop_filesize &&
-             !capture_opts.has_file_duration) {
+          if (!global_capture_opts.has_autostop_filesize &&
+             !global_capture_opts.has_file_duration) {
             cmdarg_err("Multiple capture files requested, but "
               "no maximum capture file size or duration was specified.");
             exit(1);
@@ -1330,12 +1413,12 @@ main(int argc, char *argv[])
         /* They didn't specify a "-w" flag, so we won't be saving to a
            capture file.  Check for options that only make sense if
            we're saving to a file. */
-        if (capture_opts.has_autostop_filesize) {
+        if (global_capture_opts.has_autostop_filesize) {
           cmdarg_err("Maximum capture file size specified, but "
            "capture isn't being saved to a file.");
           exit(1);
         }
-        if (capture_opts.multi_files_on) {
+        if (global_capture_opts.multi_files_on) {
           cmdarg_err("Multiple capture files requested, but "
             "the capture isn't being saved to a file.");
           exit(1);
@@ -1370,6 +1453,18 @@ main(int argc, char *argv[])
   for (i = 0; i < cfile.cinfo.num_cols; i++) {
     cfile.cinfo.col_fmt[i] = get_column_format(i);
     cfile.cinfo.col_title[i] = g_strdup(get_column_title(i));
+    if (cfile.cinfo.col_fmt[i] == COL_CUSTOM) {
+      cfile.cinfo.col_custom_field[i] = g_strdup(get_column_custom_field(i));
+      if(!dfilter_compile(cfile.cinfo.col_custom_field[i], &cfile.cinfo.col_custom_dfilter[i])) {
+        /* XXX: Should we issue a warning? */
+        g_free(cfile.cinfo.col_custom_field[i]);
+        cfile.cinfo.col_custom_field[i] = NULL;
+        cfile.cinfo.col_custom_dfilter[i] = NULL;
+      }
+    } else {
+      cfile.cinfo.col_custom_field[i] = NULL;
+      cfile.cinfo.col_custom_dfilter[i] = NULL;
+    }
     cfile.cinfo.fmt_matx[i] = (gboolean *) g_malloc0(sizeof(gboolean) *
       NUM_COL_FMTS);
     get_column_format_matches(cfile.cinfo.fmt_matx[i], cfile.cinfo.col_fmt[i]);
@@ -1379,8 +1474,8 @@ main(int argc, char *argv[])
     else
       cfile.cinfo.col_buf[i] = (gchar *) g_malloc(sizeof(gchar) * COL_MAX_LEN);
     cfile.cinfo.col_fence[i] = 0;
-    cfile.cinfo.col_expr[i] = (gchar *) g_malloc(sizeof(gchar) * COL_MAX_LEN);
-    cfile.cinfo.col_expr_val[i] = (gchar *) g_malloc(sizeof(gchar) * COL_MAX_LEN);
+    cfile.cinfo.col_expr.col_expr[i] = (gchar *) g_malloc(sizeof(gchar) * COL_MAX_LEN);
+    cfile.cinfo.col_expr.col_expr_val[i] = (gchar *) g_malloc(sizeof(gchar) * COL_MAX_LEN);
   }
 
   for (i = 0; i < cfile.cinfo.num_cols; i++) {
@@ -1397,8 +1492,8 @@ main(int argc, char *argv[])
   }
 
 #ifdef HAVE_LIBPCAP
-  capture_opts_trim_snaplen(&capture_opts, MIN_PACKET_SIZE);
-  capture_opts_trim_ring_num_files(&capture_opts);
+  capture_opts_trim_snaplen(&global_capture_opts, MIN_PACKET_SIZE);
+  capture_opts_trim_ring_num_files(&global_capture_opts);
 #endif
 
   if (rfilter != NULL) {
@@ -1500,9 +1595,9 @@ main(int argc, char *argv[])
 
     /* Process the packets in the file */
 #ifdef HAVE_LIBPCAP
-    err = load_cap_file(&cfile, capture_opts.save_file, out_file_type,
-        capture_opts.has_autostop_packets ? capture_opts.autostop_packets : 0,
-        capture_opts.has_autostop_filesize ? capture_opts.autostop_filesize : 0);
+    err = load_cap_file(&cfile, global_capture_opts.save_file, out_file_type,
+        global_capture_opts.has_autostop_packets ? global_capture_opts.autostop_packets : 0,
+        global_capture_opts.has_autostop_filesize ? global_capture_opts.autostop_filesize : 0);
 #else
     err = load_cap_file(&cfile, NULL, out_file_type, 0, 0);
 #endif
@@ -1530,15 +1625,14 @@ main(int argc, char *argv[])
 #endif
 
     /* trim the interface name and exit if that failed */
-    if (!capture_opts_trim_iface(&capture_opts,
+    if (!capture_opts_trim_iface(&global_capture_opts,
         (prefs->capture_device) ? get_if_name(prefs->capture_device) : NULL)) {
         exit(2);
     }
 
     /* if requested, list the link layer types and exit */
     if (list_link_layer_types) {
-        check_npf_sys();
-        status = capture_opts_list_link_layer_types(&capture_opts, FALSE);
+        status = capture_opts_list_link_layer_types(&global_capture_opts, FALSE);
         exit(status);
     }
 
@@ -1610,7 +1704,9 @@ typedef struct pipe_input_tag {
     int                 *child_process;
     pipe_input_cb_t     input_cb;
     guint               pipe_input_id;
-       GStaticMutex            callback_running;
+#ifdef _WIN32
+    GStaticMutex               callback_running;
+#endif
 } pipe_input_t;
 
 static pipe_input_t pipe_input;
@@ -1675,28 +1771,20 @@ pipe_timer_cb(gpointer data)
        /* we didn't stopped the timer, so let it run */
        return TRUE;
 }
-#else
-/* The timer has expired, see if there's stuff to read from the pipe,
-   if so, do the callback */
-static gint
-pipe_timer_cb(gpointer data)
-{
-    /* XXX - this has to be implemented */
-    g_assert(0);
-}
 #endif
 
 
-void pipe_input_set_handler(gint source, gpointer user_data, int *child_process, pipe_input_cb_t input_cb)
+void
+pipe_input_set_handler(gint source, gpointer user_data, int *child_process, pipe_input_cb_t input_cb)
 {
 
     pipe_input.source                  = source;
-    pipe_input.child_process   = child_process;
+    pipe_input.child_process           = child_process;
     pipe_input.user_data               = user_data;
     pipe_input.input_cb                        = input_cb;
-       pipe_input.callback_running     = G_STATIC_MUTEX_INIT;
 
 #ifdef _WIN32
+    g_static_mutex_init(&pipe_input.callback_running);
     /* 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
@@ -1704,20 +1792,6 @@ void pipe_input_set_handler(gint source, gpointer user_data, int *child_process,
        timeout. */
        /*g_log(NULL, G_LOG_LEVEL_DEBUG, "pipe_input_set_handler: new");*/
     pipe_input.pipe_input_id = g_timeout_add(200, pipe_timer_cb, &pipe_input);
-#else
-    pipe_input.pipe_input_id = g_timeout_add(200, pipe_timer_cb, &pipe_input);
-
-    /* XXX - in gui_utils.c, this was: 
-        * pipe_input->pipe_input_id = gtk_input_add_full (source,
-        *                   GDK_INPUT_READ|GDK_INPUT_EXCEPTION, */
-#if 0
-    pipe_input.pipe_input_id = g_input_add_full(source,
-                                     200,
-                                     pipe_input_cb,
-                                     NULL,
-                                     &pipe_input,
-                                     NULL);
-#endif
 #endif
 }
 
@@ -1727,7 +1801,12 @@ static int
 capture(void)
 {
   gboolean ret;
-
+#ifdef USE_TSHARK_SELECT
+  fd_set readfds;
+#endif
+#ifndef _WIN32
+  struct sigaction action, oldaction;
+#endif
 
   /*
    * XXX - dropping privileges is still required, until code cleanup is done
@@ -1737,7 +1816,7 @@ capture(void)
    * therefore we don't need to drop these privileges
    * The only thing we might want to keep is a warning if tshark is run as root,
    * as it's no longer necessary and potentially dangerous.
-   * 
+   *
    * THE FOLLOWING IS THE FORMER COMMENT WHICH IS NO LONGER REALLY VALID:
    * We've opened the capture device, so we shouldn't need any special
    * privileges any more; relinquish those privileges.
@@ -1753,62 +1832,92 @@ capture(void)
   relinquish_special_privs_perm();
   print_current_user();
 
-  check_npf_sys();
-
   /* Initialize all data structures used for dissection. */
   init_dissection();
-  
+
 #ifdef _WIN32
   /* Catch a CTRL+C event and, if we get it, clean up and exit. */
   SetConsoleCtrlHandler(capture_cleanup, TRUE);
 #else /* _WIN32 */
   /* Catch SIGINT and SIGTERM and, if we get either of them, clean up
-     and exit.
-     XXX - deal with signal semantics on various UNIX platforms.  Or just
-     use "sigaction()" and be done with it? */
-  signal(SIGTERM, capture_cleanup);
-  signal(SIGINT, capture_cleanup);
-  if ((oldhandler = signal(SIGHUP, capture_cleanup)) != SIG_DFL)
-    signal(SIGHUP, oldhandler);
+     and exit. */
+  action.sa_handler = capture_cleanup;
+  action.sa_flags = 0;
+  sigemptyset(&action.sa_mask);
+  sigaction(SIGTERM, &action, NULL);
+  sigaction(SIGINT, &action, NULL);
+  sigaction(SIGHUP, NULL, &oldaction);
+  if (oldaction.sa_handler == SIG_DFL)
+    sigaction(SIGHUP, &action, NULL);
 
 #ifdef SIGINFO
   /* Catch SIGINFO and, if we get it and we're capturing to a file in
      quiet mode, report the number of packets we've captured. */
-  signal(SIGINFO, report_counts_siginfo);
+  action.sa_handler = report_counts_siginfo;
+  action.sa_flags = 0;
+  sigemptyset(&action.sa_mask);
+  sigaction(SIGINFO, &action, NULL);
 #endif /* SIGINFO */
 #endif /* _WIN32 */
 
-  capture_opts.state = CAPTURE_PREPARING;
+  global_capture_opts.state = CAPTURE_PREPARING;
 
   /* Let the user know what interface was chosen. */
-  capture_opts.iface_descr = get_interface_descriptive_name(capture_opts.iface);
-  fprintf(stderr, "Capturing on %s\n", capture_opts.iface_descr);
-
-  ret = sync_pipe_start(&capture_opts);
-  if(ret) {
-       /* the actual capture loop
-        *
-        * XXX - glib doesn't seem to provide any event based loop handling.
-        *
-        * XXX - for whatever reason, 
-        * calling g_main_loop_new() ends up in 100% cpu load.
-        * Therefore use simple g_usleep() to poll for new input. */
-#ifdef USE_BROKEN_G_MAIN_LOOP
-    /*loop = g_main_loop_new(NULL, FALSE);*/
-       /*g_main_loop_run(loop);*/
-    loop = g_main_new(FALSE);
-       g_main_run(loop);
-#else
-    loop_running = TRUE;
+  global_capture_opts.iface_descr = get_interface_descriptive_name(global_capture_opts.iface);
+  fprintf(stderr, "Capturing on %s\n", global_capture_opts.iface_descr);
+
+  ret = sync_pipe_start(&global_capture_opts);
 
-       while(loop_running && pipe_timer_cb(&pipe_input)) {
-               g_usleep(200000);       /* 200 ms */
-       }
+  if (!ret)
+    return FALSE;
+
+    /* the actual capture loop
+     *
+     * XXX - glib doesn't seem to provide any event based loop handling.
+     *
+     * XXX - for whatever reason,
+     * calling g_main_loop_new() ends up in 100% cpu load.
+     *
+     * But that doesn't matter: in UNIX we can use select() to find an input
+     * source with something to do.
+     *
+     * But that doesn't matter because we're in a CLI (that doesn't need to
+     * update a GUI or something at the same time) so it's OK if we block
+     * trying to read from the pipe.
+     *
+     * So all the stuff in USE_TSHARK_SELECT could be removed unless I'm
+     * wrong (but I leave it there in case I am...).
+     */
+
+#ifdef USE_TSHARK_SELECT
+  FD_ZERO(&readfds);
+  FD_SET(pipe_input.source, &readfds);
 #endif
-       return TRUE;
-  } else {
+
+  loop_running = TRUE;
+
+  while (loop_running)
+  {
+#ifdef USE_TSHARK_SELECT
+    ret = select(pipe_input.source+1, &readfds, NULL, NULL, NULL);
+
+    if (ret == -1)
+    {
+      perror("select()");
+      return TRUE;
+    } else if (ret == 1) {
+#endif
+      /* Call the real handler */
+      if (!pipe_input.input_cb(pipe_input.source, pipe_input.user_data)) {
+       g_log(NULL, G_LOG_LEVEL_DEBUG, "input pipe closed");
        return FALSE;
+      }
+#ifdef USE_TSHARK_SELECT
+    }
+#endif
   }
+
+  return TRUE;
 }
 
 
@@ -1818,29 +1927,9 @@ void main_window_update(void)
 {
 }
 
-/* XXX - move the call to simple_dialog() out of capture_sync.c */
-#include "simple_dialog.h"
-
-/* capture_sync.c want's to tell us an error */
-gpointer simple_dialog(ESD_TYPE_E type, gint btn_mask,
-                                          const gchar *msg_format, ...)
-{
-       va_list ap;
-
-       /* XXX - do we need to display buttons and alike? */
-       va_start(ap, msg_format);
-       fprintf(stderr, "tshark: ");
-       vfprintf(stderr, msg_format, ap);
-       fprintf(stderr, "\n");
-       va_end(ap);
-
-       return NULL;
-}
-
-
 /* capture child detected an error */
 void
-capture_input_error_message(capture_options *capture_opts, char *error_msg, char *secondary_error_msg)
+capture_input_error_message(capture_options *capture_opts _U_, char *error_msg, char *secondary_error_msg)
 {
        cmdarg_err("%s", error_msg);
        cmdarg_err_cont("%s", secondary_error_msg);
@@ -1893,16 +1982,17 @@ capture_input_new_file(capture_options *capture_opts, gchar *new_file)
 
   g_assert(capture_opts->state == CAPTURE_PREPARING || capture_opts->state == CAPTURE_RUNNING);
 
-  capture_opts->cf = &cfile;
-
   /* free the old filename */
-  if(capture_opts->save_file != NULL) {
+  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) {
-               if(capture_opts->cf != NULL && ((capture_file *) capture_opts->cf)->wth != NULL) {
-                       wtap_close(((capture_file *) capture_opts->cf)->wth);
-               }
+      if ( ((capture_file *) capture_opts->cf)->wth != NULL) {
+        wtap_close(((capture_file *) capture_opts->cf)->wth);
+      }
+      ((capture_file *) capture_opts->cf)->state = FILE_CLOSED;
     }
+
     g_free(capture_opts->save_file);
     is_tempfile = FALSE;
   } else {
@@ -1925,7 +2015,6 @@ capture_input_new_file(capture_options *capture_opts, gchar *new_file)
       g_free(capture_opts->save_file);
       capture_opts->save_file = NULL;
       return FALSE;
-      break;
     }
   }
 
@@ -1944,7 +2033,8 @@ capture_input_new_packets(capture_options *capture_opts, int to_read)
   gchar        *err_info;
   gint64       data_offset;
   capture_file *cf = capture_opts->cf;
-
+  gboolean filtering_tap_listeners;
+  guint tap_flags;
 
 #ifdef SIGINFO
   /*
@@ -1955,25 +2045,38 @@ capture_input_new_packets(capture_options *capture_opts, int to_read)
   infodelay = TRUE;
 #endif /* SIGINFO */
 
+  /* Do we have any tap listeners with filters? */
+  filtering_tap_listeners = have_filtering_tap_listeners();
+
+  /* Get the union of the flags for all tap listeners. */
+  tap_flags = union_of_tap_listener_flags();
+
   if(do_dissection) {
-         while (to_read-- && cf->wth) {
-                 ret = wtap_read(cf->wth, &err, &err_info, &data_offset);
-                 if(ret == FALSE) {
-                         /* read from file failed, tell the capture child to stop */
-                         sync_pipe_stop(capture_opts);
-                         wtap_close(cf->wth);
-                         cf->wth = NULL;
-                 } else {
-                         ret = process_packet(cf, data_offset, wtap_phdr(cf->wth),
-                                                          wtap_pseudoheader(cf->wth), wtap_buf_ptr(cf->wth));
-                 }
-                 if (ret != FALSE) {
-                       /* packet sucessfully read and gone through the "Read Filter" */
-                       packet_count++;
-                 }
-         }
-  } 
-  
+    while (to_read-- && cf->wth) {
+      ret = wtap_read(cf->wth, &err, &err_info, &data_offset);
+      if(ret == FALSE) {
+        /* read from file failed, tell the capture child to stop */
+        sync_pipe_stop(capture_opts);
+        wtap_close(cf->wth);
+        cf->wth = NULL;
+      } else {
+        ret = process_packet(cf, data_offset, wtap_phdr(cf->wth),
+                             wtap_pseudoheader(cf->wth), wtap_buf_ptr(cf->wth),
+                             filtering_tap_listeners, tap_flags);
+      }
+      if (ret != FALSE) {
+        /* packet sucessfully read and gone through the "Read Filter" */
+        packet_count++;
+      }
+    }
+  } else {
+    /*
+     * Dumpcap's doing all the work; we're not doing any dissection.
+     * Count all the packets it wrote.
+     */
+    packet_count += to_read;
+  }
+
   if (print_packet_counts) {
       /* We're printing packet counts. */
       if (packet_count != 0) {
@@ -2000,17 +2103,11 @@ capture_input_new_packets(capture_options *capture_opts, int to_read)
 static void
 report_counts(void)
 {
-#ifdef SIGINFO
-  /* XXX - if we use sigaction, this doesn't have to be done.
-     (Yes, this isn't necessary on BSD, but just in case a system
-     where "signal()" has AT&T semantics adopts SIGINFO....) */
-  signal(SIGINFO, report_counts_siginfo);
-#endif /* SIGINFO */
-
   if (!print_packet_counts) {
     /* Report the count only if we aren't printing a packet count
        as packets arrive. */
-    fprintf(stderr, "%u packets captured\n", packet_count);
+    fprintf(stderr, "%u packet%s captured\n", packet_count,
+            plurality(packet_count, "", "s"));
   }
 #ifdef SIGINFO
   infoprint = FALSE; /* we just reported it */
@@ -2036,19 +2133,19 @@ report_counts_siginfo(int signum _U_)
 
 /* capture child detected any packet drops? */
 void
-capture_input_drops(capture_options *capture_opts, int dropped)
+capture_input_drops(capture_options *capture_opts _U_, guint32 dropped)
 {
-       if (print_packet_counts) {
-       /* We're printing packet counts to stderr.
-          Send a newline so that we move to the line after the packet count. */
-         fprintf(stderr, "\n");
-       }
-
-       if(dropped != 0) {
-               /* We're printing packet counts to stderr.
-                  Send a newline so that we move to the line after the packet count. */
-                 fprintf(stderr, "%u packet%s dropped\n", dropped, plurality(dropped, "", "s"));
-       }
+  if (print_packet_counts) {
+    /* We're printing packet counts to stderr.
+       Send a newline so that we move to the line after the packet count. */
+    fprintf(stderr, "\n");
+  }
+
+  if (dropped != 0) {
+    /* We're printing packet counts to stderr.
+       Send a newline so that we move to the line after the packet count. */
+    fprintf(stderr, "%u packet%s dropped\n", dropped, plurality(dropped, "", "s"));
+  }
 }
 
 
@@ -2056,22 +2153,16 @@ capture_input_drops(capture_options *capture_opts, int dropped)
 void
 capture_input_closed(capture_options *capture_opts)
 {
-       if (!print_packet_counts) {
-    /* Report the count only if we aren't printing a packet count
-       as packets arrive. */
-      fprintf(stderr, "%u packets captured\n", packet_count);
-  }
+  report_counts();
 
-       /*printf("capture_input_closed\n");*/
-
-       if(capture_opts->cf != NULL && ((capture_file *) capture_opts->cf)->wth != NULL) {
-               wtap_close(((capture_file *) capture_opts->cf)->wth);
-       }
+  if(capture_opts->cf != NULL && ((capture_file *) capture_opts->cf)->wth != NULL) {
+    wtap_close(((capture_file *) capture_opts->cf)->wth);
+  }
 #ifdef USE_BROKEN_G_MAIN_LOOP
-       /*g_main_loop_quit(loop);*/
-       g_main_quit(loop);
+  /*g_main_loop_quit(loop);*/
+  g_main_quit(loop);
 #else
-       loop_running = FALSE;
+  loop_running = FALSE;
 #endif
 }
 
@@ -2106,10 +2197,10 @@ capture_cleanup(DWORD ctrltype _U_)
      building it with Cygwin may make the problem go away). */
 
   /* tell the capture child to stop */
-  sync_pipe_stop(&capture_opts);
+  sync_pipe_stop(&global_capture_opts);
 
-  /* don't stop our own loop already here, otherwise status messages and 
-   * cleanup wouldn't be done properly. The child will indicate the stop of 
+  /* don't stop our own loop already here, otherwise status messages and
+   * cleanup wouldn't be done properly. The child will indicate the stop of
    * everything by calling capture_input_closed() later */
 
   return TRUE;
@@ -2118,11 +2209,8 @@ capture_cleanup(DWORD ctrltype _U_)
 static void
 capture_cleanup(int signum _U_)
 {
-  /* Longjmp back to the starting point; "pcap_dispatch()", on many
-     UNIX platforms, just keeps looping if it gets EINTR, so if we set
-     "ld.go" to FALSE and return, we won't break out of it and quit
-     capturing. */
-  longjmp(ld.stopenv, 1);
+  /* tell the capture child to stop */
+  sync_pipe_stop(&global_capture_opts);
 }
 #endif /* _WIN32 */
 #endif /* HAVE_LIBPCAP */
@@ -2138,6 +2226,8 @@ load_cap_file(capture_file *cf, char *save_file, int out_file_type,
   gchar        *err_info;
   gint64       data_offset;
   char         *save_file_string = NULL;
+  gboolean     filtering_tap_listeners;
+  guint        tap_flags;
 
   linktype = wtap_file_encap(cf->wth);
   if (save_file != NULL) {
@@ -2194,9 +2284,17 @@ load_cap_file(capture_file *cf, char *save_file, int out_file_type,
     }
     pdh = NULL;
   }
+
+  /* Do we have any tap listeners with filters? */
+  filtering_tap_listeners = have_filtering_tap_listeners();
+
+  /* Get the union of the flags for all tap listeners. */
+  tap_flags = union_of_tap_listener_flags();
+
   while (wtap_read(cf->wth, &err, &err_info, &data_offset)) {
     if (process_packet(cf, data_offset, wtap_phdr(cf->wth),
-                       wtap_pseudoheader(cf->wth), wtap_buf_ptr(cf->wth))) {
+                       wtap_pseudoheader(cf->wth), wtap_buf_ptr(cf->wth),
+                       filtering_tap_listeners, tap_flags)) {
       /* Either there's no read filtering or this packet passed the
          filter, so, if we're writing to a capture file, write
          this packet out. */
@@ -2211,10 +2309,11 @@ load_cap_file(capture_file *cf, char *save_file, int out_file_type,
         }
       }
       /* Stop reading if we have the maximum number of packets;
-       * note that a zero max_packet_count will never be matched
-       * (unless we roll over the packet number?)
+       * When the -c option has not been used, max_packet_count
+       * starts at 0, which practically means, never stop reading.
+       * (unless we roll over max_packet_count ?)
        */
-      if(max_packet_count == cf->count || (max_byte_count != 0 && data_offset >= max_byte_count)) {
+      if( (--max_packet_count == 0) || (max_byte_count != 0 && data_offset >= max_byte_count)) {
         err = 0; /* This is not an error */
         break;
       }
@@ -2227,6 +2326,7 @@ load_cap_file(capture_file *cf, char *save_file, int out_file_type,
     case WTAP_ERR_UNSUPPORTED_ENCAP:
       cmdarg_err("\"%s\" has a packet with a network type that TShark doesn't support.\n(%s)",
                  cf->filename, err_info);
+      g_free(err_info);
       break;
 
     case WTAP_ERR_CANT_READ:
@@ -2242,6 +2342,7 @@ load_cap_file(capture_file *cf, char *save_file, int out_file_type,
     case WTAP_ERR_BAD_RECORD:
       cmdarg_err("\"%s\" appears to be damaged or corrupt.\n(%s)",
                  cf->filename, err_info);
+      g_free(err_info);
       break;
 
     default:
@@ -2273,8 +2374,7 @@ out:
   wtap_close(cf->wth);
   cf->wth = NULL;
 
-  if (save_file_string != NULL)
-    g_free(save_file_string);
+  g_free(save_file_string);
 
   return err;
 }
@@ -2328,12 +2428,16 @@ fill_in_fdata(frame_data *fdata, capture_file *cf,
     cf->elapsed_time = fdata->rel_ts;
   }
 
+  /* If we don't have the time stamp of the previous displayed packet,
+     it's because this is the first packet that's being displayed.  Save the time
+     stamp of this packet as the time stamp of the previous displayed
+     packet. */
+  if (nstime_is_unset(&prev_dis_ts))
+    prev_dis_ts = fdata->abs_ts;
+
   /* Get the time elapsed between the previous displayed packet and
      this packet. */
-  if (nstime_is_unset(&prev_dis_ts))
-    nstime_set_zero(&fdata->del_dis_ts);
-  else
-    nstime_delta(&fdata->del_dis_ts, &fdata->abs_ts, &prev_dis_ts);
+  nstime_delta(&fdata->del_dis_ts, &fdata->abs_ts, &prev_dis_ts);
 
   /* Get the time elapsed between the previous captured packet and
      this packet. */
@@ -2351,10 +2455,12 @@ clear_fdata(frame_data *fdata)
 
 static gboolean
 process_packet(capture_file *cf, gint64 offset, const struct wtap_pkthdr *whdr,
-               union wtap_pseudo_header *pseudo_header, const guchar *pd)
+               union wtap_pseudo_header *pseudo_header, const guchar *pd,
+               gboolean filtering_tap_listeners, guint tap_flags)
 {
   frame_data fdata;
   gboolean create_proto_tree;
+  column_info *cinfo;
   epan_dissect_t *edt;
   gboolean passed;
 
@@ -2376,7 +2482,8 @@ process_packet(capture_file *cf, gint64 offset, const struct wtap_pkthdr *whdr,
     }
 
     passed = TRUE;
-    if (cf->rfcode || verbose || num_tap_filters!=0)
+    if (cf->rfcode || verbose || filtering_tap_listeners ||
+        (tap_flags & TL_REQUIRES_PROTO_TREE) || have_custom_cols(&cf->cinfo))
       create_proto_tree = TRUE;
     else
       create_proto_tree = FALSE;
@@ -2391,13 +2498,23 @@ process_packet(capture_file *cf, gint64 offset, const struct wtap_pkthdr *whdr,
     if (cf->rfcode)
       epan_dissect_prime_dfilter(edt, cf->rfcode);
 
+    col_custom_prime_edt(edt, &cf->cinfo);
+
     tap_queue_init(edt);
 
-    /* We only need the columns if we're printing packet info but we're
-       *not* verbose; in verbose mode, we print the protocol tree, not
-       the protocol summary. */
-    epan_dissect_run(edt, pseudo_header, pd, &fdata,
-                     (print_packet_info && !verbose) ? &cf->cinfo : NULL);
+    /* We only need the columns if either
+
+         1) some tap needs the columns
+
+       or
+
+         2) we're printing packet info but we're *not* verbose; in verbose
+            mode, we print the protocol tree, not the protocol summary. */
+    if ((tap_flags & TL_REQUIRES_COLUMNS) || (print_packet_info && !verbose))
+      cinfo = &cf->cinfo;
+    else
+      cinfo = NULL;
+    epan_dissect_run(edt, pseudo_header, pd, &fdata, cinfo);
 
     tap_push_tapped_queue(edt);
 
@@ -2516,7 +2633,6 @@ write_preamble(capture_file *cf)
 
   case WRITE_TEXT:
     return print_preamble(print_stream, cf ? cf->filename : NULL);
-    break;
 
   case WRITE_XML:
     if (verbose)
@@ -2583,7 +2699,7 @@ print_columns(capture_file *cf)
        * the same time, sort of like an "Update list of packets
        * in real time" capture in Wireshark.)
        */
-      if (capture_opts.iface != NULL)
+      if (global_capture_opts.iface != NULL)
         continue;
 #endif
       column_len = strlen(cf->cinfo.col_data[i]);
@@ -2827,7 +2943,7 @@ print_packet(capture_file *cf, epan_dissect_t *edt)
     }
   } else {
     /* Just fill in the columns. */
-    epan_dissect_fill_in_columns(edt);
+    epan_dissect_fill_in_columns(edt, TRUE);
 
     /* Now print them. */
     switch (output_action) {
@@ -2861,7 +2977,6 @@ write_finale(void)
 
   case WRITE_TEXT:
     return print_finale(print_stream);
-    break;
 
   case WRITE_XML:
     if (verbose)
@@ -3060,6 +3175,8 @@ cf_open(capture_file *cf, const char *fname, gboolean is_tempfile, int *err)
   nstime_set_unset(&prev_dis_ts);
   nstime_set_unset(&prev_cap_ts);
 
+  cf->state = FILE_READ_IN_PROGRESS;
+
   return CF_OK;
 
 fail:
@@ -3091,6 +3208,16 @@ read_failure_message(const char *filename, int err)
           filename, strerror(err));
 }
 
+/*
+ * Write errors are reported with an console message in TShark.
+ */
+static void
+write_failure_message(const char *filename, int err)
+{
+  cmdarg_err("An error occurred while writing to the file \"%s\": %s.",
+          filename, strerror(err));
+}
+
 /*
  * Report an error in command-line arguments.
  */
@@ -3100,9 +3227,7 @@ cmdarg_err(const char *fmt, ...)
   va_list ap;
 
   va_start(ap, fmt);
-  fprintf(stderr, "tshark: ");
-  vfprintf(stderr, fmt, ap);
-  fprintf(stderr, "\n");
+  failure_message(fmt, ap);
   va_end(ap);
 }
 
@@ -3119,5 +3244,3 @@ cmdarg_err_cont(const char *fmt, ...)
   fprintf(stderr, "\n");
   va_end(ap);
 }
-
-