As suggested by Anders: back out 37112.
[obnox/wireshark/wip.git] / tshark.c
index 2020ef01fb3ef7fe269097a61c206fdd52a1f804..001ad9fb303fbda4ec0332e04a065a5ea5db191c 100644 (file)
--- a/tshark.c
+++ b/tshark.c
@@ -187,44 +187,44 @@ static void write_failure_message(const char *filename, int err);
 capture_file cfile;
 
 struct string_elem {
-    const char *sstr;   /* The short string */
-    const char *lstr;   /* The long string */
+  const char *sstr;   /* The short string */
+  const char *lstr;   /* The long string */
 };
 
 static gint
 string_compare(gconstpointer a, gconstpointer b)
 {
-    return strcmp(((struct string_elem *)a)->sstr, 
-        ((struct string_elem *)b)->sstr);
-}    
+  return strcmp(((struct string_elem *)a)->sstr,
+                ((struct string_elem *)b)->sstr);
+}
 
 static void
 string_elem_print(gpointer data, gpointer not_used _U_)
-{    
-    fprintf(stderr, "    %s - %s\n",
-        ((struct string_elem *)data)->sstr,
-        ((struct string_elem *)data)->lstr);
+{
+  fprintf(stderr, "    %s - %s\n",
+          ((struct string_elem *)data)->sstr,
+          ((struct string_elem *)data)->lstr);
 }
 
 static void
 list_capture_types(void) {
-    int i;
-    struct string_elem *captypes;
-    GSList *list = NULL;
-
-    captypes = g_malloc(sizeof(struct string_elem) * WTAP_NUM_FILE_TYPES);
-
-    fprintf(stderr, "tshark: The available capture file types for the \"-F\" flag are:\n");
-    for (i = 0; i < WTAP_NUM_FILE_TYPES; i++) {
-        if (wtap_dump_can_open(i)) {
-            captypes[i].sstr = wtap_file_type_short_string(i);
-            captypes[i].lstr = wtap_file_type_string(i);
-            list = g_slist_insert_sorted(list, &captypes[i], string_compare);
-        }
+  int i;
+  struct string_elem *captypes;
+  GSList *list = NULL;
+
+  captypes = g_malloc(sizeof(struct string_elem) * WTAP_NUM_FILE_TYPES);
+
+  fprintf(stderr, "tshark: The available capture file types for the \"-F\" flag are:\n");
+  for (i = 0; i < WTAP_NUM_FILE_TYPES; i++) {
+    if (wtap_dump_can_open(i)) {
+      captypes[i].sstr = wtap_file_type_short_string(i);
+      captypes[i].lstr = wtap_file_type_string(i);
+      list = g_slist_insert_sorted(list, &captypes[i], string_compare);
     }
-    g_slist_foreach(list, string_elem_print, NULL);
-    g_slist_free(list);
-    g_free(captypes);
+  }
+  g_slist_foreach(list, string_elem_print, NULL);
+  g_slist_free(list);
+  g_free(captypes);
 }
 
 static void
@@ -297,6 +297,8 @@ print_usage(gboolean print_ver)
   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");
+  fprintf(output, "  -O <protocols>           Only show packet details of these protocols, comma\n");
+  fprintf(output, "                           separated\n");
   fprintf(output, "  -S                       display packets even when writing to a file\n");
   fprintf(output, "  -x                       add output of hex and ASCII dump (Packet Bytes)\n");
   fprintf(output, "  -T pdml|ps|psml|text|fields\n");
@@ -830,6 +832,7 @@ main(int argc, char *argv[])
   char                 badopt;
   GLogLevelFlags       log_flags;
   int                  optind_initial;
+  gchar               *output_only = NULL;
 
 #ifdef HAVE_LIBPCAP
 #if defined(_WIN32) || defined(HAVE_PCAP_CREATE)
@@ -847,7 +850,7 @@ main(int argc, char *argv[])
 #define OPTSTRING_I ""
 #endif
 
-#define OPTSTRING "a:b:" OPTSTRING_B "c:C:d:De:E:f:F:G:hH:i:" OPTSTRING_I "K:lLnN:o:pPqr:R:s:St:T:u:vVw:W:xX:y:z:"
+#define OPTSTRING "a:b:" OPTSTRING_B "c:C:d:De:E:f:F:G:hH:i:" OPTSTRING_I "K:lLnN:o:O:pPqr:R:s:St:T:u:vVw:W:xX:y:z:"
 
   static const char    optstring[] = OPTSTRING;
 
@@ -1080,288 +1083,289 @@ main(int argc, char *argv[])
   /* Now get our args */
   while ((opt = getopt(argc, argv, optstring)) != -1) {
     switch (opt) {
-      case 'a':        /* autostop criteria */
-      case 'b':        /* Ringbuffer option */
-      case 'c':        /* Capture x packets */
-      case 'f':        /* capture filter */
-      case 'i':        /* Use interface x */
-      case 'p':        /* Don't capture in promiscuous mode */
+    case 'a':        /* autostop criteria */
+    case 'b':        /* Ringbuffer option */
+    case 'c':        /* Capture x packets */
+    case 'f':        /* capture filter */
+    case 'i':        /* Use interface x */
+    case 'p':        /* Don't capture in promiscuous mode */
 #ifdef HAVE_PCAP_CREATE
-      case 'I':        /* Capture in monitor mode, if available */
+    case 'I':        /* Capture in monitor mode, if available */
 #endif
-      case 's':        /* Set the snapshot (capture) length */
-      case 'w':        /* Write to capture file x */
-      case 'y':        /* Set the pcap data link type */
+    case 's':        /* Set the snapshot (capture) length */
+    case 'w':        /* Write to capture file x */
+    case 'y':        /* Set the pcap data link type */
 #if defined(_WIN32) || defined(HAVE_PCAP_CREATE)
-      case 'B':        /* Buffer size */
+    case 'B':        /* Buffer size */
 #endif /* _WIN32 or HAVE_PCAP_CREATE */
 #ifdef HAVE_LIBPCAP
-        status = capture_opts_add_opt(&global_capture_opts, opt, optarg, &start_capture);
-        if(status != 0) {
-            return status;
-        }
+      status = capture_opts_add_opt(&global_capture_opts, opt, optarg, &start_capture);
+      if(status != 0) {
+        return status;
+      }
 #else
-        capture_option_specified = TRUE;
-        arg_error = TRUE;
+      capture_option_specified = TRUE;
+      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))
-          return 1;
-        break;
+      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))
+        return 1;
+      break;
 #if defined(HAVE_HEIMDAL_KERBEROS) || defined(HAVE_MIT_KERBEROS)
-      case 'K':        /* Kerberos keytab file */
-        read_keytab_file(optarg);
-        break;
+    case 'K':        /* Kerberos keytab file */
+      read_keytab_file(optarg);
+      break;
 #endif
-      case 'D':        /* Print a list of capture devices and exit */
+    case 'D':        /* Print a list of capture devices and exit */
 #ifdef HAVE_LIBPCAP
-        if_list = capture_interface_list(&err, &err_str);
-        if (if_list == NULL) {
-          switch (err) {
-          case CANT_GET_INTERFACE_LIST:
-            cmdarg_err("%s", err_str);
-            g_free(err_str);
-            break;
+      if_list = capture_interface_list(&err, &err_str);
+      if (if_list == NULL) {
+        switch (err) {
+        case CANT_GET_INTERFACE_LIST:
+          cmdarg_err("%s", err_str);
+          g_free(err_str);
+          break;
 
-          case NO_INTERFACES_FOUND:
-            cmdarg_err("There are no interfaces on which a capture can be done");
-            break;
-          }
-          return 2;
+        case NO_INTERFACES_FOUND:
+          cmdarg_err("There are no interfaces on which a capture can be done");
+          break;
         }
-        capture_opts_print_interfaces(if_list);
-        free_interface_list(if_list);
-        return 0;
+        return 2;
+      }
+      capture_opts_print_interfaces(if_list);
+      free_interface_list(if_list);
+      return 0;
 #else
-        capture_option_specified = TRUE;
-        arg_error = TRUE;
+      capture_option_specified = TRUE;
+      arg_error = TRUE;
 #endif
-        break;
-      case 'e':
-        /* Field entry */
-        output_fields_add(output_fields, optarg);
-        break;
-      case 'E':
-        /* Field option */
-        if(!output_fields_set_option(output_fields, optarg)) {
-          cmdarg_err("\"%s\" is not a valid field output option=value pair.", optarg);
-          output_fields_list_options(stderr);
-          return 1;
-        }
-        break;
-      case 'F':
-        out_file_type = wtap_short_string_to_file_type(optarg);
-        if (out_file_type < 0) {
-          cmdarg_err("\"%s\" isn't a valid capture file type", optarg);
-          list_capture_types();
-          return 1;
-        }
-        break;
-      case 'W':        /* Select extra information to save in our capture file */
-        /* This is patterned after the -N flag which may not be the best idea. */
-        if (strchr(optarg, 'n'))
-            out_file_name_res = TRUE;
-        break;
-      case 'H':        /* Read address to name mappings from a hosts file */
-        if (! read_hosts_file(optarg))
-        {
-          cmdarg_err("Can't read host entries from \"%s\"", optarg);
-          return 1;
-        }
+      break;
+    case 'e':
+      /* Field entry */
+      output_fields_add(output_fields, optarg);
+      break;
+    case 'E':
+      /* Field option */
+      if(!output_fields_set_option(output_fields, optarg)) {
+        cmdarg_err("\"%s\" is not a valid field output option=value pair.", optarg);
+        output_fields_list_options(stderr);
+        return 1;
+      }
+      break;
+    case 'F':
+      out_file_type = wtap_short_string_to_file_type(optarg);
+      if (out_file_type < 0) {
+        cmdarg_err("\"%s\" isn't a valid capture file type", optarg);
+        list_capture_types();
+        return 1;
+      }
+      break;
+    case 'W':        /* Select extra information to save in our capture file */
+      /* This is patterned after the -N flag which may not be the best idea. */
+      if (strchr(optarg, 'n'))
         out_file_name_res = TRUE;
-        break;
+      break;
+    case 'H':        /* Read address to name mappings from a hosts file */
+      if (! read_hosts_file(optarg))
+      {
+        cmdarg_err("Can't read host entries from \"%s\"", optarg);
+        return 1;
+      }
+      out_file_name_res = TRUE;
+      break;
 
-      case 'h':        /* Print help and exit */
-        print_usage(TRUE);
-        return 0;
-        break;
-      case 'l':        /* "Line-buffer" standard output */
-        /* This isn't line-buffering, strictly speaking, it's just
-           flushing the standard output after the information for
-           each packet is printed; however, that should be good
-           enough for all the purposes to which "-l" is put (and
-           is probably actually better for "-V", as it does fewer
-           writes).
-
-           See the comment in "process_packet()" for an explanation of
-           why we do that, and why we don't just use "setvbuf()" to
-           make the standard output line-buffered (short version: in
-           Windows, "line-buffered" is the same as "fully-buffered",
-           and the output buffer is only flushed when it fills up). */
-        line_buffered = TRUE;
-        break;
-      case 'L':        /* Print list of link-layer types and exit */
+    case 'h':        /* Print help and exit */
+      print_usage(TRUE);
+      return 0;
+      break;
+    case 'l':        /* "Line-buffer" standard output */
+      /* This isn't line-buffering, strictly speaking, it's just
+         flushing the standard output after the information for
+         each packet is printed; however, that should be good
+         enough for all the purposes to which "-l" is put (and
+         is probably actually better for "-V", as it does fewer
+         writes).
+
+         See the comment in "process_packet()" for an explanation of
+         why we do that, and why we don't just use "setvbuf()" to
+         make the standard output line-buffered (short version: in
+         Windows, "line-buffered" is the same as "fully-buffered",
+         and the output buffer is only flushed when it fills up). */
+      line_buffered = TRUE;
+      break;
+    case 'L':        /* Print list of link-layer types and exit */
 #ifdef HAVE_LIBPCAP
-        list_link_layer_types = TRUE;
+      list_link_layer_types = TRUE;
 #else
-        capture_option_specified = TRUE;
-        arg_error = TRUE;
-#endif
-        break;
-#if GLIB_CHECK_VERSION(2,10,0)
-      case 'P':        /* Perform two pass analysis */
-        perform_two_pass_analysis = TRUE;
-        break;
+      capture_option_specified = TRUE;
+      arg_error = TRUE;
 #endif
-      case 'n':        /* No name resolution */
+      break;
+    case 'P':        /* Perform two pass analysis */
+      perform_two_pass_analysis = TRUE;
+      break;
+    case 'n':        /* No name resolution */
+      gbl_resolv_flags = RESOLV_NONE;
+      break;
+    case 'N':        /* Select what types of addresses/port #s to resolve */
+      if (gbl_resolv_flags == RESOLV_ALL)
         gbl_resolv_flags = RESOLV_NONE;
-        break;
-      case 'N':        /* Select what types of addresses/port #s to resolve */
-        if (gbl_resolv_flags == RESOLV_ALL)
-          gbl_resolv_flags = RESOLV_NONE;
-        badopt = string_to_name_resolve(optarg, &gbl_resolv_flags);
-        if (badopt != '\0') {
-          cmdarg_err("-N specifies unknown resolving option '%c';",
-                     badopt);
-          cmdarg_err_cont( "           Valid options are 'm', 'n', 't', and 'C'");
-          return 1;
-        }
-        break;
-      case 'o':        /* Override preference from command line */
-        switch (prefs_set_pref(optarg)) {
-
-        case PREFS_SET_OK:
-          break;
-
-        case PREFS_SET_SYNTAX_ERR:
-          cmdarg_err("Invalid -o flag \"%s\"", optarg);
-          return 1;
-          break;
+      badopt = string_to_name_resolve(optarg, &gbl_resolv_flags);
+      if (badopt != '\0') {
+        cmdarg_err("-N specifies unknown resolving option '%c';",
+                   badopt);
+        cmdarg_err_cont( "           Valid options are 'm', 'n', 't', and 'C'");
+        return 1;
+      }
+      break;
+    case 'o':        /* Override preference from command line */
+      switch (prefs_set_pref(optarg)) {
 
-        case PREFS_SET_NO_SUCH_PREF:
-        case PREFS_SET_OBSOLETE:
-          cmdarg_err("-o flag \"%s\" specifies unknown preference", optarg);
-          return 1;
-          break;
-        }
+      case PREFS_SET_OK:
         break;
-      case 'q':        /* Quiet */
-        quiet = TRUE;
-        break;
-      case 'r':        /* Read capture file x */
-        cf_name = g_strdup(optarg);
-        break;
-      case 'R':        /* Read file filter */
-        rfilter = optarg;
-        break;
-      case 'S':        /* show packets in real time */
-        print_packet_info = TRUE;
-        break;
-      case 't':        /* Time stamp type */
-        if (strcmp(optarg, "r") == 0)
-          timestamp_set_type(TS_RELATIVE);
-        else if (strcmp(optarg, "a") == 0)
-          timestamp_set_type(TS_ABSOLUTE);
-        else if (strcmp(optarg, "ad") == 0)
-          timestamp_set_type(TS_ABSOLUTE_WITH_DATE);
-        else if (strcmp(optarg, "d") == 0)
-          timestamp_set_type(TS_DELTA);
-        else if (strcmp(optarg, "dd") == 0)
-          timestamp_set_type(TS_DELTA_DIS);
-        else if (strcmp(optarg, "e") == 0)
-          timestamp_set_type(TS_EPOCH);
-        else {
-          cmdarg_err("Invalid time stamp type \"%s\"",
-            optarg);
-          cmdarg_err_cont("It must be \"r\" for relative, \"a\" for absolute,");
-          cmdarg_err_cont("\"ad\" for absolute with date, or \"d\" for delta.");
-          return 1;
-        }
-        break;
-      case 'T':        /* printing Type */
-        if (strcmp(optarg, "text") == 0) {
-          output_action = WRITE_TEXT;
-          print_format = PR_FMT_TEXT;
-        } else if (strcmp(optarg, "ps") == 0) {
-          output_action = WRITE_TEXT;
-          print_format = PR_FMT_PS;
-        } else if (strcmp(optarg, "pdml") == 0) {
-          output_action = WRITE_XML;
-          verbose = TRUE;
-        } else if (strcmp(optarg, "psml") == 0) {
-          output_action = WRITE_XML;
-          verbose = FALSE;
-        } else if(strcmp(optarg, "fields") == 0) {
-          output_action = WRITE_FIELDS;
-          verbose = TRUE; /* Need full tree info */
-        } else {
-          cmdarg_err("Invalid -T parameter.");
-          cmdarg_err_cont("It must be \"ps\", \"text\", \"pdml\", \"psml\" or \"fields\".");
-          return 1;
-        }
-        break;
-      case 'u':        /* Seconds type */
-        if (strcmp(optarg, "s") == 0)
-          timestamp_set_seconds_type(TS_SECONDS_DEFAULT);
-        else if (strcmp(optarg, "hms") == 0)
-          timestamp_set_seconds_type(TS_SECONDS_HOUR_MIN_SEC);
-        else {
-          cmdarg_err("Invalid seconds type \"%s\"", optarg);
-          cmdarg_err_cont("It must be \"s\" for seconds or \"hms\" for hours, minutes and seconds.");
-          return 1;
-        }
+
+      case PREFS_SET_SYNTAX_ERR:
+        cmdarg_err("Invalid -o flag \"%s\"", optarg);
+        return 1;
         break;
-      case 'v':         /* Show version and exit */
-      {
-        GString             *comp_info_str;
-        GString             *runtime_info_str;
-        /* Assemble the compile-time version information string */
-        comp_info_str = g_string_new("Compiled ");
-        get_compiled_version_info(comp_info_str, NULL, epan_get_compiled_version_info);
-
-        /* Assemble the run-time version information string */
-        runtime_info_str = g_string_new("Running ");
-        get_runtime_version_info(runtime_info_str, NULL);
-        show_version(comp_info_str, runtime_info_str);
-        g_string_free(comp_info_str, TRUE);
-        g_string_free(runtime_info_str, TRUE);
-        return 0;
+
+      case PREFS_SET_NO_SUCH_PREF:
+      case PREFS_SET_OBSOLETE:
+        cmdarg_err("-o flag \"%s\" specifies unknown preference", optarg);
+        return 1;
         break;
       }
-      case 'V':        /* Verbose */
+      break;
+    case 'O':        /* Only output these protocols */
+      output_only = g_strdup(optarg);
+      break;
+    case 'q':        /* Quiet */
+      quiet = TRUE;
+      break;
+    case 'r':        /* Read capture file x */
+      cf_name = g_strdup(optarg);
+      break;
+    case 'R':        /* Read file filter */
+      rfilter = optarg;
+      break;
+    case 'S':        /* show packets in real time */
+      print_packet_info = TRUE;
+      break;
+    case 't':        /* Time stamp type */
+      if (strcmp(optarg, "r") == 0)
+        timestamp_set_type(TS_RELATIVE);
+      else if (strcmp(optarg, "a") == 0)
+        timestamp_set_type(TS_ABSOLUTE);
+      else if (strcmp(optarg, "ad") == 0)
+        timestamp_set_type(TS_ABSOLUTE_WITH_DATE);
+      else if (strcmp(optarg, "d") == 0)
+        timestamp_set_type(TS_DELTA);
+      else if (strcmp(optarg, "dd") == 0)
+        timestamp_set_type(TS_DELTA_DIS);
+      else if (strcmp(optarg, "e") == 0)
+        timestamp_set_type(TS_EPOCH);
+      else {
+        cmdarg_err("Invalid time stamp type \"%s\"",
+                   optarg);
+        cmdarg_err_cont("It must be \"r\" for relative, \"a\" for absolute,");
+        cmdarg_err_cont("\"ad\" for absolute with date, or \"d\" for delta.");
+        return 1;
+      }
+      break;
+    case 'T':        /* printing Type */
+      if (strcmp(optarg, "text") == 0) {
+        output_action = WRITE_TEXT;
+        print_format = PR_FMT_TEXT;
+      } else if (strcmp(optarg, "ps") == 0) {
+        output_action = WRITE_TEXT;
+        print_format = PR_FMT_PS;
+      } else if (strcmp(optarg, "pdml") == 0) {
+        output_action = WRITE_XML;
         verbose = TRUE;
-        /*  The user asked for a verbose output, so let's ensure they get it,
-         *  even if they're writing to a file.
-         */
-        print_packet_info = TRUE;
-        break;
-      case 'x':        /* Print packet data in hex (and ASCII) */
-        print_hex = TRUE;
-        /*  The user asked for hex output, so let's ensure they get it,
-         *  even if they're writing to a file.
-         */
-        print_packet_info = TRUE;
-        break;
-      case 'X':
-        break;
-      case 'z':
-        /* We won't call the init function for the stat this soon
-           as it would disallow MATE's fields (which are registered
-           by the preferences set callback) from being used as
-           part of a tap filter.  Instead, we just add the argument
-           to a list of stat arguments. */
-        if (!process_stat_cmd_arg(optarg)) {
-          cmdarg_err("invalid -z argument.");
-          cmdarg_err_cont("  -z argument must be one of :");
-          list_stat_cmd_args();
-          return 1;
-        }
-        break;
-      default:
-      case '?':        /* Bad flag - print usage message */
-        switch(optopt) {
-        case 'F':
-          list_capture_types();
-          break;
-        default:
-          print_usage(TRUE);
-        }
+      } else if (strcmp(optarg, "psml") == 0) {
+        output_action = WRITE_XML;
+        verbose = FALSE;
+      } else if(strcmp(optarg, "fields") == 0) {
+        output_action = WRITE_FIELDS;
+        verbose = TRUE; /* Need full tree info */
+      } else {
+        cmdarg_err("Invalid -T parameter.");
+        cmdarg_err_cont("It must be \"ps\", \"text\", \"pdml\", \"psml\" or \"fields\".");
         return 1;
+      }
+      break;
+    case 'u':        /* Seconds type */
+      if (strcmp(optarg, "s") == 0)
+        timestamp_set_seconds_type(TS_SECONDS_DEFAULT);
+      else if (strcmp(optarg, "hms") == 0)
+        timestamp_set_seconds_type(TS_SECONDS_HOUR_MIN_SEC);
+      else {
+        cmdarg_err("Invalid seconds type \"%s\"", optarg);
+        cmdarg_err_cont("It must be \"s\" for seconds or \"hms\" for hours, minutes and seconds.");
+        return 1;
+      }
+      break;
+    case 'v':         /* Show version and exit */
+    {
+      GString             *comp_info_str;
+      GString             *runtime_info_str;
+      /* Assemble the compile-time version information string */
+      comp_info_str = g_string_new("Compiled ");
+      get_compiled_version_info(comp_info_str, NULL, epan_get_compiled_version_info);
+
+      /* Assemble the run-time version information string */
+      runtime_info_str = g_string_new("Running ");
+      get_runtime_version_info(runtime_info_str, NULL);
+      show_version(comp_info_str, runtime_info_str);
+      g_string_free(comp_info_str, TRUE);
+      g_string_free(runtime_info_str, TRUE);
+      return 0;
+      break;
+    }
+    case 'V':        /* Verbose */
+      verbose = TRUE;
+      /*  The user asked for a verbose output, so let's ensure they get it,
+       *  even if they're writing to a file.
+       */
+      print_packet_info = TRUE;
+      break;
+    case 'x':        /* Print packet data in hex (and ASCII) */
+      print_hex = TRUE;
+      /*  The user asked for hex output, so let's ensure they get it,
+       *  even if they're writing to a file.
+       */
+      print_packet_info = TRUE;
+      break;
+    case 'X':
+      break;
+    case 'z':
+      /* We won't call the init function for the stat this soon
+         as it would disallow MATE's fields (which are registered
+         by the preferences set callback) from being used as
+         part of a tap filter.  Instead, we just add the argument
+         to a list of stat arguments. */
+      if (!process_stat_cmd_arg(optarg)) {
+        cmdarg_err("invalid -z argument.");
+        cmdarg_err_cont("  -z argument must be one of :");
+        list_stat_cmd_args();
+        return 1;
+      }
+      break;
+    default:
+    case '?':        /* Bad flag - print usage message */
+      switch(optopt) {
+      case 'F':
+        list_capture_types();
         break;
+      default:
+        print_usage(TRUE);
+      }
+      return 1;
+      break;
     }
   }
 
@@ -1458,6 +1462,20 @@ main(int argc, char *argv[])
     }
   }
 
+  if (output_only != NULL) {
+    char *ps;
+
+    if (!verbose) {
+      cmdarg_err("-O requires -V");
+      return 1;
+    }
+
+    output_only_tables = g_hash_table_new (g_str_hash, g_str_equal);
+    for (ps = strtok (output_only, ","); ps; ps = strtok (NULL, ",")) {
+      g_hash_table_insert(output_only_tables, (gpointer)ps, (gpointer)ps);
+    }
+  }
+
 #ifdef HAVE_LIBPCAP
   if (list_link_layer_types) {
     /* We're supposed to list the link-layer types for an interface;
@@ -1587,7 +1605,7 @@ main(int argc, char *argv[])
      if we're writing to a pipe. */
   if (global_capture_opts.saving_to_file &&
       global_capture_opts.output_to_pipe) {
-    if (have_tap_listeners()) {
+    if (tap_listeners_require_dissection()) {
       cmdarg_err("Taps aren't supported when saving to a pipe.");
       return 1;
     }
@@ -1662,8 +1680,8 @@ main(int argc, char *argv[])
 
         we're using a read filter on the packets;
 
-        we're using any taps. */
-  do_dissection = print_packet_info || rfcode || have_tap_listeners();
+        we're using any taps that need dissection. */
+  do_dissection = print_packet_info || rfcode || tap_listeners_require_dissection();
 
   if (cf_name) {
     /*
@@ -1762,7 +1780,7 @@ main(int argc, char *argv[])
             cmdarg_err("The capture device \"%s\" has no data link types.", global_capture_opts.iface);
             return 2;
         }
-        capture_opts_print_if_capabilities(caps,
+        capture_opts_print_if_capabilities(caps, global_capture_opts.iface,
                                            global_capture_opts.monitor_mode);
         free_if_capabilities(caps);
         return 0;
@@ -1818,10 +1836,10 @@ main(int argc, char *argv[])
 
   g_free(cf_name);
 
-#if GLIB_CHECK_VERSION(2,10,0)
-  if (cfile.plist_start != NULL)
-    g_slice_free_chain(frame_data, cfile.plist_start, next);
-#endif
+  if (cfile.frames != NULL) {
+    free_frame_data_sequence(cfile.frames);
+    cfile.frames = NULL;
+  }
 
   draw_tap_listeners(TRUE);
   funnel_dump_all_text_windows();
@@ -1848,13 +1866,13 @@ main(int argc, char *argv[])
 typedef gboolean (*pipe_input_cb_t) (gint source, gpointer user_data);
 
 typedef struct pipe_input_tag {
-    gint                source;
-    gpointer            user_data;
-    int                 *child_process;
-    pipe_input_cb_t     input_cb;
-    guint               pipe_input_id;
+  gint                source;
+  gpointer            user_data;
+  int                 *child_process;
+  pipe_input_cb_t     input_cb;
+  guint               pipe_input_id;
 #ifdef _WIN32
-    GStaticMutex        callback_running;
+  GStaticMutex        callback_running;
 #endif
 } pipe_input_t;
 
@@ -1870,11 +1888,11 @@ pipe_timer_cb(gpointer data)
   DWORD avail = 0;
   gboolean result, result1;
   DWORD childstatus;
-  pipe_input_t *pipe_input = data;
+  pipe_input_t *pipe_input_p = data;
   gint iterations = 0;
 
 
-  g_static_mutex_lock (&pipe_input->callback_running);
+  g_static_mutex_lock (&pipe_input_p->callback_running);
 
   /* try to read data from the pipe only 5 times, to avoid blocking */
   while(iterations < 5) {
@@ -1882,11 +1900,11 @@ pipe_timer_cb(gpointer data)
 
     /* Oddly enough although Named pipes don't work on win9x,
        PeekNamedPipe does !!! */
-    handle = (HANDLE) _get_osfhandle (pipe_input->source);
+    handle = (HANDLE) _get_osfhandle (pipe_input_p->source);
     result = PeekNamedPipe(handle, NULL, 0, NULL, &avail, NULL);
 
     /* Get the child process exit status */
-    result1 = GetExitCodeProcess((HANDLE)*(pipe_input->child_process),
+    result1 = GetExitCodeProcess((HANDLE)*(pipe_input_p->child_process),
                                  &childstatus);
 
     /* If the Peek returned an error, or there are bytes to be read
@@ -1897,10 +1915,10 @@ pipe_timer_cb(gpointer data)
       /*g_log(NULL, G_LOG_LEVEL_DEBUG, "pipe_timer_cb: data avail");*/
 
       /* And call the real handler */
-      if (!pipe_input->input_cb(pipe_input->source, pipe_input->user_data)) {
+      if (!pipe_input_p->input_cb(pipe_input_p->source, pipe_input_p->user_data)) {
         g_log(NULL, G_LOG_LEVEL_DEBUG, "pipe_timer_cb: input pipe closed, iterations: %u", iterations);
         /* pipe closed, return false so that the timer is stopped */
-        g_static_mutex_unlock (&pipe_input->callback_running);
+        g_static_mutex_unlock (&pipe_input_p->callback_running);
         return FALSE;
       }
     }
@@ -1915,7 +1933,7 @@ pipe_timer_cb(gpointer data)
 
   /*g_log(NULL, G_LOG_LEVEL_DEBUG, "pipe_timer_cb: finished with iterations: %u, new timer", iterations);*/
 
-  g_static_mutex_unlock (&pipe_input->callback_running);
+  g_static_mutex_unlock (&pipe_input_p->callback_running);
 
   /* we didn't stopped the timer, so let it run */
   return TRUE;
@@ -2324,9 +2342,9 @@ capture_input_closed(capture_options *capture_opts, gchar *msg)
 
   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)->user_saved == FALSE){
-               ws_unlink(((capture_file *) capture_opts->cf)->filename);
-       }
+    if(((capture_file *) capture_opts->cf)->user_saved == FALSE) {
+      ws_unlink(((capture_file *) capture_opts->cf)->filename);
+    }
   }
 #ifdef USE_BROKEN_G_MAIN_LOOP
   /*g_main_loop_quit(loop);*/
@@ -2387,25 +2405,26 @@ capture_cleanup(int signum _U_)
 #endif /* _WIN32 */
 #endif /* HAVE_LIBPCAP */
 
-#if GLIB_CHECK_VERSION(2,10,0)
 static gboolean
 process_packet_first_pass(capture_file *cf,
                gint64 offset, const struct wtap_pkthdr *whdr,
                union wtap_pseudo_header *pseudo_header, const guchar *pd)
 {
-  frame_data *fdata = g_slice_new(frame_data);
+  frame_data fdlocal;
+  guint32 framenum;
   epan_dissect_t edt;
   gboolean passed;
 
-  /* Count this packet. */
-  cf->count++;
+  /* The frame number of this packet is one more than the count of
+     frames in this packet. */
+  framenum = cf->count + 1;
 
   /* If we're not running a display filter and we're not printing any
      packet information, we don't need to do a dissection. This means
      that all packets can be marked as 'passed'. */
   passed = TRUE;
 
-  frame_data_init(fdata, cf->count, whdr, offset, cum_bytes);
+  frame_data_init(&fdlocal, framenum, whdr, offset, cum_bytes);
 
   /* If we're going to print packet information, or we're going to
      run a read filter, or we're going to process taps, set up to
@@ -2426,10 +2445,10 @@ process_packet_first_pass(capture_file *cf,
     if (cf->rfcode)
       epan_dissect_prime_dfilter(&edt, cf->rfcode);
 
-    frame_data_set_before_dissect(fdata, &cf->elapsed_time,
+    frame_data_set_before_dissect(&fdlocal, &cf->elapsed_time,
                                   &first_ts, &prev_dis_ts, &prev_cap_ts);
 
-    epan_dissect_run(&edt, pseudo_header, pd, fdata, NULL);
+    epan_dissect_run(&edt, pseudo_header, pd, &fdlocal, NULL);
 
     /* Run the read filter if we have one. */
     if (cf->rfcode)
@@ -2437,11 +2456,10 @@ process_packet_first_pass(capture_file *cf,
   }
 
   if (passed) {
-    frame_data_set_after_dissect(fdata, &cum_bytes, &prev_dis_ts);
-    cap_file_add_fdata(cf, fdata);
+    frame_data_set_after_dissect(&fdlocal, &cum_bytes, &prev_dis_ts);
+    frame_data_sequence_add(cf->frames, &fdlocal);
+    cf->count++;
   }
-  else
-    g_slice_free(frame_data, fdata);
 
   if (do_dissection)
     epan_dissect_cleanup(&edt);
@@ -2560,7 +2578,6 @@ process_packet_second_pass(capture_file *cf, frame_data *fdata,
   }
   return passed;
 }
-#endif
 
 static int
 load_cap_file(capture_file *cf, char *save_file, int out_file_type,
@@ -2646,10 +2663,13 @@ load_cap_file(capture_file *cf, char *save_file, int out_file_type,
   tap_flags = union_of_tap_listener_flags();
 
   if (perform_two_pass_analysis) {
-#if GLIB_CHECK_VERSION(2,10,0)
+    guint32 framenum;
     frame_data *fdata;
     int old_max_packet_count = max_packet_count;
 
+    /* Allocate a frame_data_sequence for all the frames. */
+    cf->frames = new_frame_data_sequence();
+
     while (wtap_read(cf->wth, &err, &err_info, &data_offset)) {
       if (process_packet_first_pass(cf, data_offset, wtap_phdr(cf->wth),
                          wtap_pseudoheader(cf->wth), wtap_buf_ptr(cf->wth))) {
@@ -2674,7 +2694,8 @@ load_cap_file(capture_file *cf, char *save_file, int out_file_type,
 
     max_packet_count = old_max_packet_count;
 
-    for (fdata = cf->plist_start; err == 0 && fdata != NULL; fdata = fdata->next) {
+    for (framenum = 1; err == 0 && framenum <= cf->count; framenum++) {
+      fdata = frame_data_sequence_find(cf->frames, framenum);
       if (wtap_seek_read(cf->wth, fdata->file_off, &cf->pseudo_header,
           cf->pd, fdata->cap_len, &err, &err_info)) {
         if (process_packet_second_pass(cf, fdata,
@@ -2705,7 +2726,6 @@ load_cap_file(capture_file *cf, char *save_file, int out_file_type,
         }
       }
     }
-#endif
   }
   else {
     while (wtap_read(cf->wth, &err, &err_info, &data_offset)) {
@@ -2766,29 +2786,34 @@ load_cap_file(capture_file *cf, char *save_file, int out_file_type,
     switch (err) {
 
     case WTAP_ERR_UNSUPPORTED_ENCAP:
-      cmdarg_err("\"%s\" has a packet with a network type that TShark doesn't support.\n(%s)",
+      cmdarg_err("The file \"%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:
-      cmdarg_err("An attempt to read from \"%s\" failed for some unknown reason.",
+      cmdarg_err("An attempt to read from the file \"%s\" failed for some unknown reason.",
                  cf->filename);
       break;
 
     case WTAP_ERR_SHORT_READ:
-      cmdarg_err("\"%s\" appears to have been cut short in the middle of a packet.",
+      cmdarg_err("The file \"%s\" appears to have been cut short in the middle of a packet.",
                  cf->filename);
       break;
 
     case WTAP_ERR_BAD_RECORD:
-      cmdarg_err("\"%s\" appears to be damaged or corrupt.\n(%s)",
+      cmdarg_err("The file \"%s\" appears to be damaged or corrupt.\n(%s)",
                  cf->filename, err_info);
       g_free(err_info);
       break;
 
+    case WTAP_ERR_DECOMPRESS:
+      cmdarg_err("The compressed file \"%s\" appears to be damaged or corrupt.\n"
+                 "(%s)", cf->filename, err_info);
+      break;
+
     default:
-      cmdarg_err("An error occurred while reading \"%s\": %s.",
+      cmdarg_err("An error occurred while reading the file \"%s\": %s.",
                  cf->filename, wtap_strerror(err));
       break;
     }
@@ -3025,7 +3050,7 @@ print_columns(capture_file *cf)
       if (column_len < 3)
         column_len = 3;
       line_bufp = get_line_buf(buf_offset + column_len);
-      sprintf(line_bufp + buf_offset, "%3s", cf->cinfo.col_data[i]);
+      g_snprintf(line_bufp + buf_offset, (int)column_len + 1, "%3s", cf->cinfo.col_data[i]);
       break;
 
     case COL_CLS_TIME:
@@ -3036,7 +3061,7 @@ print_columns(capture_file *cf)
       if (column_len < 10)
         column_len = 10;
       line_bufp = get_line_buf(buf_offset + column_len);
-      sprintf(line_bufp + buf_offset, "%10s", cf->cinfo.col_data[i]);
+      g_snprintf(line_bufp + buf_offset, (int)column_len + 1, "%10s", cf->cinfo.col_data[i]);
       break;
 
     case COL_DEF_SRC:
@@ -3052,7 +3077,7 @@ print_columns(capture_file *cf)
       if (column_len < 12)
         column_len = 12;
       line_bufp = get_line_buf(buf_offset + column_len);
-      sprintf(line_bufp + buf_offset, "%12s", cf->cinfo.col_data[i]);
+      g_snprintf(line_bufp + buf_offset, (int)column_len + 1, "%12s", cf->cinfo.col_data[i]);
       break;
 
     case COL_DEF_DST:
@@ -3068,13 +3093,13 @@ print_columns(capture_file *cf)
       if (column_len < 12)
         column_len = 12;
       line_bufp = get_line_buf(buf_offset + column_len);
-      sprintf(line_bufp + buf_offset, "%-12s", cf->cinfo.col_data[i]);
+      g_snprintf(line_bufp + buf_offset, (int)column_len + 1, "%-12s", cf->cinfo.col_data[i]);
       break;
 
     default:
       column_len = strlen(cf->cinfo.col_data[i]);
       line_bufp = get_line_buf(buf_offset + column_len);
-      strcat(line_bufp + buf_offset, cf->cinfo.col_data[i]);
+      g_strlcat(line_bufp + buf_offset, cf->cinfo.col_data[i], column_len + 1);
       break;
     }
     buf_offset += column_len;
@@ -3085,9 +3110,9 @@ print_columns(capture_file *cf)
        *
        * If we printed a network source and are printing a
        * network destination of the same type next, separate
-       * them with "->"; if we printed a network destination
+       * them with " -> "; if we printed a network destination
        * and are printing a network source of the same type
-       * next, separate them with "<-"; otherwise separate them
+       * next, separate them with " <- "; otherwise separate them
        * with a space.
        *
        * We add enough space to the buffer for " <- " or " -> ",
@@ -3104,12 +3129,12 @@ print_columns(capture_file *cf)
         case COL_DEF_DST:
         case COL_RES_DST:
         case COL_UNRES_DST:
-          strcat(line_bufp + buf_offset, " -> ");
+          g_strlcat(line_bufp + buf_offset, " -> ", 5);
           buf_offset += 4;
           break;
 
         default:
-          strcat(line_bufp + buf_offset, " ");
+          g_strlcat(line_bufp + buf_offset, " ", 5);
           buf_offset += 1;
           break;
         }
@@ -3123,12 +3148,12 @@ print_columns(capture_file *cf)
         case COL_DEF_DL_DST:
         case COL_RES_DL_DST:
         case COL_UNRES_DL_DST:
-          strcat(line_bufp + buf_offset, " -> ");
+          g_strlcat(line_bufp + buf_offset, " -> ", 5);
           buf_offset += 4;
           break;
 
         default:
-          strcat(line_bufp + buf_offset, " ");
+          g_strlcat(line_bufp + buf_offset, " ", 5);
           buf_offset += 1;
           break;
         }
@@ -3142,12 +3167,12 @@ print_columns(capture_file *cf)
         case COL_DEF_NET_DST:
         case COL_RES_NET_DST:
         case COL_UNRES_NET_DST:
-          strcat(line_bufp + buf_offset, " -> ");
+          g_strlcat(line_bufp + buf_offset, " -> ", 5);
           buf_offset += 4;
           break;
 
         default:
-          strcat(line_bufp + buf_offset, " ");
+          g_strlcat(line_bufp + buf_offset, " ", 5);
           buf_offset += 1;
           break;
         }
@@ -3161,12 +3186,12 @@ print_columns(capture_file *cf)
         case COL_DEF_SRC:
         case COL_RES_SRC:
         case COL_UNRES_SRC:
-          strcat(line_bufp + buf_offset, " <- ");
+          g_strlcat(line_bufp + buf_offset, " <- ", 5);
           buf_offset += 4;
           break;
 
         default:
-          strcat(line_bufp + buf_offset, " ");
+          g_strlcat(line_bufp + buf_offset, " ", 5);
           buf_offset += 1;
           break;
         }
@@ -3180,12 +3205,12 @@ print_columns(capture_file *cf)
         case COL_DEF_DL_SRC:
         case COL_RES_DL_SRC:
         case COL_UNRES_DL_SRC:
-          strcat(line_bufp + buf_offset, " <- ");
+          g_strlcat(line_bufp + buf_offset, " <- ", 5);
           buf_offset += 4;
           break;
 
         default:
-          strcat(line_bufp + buf_offset, " ");
+          g_strlcat(line_bufp + buf_offset, " ", 5);
           buf_offset += 1;
           break;
         }
@@ -3199,19 +3224,19 @@ print_columns(capture_file *cf)
         case COL_DEF_NET_SRC:
         case COL_RES_NET_SRC:
         case COL_UNRES_NET_SRC:
-          strcat(line_bufp + buf_offset, " <- ");
+          g_strlcat(line_bufp + buf_offset, " <- ", 5);
           buf_offset += 4;
           break;
 
         default:
-          strcat(line_bufp + buf_offset, " ");
+          g_strlcat(line_bufp + buf_offset, " ", 5);
           buf_offset += 1;
           break;
         }
         break;
 
       default:
-        strcat(line_bufp + buf_offset, " ");
+        g_strlcat(line_bufp + buf_offset, " ", 5);
         buf_offset += 1;
         break;
       }
@@ -3532,6 +3557,15 @@ cf_open_error_message(int err, gchar *err_info, gboolean for_writing,
       errmsg = "A full header couldn't be written to the file \"%s\".";
       break;
 
+    case WTAP_ERR_DECOMPRESS:
+      /* Seen only when opening a capture file for reading. */
+      g_snprintf(errmsg_errno, sizeof(errmsg_errno),
+                 "The compressed file \"%%s\" appears to be damaged or corrupt.\n"
+                 "(%s)", err_info);
+      g_free(err_info);
+      errmsg = errmsg_errno;
+      break;
+
     default:
       g_snprintf(errmsg_errno, sizeof(errmsg_errno),
                  "The file \"%%s\" could not be %s: %s.",