Comment out a lot more hf[] entries and related.
[metze/wireshark/wip.git] / capinfos.c
index 5adc480618a9b82c76f884983d77b5450d06ac92..c3de1f74161660f675453b275b956c4f2159c92a 100644 (file)
 
 #include <glib.h>
 
-#include <epan/packet.h>
-#include <epan/filesystem.h>
-#include <epan/plugins.h>
-#include <epan/report_err.h>
+#include <wsutil/privileges.h>
+#include <wsutil/filesystem.h>
+
+#ifdef HAVE_PLUGINS
+#include <wsutil/plugins.h>
+#endif
+
 #include "wtap.h"
+#include <wsutil/report_err.h>
 #include <wsutil/privileges.h>
 #include <wsutil/str_util.h>
 
@@ -114,11 +118,11 @@ static gboolean continue_after_wtap_open_offline_failure = TRUE;
  * table report variables
  */
 
-static gboolean long_report = TRUE;         /* By default generate long report       */
-static gchar table_report_header = TRUE;    /* Generate column header by default     */
-static gchar field_separator = '\t';        /* Use TAB as field separator by default */
-static gchar quote_char = '\0';             /* Do NOT quote fields by default        */
-static gboolean machine_readable = FALSE;   /* Display machine-readable numbers      */
+static gboolean long_report        = TRUE;  /* By default generate long report       */
+static gchar table_report_header   = TRUE;  /* Generate column header by default     */
+static gchar field_separator       = '\t';  /* Use TAB as field separator by default */
+static gchar quote_char            = '\0';  /* Do NOT quote fields by default        */
+static gboolean machine_readable   = FALSE; /* Display machine-readable numbers      */
 
 /*
  * capinfos has the ability to report on a number of
@@ -131,37 +135,38 @@ static gboolean machine_readable = FALSE;   /* Display machine-readable numbers
  * individual options.
  */
 
-static gboolean report_all_infos = TRUE;    /* Report all infos           */
+static gboolean report_all_infos   = TRUE;  /* Report all infos           */
 
-static gboolean cap_file_type = TRUE;       /* Report capture type        */
-static gboolean cap_file_encap = TRUE;      /* Report encapsulation       */
-static gboolean cap_snaplen = TRUE;         /* Packet size limit (snaplen)*/
-static gboolean cap_packet_count = TRUE;    /* Report packet count        */
-static gboolean cap_file_size = TRUE;       /* Report file size           */
+static gboolean cap_file_type      = TRUE;  /* Report capture type        */
+static gboolean cap_file_encap     = TRUE;  /* Report encapsulation       */
+static gboolean cap_snaplen        = TRUE;  /* Packet size limit (snaplen)*/
+static gboolean cap_packet_count   = TRUE;  /* Report packet count        */
+static gboolean cap_file_size      = TRUE;  /* Report file size           */
+static gboolean cap_comment        = TRUE;  /* Display the capture comment */
 
-static gboolean cap_data_size = TRUE;       /* Report packet byte size    */
-static gboolean cap_duration = TRUE;        /* Report capture duration    */
-static gboolean cap_start_time = TRUE;      /* Report capture start time  */
-static gboolean cap_end_time = TRUE;        /* Report capture end time    */
-static gboolean time_as_secs = FALSE;       /* Report time values as raw seconds */
+static gboolean cap_data_size      = TRUE;  /* Report packet byte size    */
+static gboolean cap_duration       = TRUE;  /* Report capture duration    */
+static gboolean cap_start_time     = TRUE;  /* Report capture start time  */
+static gboolean cap_end_time       = TRUE;  /* Report capture end time    */
+static gboolean time_as_secs       = FALSE; /* Report time values as raw seconds */
 
 static gboolean cap_data_rate_byte = TRUE;  /* Report data rate bytes/sec */
-static gboolean cap_data_rate_bit = TRUE;   /* Report data rate bites/sec */
-static gboolean cap_packet_size = TRUE;     /* Report average packet size */
-static gboolean cap_packet_rate = TRUE;     /* Report average packet rate */
-static gboolean cap_order = TRUE;           /* Report if packets are in chronological order (True/False) */
+static gboolean cap_data_rate_bit  = TRUE;  /* Report data rate bites/sec */
+static gboolean cap_packet_size    = TRUE;  /* Report average packet size */
+static gboolean cap_packet_rate    = TRUE;  /* Report average packet rate */
+static gboolean cap_order          = TRUE;  /* Report if packets are in chronological order (True/False) */
 
 #ifdef HAVE_LIBGCRYPT
-static gboolean cap_file_hashes = TRUE;     /* Calculate file hashes */
+static gboolean cap_file_hashes    = TRUE;  /* Calculate file hashes */
 #endif
 
 #ifdef USE_GOPTION
-static gboolean cap_help = FALSE;
+static gboolean cap_help     = FALSE;
 static gboolean table_report = FALSE;
 
 static GOptionEntry general_entries[] =
 {
-/* General */
+  /* General */
   { "type", 't', 0, G_OPTION_ARG_NONE, &cap_file_type,
     "display the capture file type", NULL },
   { "Encapsulation", 'E', 0, G_OPTION_ARG_NONE, &cap_file_encap,
@@ -170,11 +175,13 @@ static GOptionEntry general_entries[] =
   { "Hash", 'H', 0, G_OPTION_ARG_NONE, &cap_file_hashes,
     "display the SHA1, RMD160, and MD5 hashes of the file", NULL },
 #endif /* HAVE_LIBGCRYPT */
- { NULL,'\0',0,G_OPTION_ARG_NONE,NULL,NULL,NULL }
+  { "capture-comment", 'k', 0, G_OPTION_ARG_NONE, &cap_comment,
+    "display the capture comment ", NULL },
+  { NULL,'\0',0,G_OPTION_ARG_NONE,NULL,NULL,NULL }
 };
 static GOptionEntry size_entries[] =
 {
-/* Size */
+  /* Size */
   { "packets", 'c', 0, G_OPTION_ARG_NONE, &cap_packet_count,
     "display the number of packets", NULL },
   { "size", 's', 0, G_OPTION_ARG_NONE, &cap_file_size,
@@ -187,7 +194,7 @@ static GOptionEntry size_entries[] =
 };
 static GOptionEntry time_entries[] =
 {
-/* Time */
+  /* Time */
   { "duration", 'u', 0, G_OPTION_ARG_NONE, &cap_duration,
     "display the capture duration (in seconds)", NULL },
   { "start", 'a', 0, G_OPTION_ARG_NONE, &cap_start_time,
@@ -203,7 +210,7 @@ static GOptionEntry time_entries[] =
 
 static GOptionEntry stats_entries[] =
 {
-/* Statistics */
+  /* Statistics */
   { "bytes", 'y', 0, G_OPTION_ARG_NONE, &cap_data_rate_byte,
     "display average data rate (in bytes/s)", NULL },
   { "bits", 'i', 0, G_OPTION_ARG_NONE, &cap_data_rate_bit,
@@ -217,7 +224,7 @@ static GOptionEntry stats_entries[] =
 
 static GOptionEntry output_format_entries[] =
 {
-/* Output format */
+  /* Output format */
   { "long", 'L', 0, G_OPTION_ARG_NONE, &long_report,
     "generate long report (default)", NULL },
   { "Table", 'T', 0, G_OPTION_ARG_NONE, &table_report,
@@ -229,7 +236,7 @@ static GOptionEntry output_format_entries[] =
 
 static GOptionEntry table_report_entries[] =
 {
-/* Table report */
+  /* Table report */
   { "header-rec", 'R', 0, G_OPTION_ARG_NONE, &table_report_header,
     "generate header record (default)", NULL },
   { "no-table", 'r', 0, G_OPTION_ARG_NONE, &table_report_header,
@@ -241,7 +248,7 @@ static GOptionEntry misc_entries[] =
 {
   { "helpcompat", 'h', 0, G_OPTION_ARG_NONE, &cap_help,
     "display help", NULL },
- { NULL,'\0',0,G_OPTION_ARG_NONE,NULL,NULL,NULL }
 { NULL,'\0',0,G_OPTION_ARG_NONE,NULL,NULL,NULL }
 };
 
 GOptionContext *ctx;
@@ -251,9 +258,9 @@ GError *parse_err = NULL;
 #endif /* USE_GOPTION */
 
 #ifdef HAVE_LIBGCRYPT
-#define HASH_SIZE_SHA1 20
+#define HASH_SIZE_SHA1   20
 #define HASH_SIZE_RMD160 20
-#define HASH_SIZE_MD5 16
+#define HASH_SIZE_MD5    16
 
 #define HASH_STR_SIZE (41) /* Max hash size * 2 + '\0' */
 #define HASH_BUF_SIZE (1024 * 1024)
@@ -286,58 +293,60 @@ typedef enum {
 
 typedef struct _capture_info {
   const char    *filename;
-  guint16       file_type;
-  gboolean      iscompressed;
-  int           file_encap;
-  gint64        filesize;
-
-  guint64       packet_bytes;
-  gboolean      times_known;
-  double        start_time;
-  double        stop_time;
-  guint32       packet_count;
-  gboolean      snap_set;                /* If set in capture file header      */
-  guint32       snaplen;                 /* value from the capture file header */
-  guint32       snaplen_min_inferred;    /* If caplen < len for 1 or more rcds */
-  guint32       snaplen_max_inferred;    /*  ...                               */
-  gboolean      drops_known;
-  guint32       drop_count;
-
-  double        duration;
-  double        packet_rate;
-  double        packet_size;
-  double        data_rate;              /* in bytes */
-  gboolean      know_order;
-  order_t       order;
-
-  int          *encap_counts;           /* array of per_packet encap counts; array has one entry per wtap_encap type */
+  guint16        file_type;
+  gboolean       iscompressed;
+  int            file_encap;
+  gint64         filesize;
+  gchar         *comment;
+
+  guint64        packet_bytes;
+  gboolean       times_known;
+  double         start_time;
+  double         stop_time;
+  guint32        packet_count;
+  gboolean       snap_set;                /* If set in capture file header      */
+  guint32        snaplen;                 /* value from the capture file header */
+  guint32        snaplen_min_inferred;    /* If caplen < len for 1 or more rcds */
+  guint32        snaplen_max_inferred;    /*  ...                               */
+  gboolean       drops_known;
+  guint32        drop_count;
+
+  double         duration;
+  double         packet_rate;
+  double         packet_size;
+  double         data_rate;              /* in bytes */
+  gboolean       know_order;
+  order_t        order;
+
+  int           *encap_counts;           /* array of per_packet encap counts; array has one entry per wtap_encap type */
 } capture_info;
 
 
 static void
 enable_all_infos(void)
 {
-  report_all_infos = TRUE;
+  report_all_infos   = TRUE;
 
-  cap_file_type = TRUE;
-  cap_file_encap = TRUE;
-  cap_snaplen = TRUE;
-  cap_packet_count = TRUE;
-  cap_file_size = TRUE;
+  cap_file_type      = TRUE;
+  cap_file_encap     = TRUE;
+  cap_snaplen        = TRUE;
+  cap_packet_count   = TRUE;
+  cap_file_size      = TRUE;
+  cap_comment        = TRUE;
 
-  cap_data_size = TRUE;
-  cap_duration = TRUE;
-  cap_start_time = TRUE;
-  cap_end_time = TRUE;
-  cap_order = TRUE;
+  cap_data_size      = TRUE;
+  cap_duration       = TRUE;
+  cap_start_time     = TRUE;
+  cap_end_time       = TRUE;
+  cap_order          = TRUE;
 
   cap_data_rate_byte = TRUE;
-  cap_data_rate_bit = TRUE;
-  cap_packet_size = TRUE;
-  cap_packet_rate = TRUE;
+  cap_data_rate_bit  = TRUE;
+  cap_packet_size    = TRUE;
+  cap_packet_rate    = TRUE;
 
 #ifdef HAVE_LIBGCRYPT
-  cap_file_hashes = TRUE;
+  cap_file_hashes    = TRUE;
 #endif /* HAVE_LIBGCRYPT */
 }
 
@@ -351,6 +360,7 @@ disable_all_infos(void)
   cap_snaplen        = FALSE;
   cap_packet_count   = FALSE;
   cap_file_size      = FALSE;
+  cap_comment        = FALSE;
 
   cap_data_size      = FALSE;
   cap_duration       = FALSE;
@@ -364,7 +374,7 @@ disable_all_infos(void)
   cap_packet_rate    = FALSE;
 
 #ifdef HAVE_LIBGCRYPT
-  cap_file_hashes = FALSE;
+  cap_file_hashes    = FALSE;
 #endif /* HAVE_LIBGCRYPT */
 }
 
@@ -373,26 +383,26 @@ order_string(order_t order)
 {
   switch (order) {
 
-  case IN_ORDER:
-    return "True";
+    case IN_ORDER:
+      return "True";
 
-  case NOT_IN_ORDER:
-    return "False";
+    case NOT_IN_ORDER:
+      return "False";
 
-  case ORDER_UNKNOWN:
-    return "Unknown";
+    case ORDER_UNKNOWN:
+      return "Unknown";
 
-  default:
-    return "???";  /* "cannot happen" (the next step is "Profit!") */
+    default:
+      return "???";  /* "cannot happen" (the next step is "Profit!") */
   }
 }
 
 static gchar *
 time_string(time_t timer, capture_info *cf_info, gboolean want_lf)
 {
-  const gchar *lf = want_lf ? "\n" : "";
-  static gchar time_string_buf[20];
-  char *time_string_ctime;
+  const gchar  *lf = want_lf ? "\n" : "";
+  static gchar  time_string_buf[20];
+  char         *time_string_ctime;
 
   if (cf_info->times_known && cf_info->packet_count > 0) {
     if (time_as_secs) {
@@ -400,17 +410,9 @@ time_string(time_t timer, capture_info *cf_info, gboolean want_lf)
       g_snprintf(time_string_buf, 20, "%lu%s", (unsigned long)timer, lf);
       return time_string_buf;
     } else {
-#if (defined _WIN32) && (_MSC_VER < 1500)
-      /* calling localtime(), and thus ctime(), on MSVC 2005 with huge values causes it to crash */
-      /* XXX - find the exact value that still does work */
-      /* XXX - using _USE_32BIT_TIME_T might be another way to circumvent this problem */
-      if (timer > 2000000000) {
-        time_string_ctime = NULL;
-      } else
-#endif
       time_string_ctime = ctime(&timer);
       if (time_string_ctime == NULL) {
-       g_snprintf(time_string_buf, 20, "Not representable%s", lf);
+        g_snprintf(time_string_buf, 20, "Not representable%s", lf);
         return time_string_buf;
       }
       if (!want_lf) {
@@ -429,12 +431,6 @@ time_string(time_t timer, capture_info *cf_info, gboolean want_lf)
   return time_string_buf;
 }
 
-static double
-secs_nsecs(const struct wtap_nstime * nstime)
-{
-  return (nstime->nsecs / 1000000000.0) + (double)nstime->secs;
-}
-
 static void print_value(const gchar *text_p1, gint width, const gchar *text_p2, double value) {
   if (value > 0.0)
     printf("%s%.*f%s\n", text_p1, width, value, text_p2);
@@ -446,20 +442,20 @@ static void
 print_stats(const gchar *filename, capture_info *cf_info)
 {
   const gchar           *file_type_string, *file_encap_string;
-  time_t                start_time_t;
-  time_t                stop_time_t;
+  time_t                 start_time_t;
+  time_t                 stop_time_t;
   gchar                 *size_string;
 
   /* Build printable strings for various stats */
-  file_type_string = wtap_file_type_string(cf_info->file_type);
+  file_type_string = wtap_file_type_subtype_string(cf_info->file_type);
   file_encap_string = wtap_encap_string(cf_info->file_encap);
   start_time_t = (time_t)cf_info->start_time;
   stop_time_t = (time_t)cf_info->stop_time;
 
   if (filename)           printf     ("File name:           %s\n", filename);
   if (cap_file_type)      printf     ("File type:           %s%s\n",
-                                      file_type_string,
-                                      cf_info->iscompressed ? " (gzip compressed)" : "");
+      file_type_string,
+      cf_info->iscompressed ? " (gzip compressed)" : "");
   if (cap_file_encap)     printf     ("File encapsulation:  %s\n", file_encap_string);
   if (cap_file_encap && (cf_info->file_encap == WTAP_ENCAP_PER_PACKET)) {
     int i;
@@ -469,15 +465,15 @@ print_stats(const gchar *filename, capture_info *cf_info)
     }
   }
   if (cap_snaplen && cf_info->snap_set)
-                          printf     ("Packet size limit:   file hdr: %u bytes\n", cf_info->snaplen);
-  else if(cap_snaplen && !cf_info->snap_set)
-                          printf     ("Packet size limit:   file hdr: (not set)\n");
+    printf     ("Packet size limit:   file hdr: %u bytes\n", cf_info->snaplen);
+  else if (cap_snaplen && !cf_info->snap_set)
+    printf     ("Packet size limit:   file hdr: (not set)\n");
   if (cf_info->snaplen_max_inferred > 0) {
     if (cf_info->snaplen_min_inferred == cf_info->snaplen_max_inferred)
-                          printf     ("Packet size limit:   inferred: %u bytes\n", cf_info->snaplen_min_inferred);
+      printf     ("Packet size limit:   inferred: %u bytes\n", cf_info->snaplen_min_inferred);
     else
-                          printf     ("Packet size limit:   inferred: %u bytes - %u bytes (range)\n",
-                                      cf_info->snaplen_min_inferred, cf_info->snaplen_max_inferred);
+      printf     ("Packet size limit:   inferred: %u bytes - %u bytes (range)\n",
+          cf_info->snaplen_min_inferred, cf_info->snaplen_max_inferred);
   }
   if (cap_packet_count) {
     printf     ("Number of packets:   ");
@@ -519,11 +515,11 @@ print_stats(const gchar *filename, capture_info *cf_info)
     if (cap_data_rate_byte) {
                           printf     ("Data byte rate:      ");
       if (machine_readable) {
-       print_value("", 2, " bytes/sec",   cf_info->data_rate);
+        print_value("", 2, " bytes/sec",   cf_info->data_rate);
       } else {
-       size_string = format_size((gint64)cf_info->data_rate, format_size_unit_bytes_s);
-       printf ("%s\n", size_string);
-       g_free(size_string);
+        size_string = format_size((gint64)cf_info->data_rate, format_size_unit_bytes_s);
+        printf ("%s\n", size_string);
+        g_free(size_string);
       }
     }
     if (cap_data_rate_bit) {
@@ -531,9 +527,9 @@ print_stats(const gchar *filename, capture_info *cf_info)
       if (machine_readable) {
         print_value("", 2, " bits/sec",    cf_info->data_rate*8);
       } else {
-       size_string = format_size((gint64)(cf_info->data_rate*8), format_size_unit_bits_s);
-       printf ("%s\n", size_string);
-       g_free(size_string);
+        size_string = format_size((gint64)(cf_info->data_rate*8), format_size_unit_bits_s);
+        printf ("%s\n", size_string);
+        g_free(size_string);
       }
     }
   }
@@ -542,22 +538,24 @@ print_stats(const gchar *filename, capture_info *cf_info)
     if (cap_packet_rate) {
                           printf     ("Average packet rate: ");
       if (machine_readable) {
-       print_value("", 2, " packets/sec", cf_info->packet_rate);
+        print_value("", 2, " packets/sec", cf_info->packet_rate);
       } else {
-       size_string = format_size((gint64)cf_info->packet_rate, format_size_unit_none);
-       printf ("%spackets/sec\n", size_string);
-       g_free(size_string);
+        size_string = format_size((gint64)cf_info->packet_rate, format_size_unit_none);
+        printf ("%spackets/sec\n", size_string);
+        g_free(size_string);
       }
     }
   }
 #ifdef HAVE_LIBGCRYPT
   if (cap_file_hashes) {
-                          printf     ("SHA1:                %s\n", file_sha1);
-                          printf     ("RIPEMD160:           %s\n", file_rmd160);
-                          printf     ("MD5:                 %s\n", file_md5);
+    printf     ("SHA1:                %s\n", file_sha1);
+    printf     ("RIPEMD160:           %s\n", file_rmd160);
+    printf     ("MD5:                 %s\n", file_md5);
   }
 #endif /* HAVE_LIBGCRYPT */
   if (cap_order)          printf     ("Strict time order:   %s\n", order_string(cf_info->order));
+  if (cap_comment && cf_info->comment)
+    printf     ("Capture comment:     %s\n", cf_info->comment);
 }
 
 static void
@@ -605,12 +603,13 @@ print_stats_table_header(void)
   if (cap_packet_rate)    print_stats_table_header_label("Average packet rate (packets/sec)");
 #ifdef HAVE_LIBGCRYPT
   if (cap_file_hashes) {
-                          print_stats_table_header_label("SHA1");
-                          print_stats_table_header_label("RIPEMD160");
-                          print_stats_table_header_label("MD5");
+    print_stats_table_header_label("SHA1");
+    print_stats_table_header_label("RIPEMD160");
+    print_stats_table_header_label("MD5");
   }
 #endif /* HAVE_LIBGCRYPT */
   if (cap_order)          print_stats_table_header_label("Strict time order");
+  if (cap_comment)        print_stats_table_header_label("Capture comment");
 
   printf("\n");
 }
@@ -619,11 +618,11 @@ static void
 print_stats_table(const gchar *filename, capture_info *cf_info)
 {
   const gchar           *file_type_string, *file_encap_string;
-  time_t                start_time_t;
-  time_t                stop_time_t;
+  time_t                 start_time_t;
+  time_t                 stop_time_t;
 
   /* Build printable strings for various stats */
-  file_type_string = wtap_file_type_string(cf_info->file_type);
+  file_type_string = wtap_file_type_subtype_string(cf_info->file_type);
   file_encap_string = wtap_encap_string(cf_info->file_encap);
   start_time_t = (time_t)cf_info->start_time;
   stop_time_t = (time_t)cf_info->stop_time;
@@ -656,7 +655,7 @@ print_stats_table(const gchar *filename, capture_info *cf_info)
   if (cap_snaplen) {
     putsep();
     putquote();
-    if(cf_info->snap_set)
+    if (cf_info->snap_set)
       printf("%u", cf_info->snaplen);
     else
       printf("(not set)");
@@ -791,14 +790,23 @@ print_stats_table(const gchar *filename, capture_info *cf_info)
     putquote();
   }
 
+  if (cap_comment) {
+    putsep();
+    putquote();
+    printf("%s", cf_info->comment);
+    putquote();
+  }
+
+
   printf("\n");
 }
 
 static int
 process_cap_file(wtap *wth, const char *filename)
 {
+  int                   status = 0;
   int                   err;
-  gchar                 *err_info;
+  gchar                *err_info;
   gint64                size;
   gint64                data_offset;
 
@@ -808,13 +816,16 @@ process_cap_file(wtap *wth, const char *filename)
   guint32               snaplen_max_inferred =          0;
   const struct wtap_pkthdr *phdr;
   capture_info          cf_info;
-  gboolean             have_times = TRUE;
+  gboolean              have_times = TRUE;
   double                start_time = 0;
   double                stop_time  = 0;
   double                cur_time   = 0;
-  double               prev_time = 0;
-  gboolean             know_order = FALSE;
-  order_t              order = IN_ORDER;
+  double                prev_time = 0;
+  gboolean              know_order = FALSE;
+  order_t               order = IN_ORDER;
+  wtapng_section_t     *shb_inf;
+  gchar                *p;
+
 
   cf_info.encap_counts = g_new0(int,WTAP_NUM_ENCAP_TYPES);
 
@@ -823,11 +834,11 @@ process_cap_file(wtap *wth, const char *filename)
     phdr = wtap_phdr(wth);
     if (phdr->presence_flags & WTAP_HAS_TS) {
       prev_time = cur_time;
-      cur_time = secs_nsecs(&phdr->ts);
-      if(packet==0) {
+      cur_time = nstime_to_sec(&phdr->ts);
+      if (packet == 0) {
         start_time = cur_time;
-        stop_time = cur_time;
-        prev_time = cur_time;
+        stop_time  = cur_time;
+        prev_time  = cur_time;
       }
       if (cur_time < prev_time) {
         order = NOT_IN_ORDER;
@@ -860,39 +871,47 @@ process_cap_file(wtap *wth, const char *filename)
 
     /* Per-packet encapsulation */
     if (wtap_file_encap(wth) == WTAP_ENCAP_PER_PACKET) {
-        if ((phdr->pkt_encap > 0) && (phdr->pkt_encap < WTAP_NUM_ENCAP_TYPES)) {
-            cf_info.encap_counts[phdr->pkt_encap] += 1;
-        } else {
-            fprintf(stderr, "capinfos: Unknown per-packet encapsulation: %d [frame number: %d]\n", phdr->pkt_encap, packet);
-        }
+      if ((phdr->pkt_encap > 0) && (phdr->pkt_encap < WTAP_NUM_ENCAP_TYPES)) {
+        cf_info.encap_counts[phdr->pkt_encap] += 1;
+      } else {
+        fprintf(stderr, "capinfos: Unknown per-packet encapsulation: %d [frame number: %d]\n", phdr->pkt_encap, packet);
+      }
     }
 
   } /* while */
 
   if (err != 0) {
     fprintf(stderr,
-            "capinfos: An error occurred after reading %u packets from \"%s\": %s.\n",
-            packet, filename, wtap_strerror(err));
+        "capinfos: An error occurred after reading %u packets from \"%s\": %s.\n",
+        packet, filename, wtap_strerror(err));
     switch (err) {
 
-    case WTAP_ERR_UNSUPPORTED:
-    case WTAP_ERR_UNSUPPORTED_ENCAP:
-    case WTAP_ERR_BAD_FILE:
-    case WTAP_ERR_DECOMPRESS:
-      fprintf(stderr, "(%s)\n", err_info);
-      g_free(err_info);
-      break;
+      case WTAP_ERR_SHORT_READ:
+        status = 1;
+        fprintf(stderr,
+          "  (will continue anyway, checksums might be incorrect)\n");
+        break;
+
+      case WTAP_ERR_UNSUPPORTED:
+      case WTAP_ERR_UNSUPPORTED_ENCAP:
+      case WTAP_ERR_BAD_FILE:
+      case WTAP_ERR_DECOMPRESS:
+        fprintf(stderr, "(%s)\n", err_info);
+        g_free(err_info);
+        /* fallthrough */
+
+      default:
+        g_free(cf_info.encap_counts);
+        return 1;
     }
-    g_free(cf_info.encap_counts);
-    return 1;
   }
 
   /* File size */
   size = wtap_file_size(wth, &err);
   if (size == -1) {
     fprintf(stderr,
-            "capinfos: Can't get size of \"%s\": %s.\n",
-            filename, g_strerror(err));
+        "capinfos: Can't get size of \"%s\": %s.\n",
+        filename, g_strerror(err));
     g_free(cf_info.encap_counts);
     return 1;
   }
@@ -900,7 +919,7 @@ process_cap_file(wtap *wth, const char *filename)
   cf_info.filesize = size;
 
   /* File Type */
-  cf_info.file_type = wtap_file_type(wth);
+  cf_info.file_type = wtap_file_type_subtype(wth);
   cf_info.iscompressed = wtap_iscompressed(wth);
 
   /* File Encapsulation */
@@ -908,7 +927,7 @@ process_cap_file(wtap *wth, const char *filename)
 
   /* Packet size limit (snaplen) */
   cf_info.snaplen = wtap_snapshot_length(wth);
-  if(cf_info.snaplen > 0)
+  if (cf_info.snaplen > 0)
     cf_info.snap_set = TRUE;
   else
     cf_info.snap_set = FALSE;
@@ -942,15 +961,34 @@ process_cap_file(wtap *wth, const char *filename)
     cf_info.packet_size = (double)bytes / packet;                  /* Avg packet size      */
   }
 
-  if(long_report) {
+  cf_info.comment = NULL;
+  shb_inf = wtap_file_get_shb_info(wth);
+  if (shb_inf) {
+    /* opt_comment is always 0-terminated by pcapng_read_section_header_block */
+    cf_info.comment = g_strdup(shb_inf->opt_comment);
+  }
+  g_free(shb_inf);
+  if (cf_info.comment) {
+    /* multi-line comments would conflict with the formatting that capinfos uses
+       we replace linefeeds with spaces */
+    p = cf_info.comment;
+    while (*p != '\0') {
+      if (*p == '\n')
+        *p = ' ';
+      p++;
+    }
+  }
+
+  if (long_report) {
     print_stats(filename, &cf_info);
   } else {
     print_stats_table(filename, &cf_info);
   }
 
   g_free(cf_info.encap_counts);
+  g_free(cf_info.comment);
 
-  return 0;
+  return status;
 }
 
 static void
@@ -968,9 +1006,9 @@ usage(gboolean is_error)
 
   fprintf(output, "Capinfos %s"
 #ifdef SVNVERSION
-          " (" SVNVERSION " from " SVNPATH ")"
+      " (" SVNVERSION " from " SVNPATH ")"
 #endif
-          "\n", VERSION);
+      "\n", VERSION);
   fprintf(output, "Prints various information (infos) about capture files.\n");
   fprintf(output, "See http://www.wireshark.org for more information.\n");
   fprintf(output, "\n");
@@ -982,6 +1020,7 @@ usage(gboolean is_error)
 #ifdef HAVE_LIBGCRYPT
   fprintf(output, "  -H display the SHA1, RMD160, and MD5 hashes of the file\n");
 #endif
+  fprintf(output, "  -k display the capture comment\n");
   fprintf(output, "\n");
   fprintf(output, "Size infos:\n");
   fprintf(output, "  -c display the number of packets\n");
@@ -1024,7 +1063,7 @@ usage(gboolean is_error)
   fprintf(output, "  -C cancel processing if file open fails (default is to continue)\n");
   fprintf(output, "  -A generate all infos (default)\n");
   fprintf(output, "\n");
-  fprintf(output, "Options are processed from left to right order with later options superceeding\n");
+  fprintf(output, "Options are processed from left to right order with later options superceding\n");
   fprintf(output, "or adding to earlier options.\n");
   fprintf(output, "\n");
   fprintf(output, "If no options are given the default is to display all infos in long report\n");
@@ -1089,14 +1128,21 @@ main(int argc, char *argv[])
   init_process_policies();
 
 #ifdef HAVE_PLUGINS
-  /* Register wiretap plugins */
-
   if ((init_progfile_dir_error = init_progfile_dir(argv[0], main))) {
     g_warning("capinfos: init_progfile_dir(): %s", init_progfile_dir_error);
     g_free(init_progfile_dir_error);
   } else {
-    init_report_err(failure_message,NULL,NULL,NULL);
-    init_plugins();
+    /* Register all the plugin types we have. */
+    wtap_register_plugin_types(); /* Types known to libwiretap */
+
+    init_report_err(failure_message, NULL, NULL, NULL);
+
+    /* Scan for plugins.  This does *not* call their registration routines;
+       that's done later. */
+    scan_plugins();
+
+    /* Register all libwiretap plugin modules. */
+    register_all_wiretap_modules();
   }
 #endif
 
@@ -1104,17 +1150,17 @@ main(int argc, char *argv[])
 #ifdef USE_GOPTION
   ctx = g_option_context_new(" <infile> ... - print information about capture file(s)");
   general_grp = g_option_group_new("gen", "General infos:",
-                               "Show general options", NULL, NULL);
+      "Show general options", NULL, NULL);
   size_grp = g_option_group_new("size", "Size infos:",
-                              "Show size options", NULL, NULL);
+      "Show size options", NULL, NULL);
   time_grp = g_option_group_new("time", "Time infos:",
-                              "Show time options", NULL, NULL);
+      "Show time options", NULL, NULL);
   stats_grp = g_option_group_new("stats", "Statistics infos:",
-                                "Show statistics options", NULL, NULL);
+      "Show statistics options", NULL, NULL);
   output_grp = g_option_group_new("output", "Output format:",
-                                "Show output format options", NULL, NULL);
+      "Show output format options", NULL, NULL);
   table_report_grp = g_option_group_new("table", "Table report options:",
-                                "Show table report options", NULL, NULL);
+      "Show table report options", NULL, NULL);
   g_option_group_add_entries(general_grp, general_entries);
   g_option_group_add_entries(size_grp, size_entries);
   g_option_group_add_entries(time_grp, time_entries);
@@ -1135,166 +1181,180 @@ main(int argc, char *argv[])
      displayed is not exactly the same as the command the user used
      ran.
    */
-  argv[0] = "capinfos";
-  ;
-  if( !g_option_context_parse(ctx, &argc, &argv, &parse_err) ) {
-    if(parse_err) g_print ("option parsing failed: %s\n", parse_err->message);
-    g_print("%s", g_option_context_get_help (ctx, TRUE, NULL));
+  argv[0] = (char *)"capinfos";
+
+  /* if we have at least one cmdline option, we disable printing all infos */
+  if (argc > 2 && report_all_infos)
+    disable_all_infos();
+
+  if ( !g_option_context_parse(ctx, &argc, &argv, &parse_err) ) {
+    if (parse_err)
+      g_printerr ("option parsing failed: %s\n", parse_err->message);
+    g_printerr("%s", g_option_context_get_help (ctx, TRUE, NULL));
     exit(1);
   }
-  if( cap_help || (argc < 2) ) {
+  if ( cap_help ) {
     g_print("%s", g_option_context_get_help (ctx, FALSE, NULL));
     exit(0);
   }
+  if ( argc < 2 ) {
+    g_printerr("%s", g_option_context_get_help (ctx, FALSE, NULL));
+    exit(1);
+  }
   g_option_context_free(ctx);
 
 #endif /* USE_GOPTION */
-  while ((opt = getopt(argc, argv, "tEcs" FILE_HASH_OPT "dluaeyizvhxoCALTMRrSNqQBmb")) !=-1) {
+  while ((opt = getopt(argc, argv, "tEcs" FILE_HASH_OPT "dluaeyizvhxokCALTMRrSNqQBmb")) !=-1) {
 
     switch (opt) {
 
-    case 't':
-      if (report_all_infos) disable_all_infos();
-      cap_file_type = TRUE;
-      break;
-
-    case 'E':
-      if (report_all_infos) disable_all_infos();
-      cap_file_encap = TRUE;
-      break;
-
-    case 'l':
-      if (report_all_infos) disable_all_infos();
-      cap_snaplen = TRUE;
-      break;
-
-    case 'c':
-      if (report_all_infos) disable_all_infos();
-      cap_packet_count = TRUE;
-      break;
-
-    case 's':
-      if (report_all_infos) disable_all_infos();
-      cap_file_size = TRUE;
-      break;
-
-    case 'd':
-      if (report_all_infos) disable_all_infos();
-      cap_data_size = TRUE;
-      break;
-
-    case 'u':
-      if (report_all_infos) disable_all_infos();
-      cap_duration = TRUE;
-      break;
-
-    case 'a':
-      if (report_all_infos) disable_all_infos();
-      cap_start_time = TRUE;
-      break;
-
-    case 'e':
-      if (report_all_infos) disable_all_infos();
-      cap_end_time = TRUE;
-      break;
-
-    case 'S':
-      time_as_secs = TRUE;
-      break;
-
-    case 'y':
-      if (report_all_infos) disable_all_infos();
-      cap_data_rate_byte = TRUE;
-      break;
-
-    case 'i':
-      if (report_all_infos) disable_all_infos();
-      cap_data_rate_bit = TRUE;
-      break;
-
-    case 'z':
-      if (report_all_infos) disable_all_infos();
-      cap_packet_size = TRUE;
-      break;
-
-    case 'x':
-      if (report_all_infos) disable_all_infos();
-      cap_packet_rate = TRUE;
-      break;
+      case 't':
+        if (report_all_infos) disable_all_infos();
+        cap_file_type = TRUE;
+        break;
+
+      case 'E':
+        if (report_all_infos) disable_all_infos();
+        cap_file_encap = TRUE;
+        break;
+
+      case 'l':
+        if (report_all_infos) disable_all_infos();
+        cap_snaplen = TRUE;
+        break;
+
+      case 'c':
+        if (report_all_infos) disable_all_infos();
+        cap_packet_count = TRUE;
+        break;
+
+      case 's':
+        if (report_all_infos) disable_all_infos();
+        cap_file_size = TRUE;
+        break;
+
+      case 'd':
+        if (report_all_infos) disable_all_infos();
+        cap_data_size = TRUE;
+        break;
+
+      case 'u':
+        if (report_all_infos) disable_all_infos();
+        cap_duration = TRUE;
+        break;
+
+      case 'a':
+        if (report_all_infos) disable_all_infos();
+        cap_start_time = TRUE;
+        break;
+
+      case 'e':
+        if (report_all_infos) disable_all_infos();
+        cap_end_time = TRUE;
+        break;
+
+      case 'S':
+        time_as_secs = TRUE;
+        break;
+
+      case 'y':
+        if (report_all_infos) disable_all_infos();
+        cap_data_rate_byte = TRUE;
+        break;
+
+      case 'i':
+        if (report_all_infos) disable_all_infos();
+        cap_data_rate_bit = TRUE;
+        break;
+
+      case 'z':
+        if (report_all_infos) disable_all_infos();
+        cap_packet_size = TRUE;
+        break;
+
+      case 'x':
+        if (report_all_infos) disable_all_infos();
+        cap_packet_rate = TRUE;
+        break;
 
 #ifdef HAVE_LIBGCRYPT
-    case 'H':
-      if (report_all_infos) disable_all_infos();
-      cap_file_hashes = TRUE;
-      break;
+      case 'H':
+        if (report_all_infos) disable_all_infos();
+        cap_file_hashes = TRUE;
+        break;
 #endif
 
-    case 'o':
-      if (report_all_infos) disable_all_infos();
-      cap_order = TRUE;
-      break;
+      case 'o':
+        if (report_all_infos) disable_all_infos();
+        cap_order = TRUE;
+        break;
+
+      case 'k':
+        if (report_all_infos) disable_all_infos();
+        cap_comment = TRUE;
+        break;
 
-    case 'C':
-      continue_after_wtap_open_offline_failure = FALSE;
-      break;
+      case 'C':
+        continue_after_wtap_open_offline_failure = FALSE;
+        break;
 
-    case 'A':
-      enable_all_infos();
-      break;
+      case 'A':
+        enable_all_infos();
+        break;
 
-    case 'L':
-      long_report = TRUE;
-      break;
+      case 'L':
+        long_report = TRUE;
+        break;
 
-    case 'T':
-      long_report = FALSE;
-      break;
+      case 'T':
+        long_report = FALSE;
+        break;
 
-    case 'M':
-      machine_readable = TRUE;
-      break;
+      case 'M':
+        machine_readable = TRUE;
+        break;
 
-    case 'R':
-      table_report_header = TRUE;
-      break;
+      case 'R':
+        table_report_header = TRUE;
+        break;
 
-    case 'r':
-      table_report_header = FALSE;
-      break;
+      case 'r':
+        table_report_header = FALSE;
+        break;
 
-    case 'N':
-      quote_char = '\0';
-      break;
+      case 'N':
+        quote_char = '\0';
+        break;
 
-    case 'q':
-      quote_char = '\'';
-      break;
+      case 'q':
+        quote_char = '\'';
+        break;
 
-    case 'Q':
-      quote_char = '"';
-      break;
+      case 'Q':
+        quote_char = '"';
+        break;
 
-    case 'B':
-      field_separator = '\t';
-      break;
+      case 'B':
+        field_separator = '\t';
+        break;
 
-    case 'm':
-      field_separator = ',';
-      break;
+      case 'm':
+        field_separator = ',';
+        break;
 
-    case 'b':
-      field_separator = ' ';
-      break;
+      case 'b':
+        field_separator = ' ';
+        break;
 
-    case 'h':
-      usage(FALSE);
-      exit(1);
-      break;
+      case 'h':
+        usage(FALSE);
+        exit(0);
+        break;
 
-    case '?':              /* Bad flag - print usage message */
-      usage(TRUE);
-      exit(1);
-      break;
+      case '?':              /* Bad flag - print usage message */
+        usage(TRUE);
+        exit(1);
+        break;
     }
   }
 
@@ -1306,7 +1366,7 @@ main(int argc, char *argv[])
     exit(1);
   }
 
-  if(!long_report && table_report_header) {
+  if (!long_report && table_report_header) {
     print_stats_table_header();
   }
 
@@ -1351,22 +1411,22 @@ main(int argc, char *argv[])
 
     if (!wth) {
       fprintf(stderr, "capinfos: Can't open %s: %s\n", argv[opt],
-        wtap_strerror(err));
+          wtap_strerror(err));
       switch (err) {
 
-      case WTAP_ERR_UNSUPPORTED:
-      case WTAP_ERR_UNSUPPORTED_ENCAP:
-      case WTAP_ERR_BAD_FILE:
-        fprintf(stderr, "(%s)\n", err_info);
-        g_free(err_info);
-        break;
+        case WTAP_ERR_UNSUPPORTED:
+        case WTAP_ERR_UNSUPPORTED_ENCAP:
+        case WTAP_ERR_BAD_FILE:
+          fprintf(stderr, "(%s)\n", err_info);
+          g_free(err_info);
+          break;
       }
       overall_error_status = 1; /* remember that an error has occurred */
-      if(!continue_after_wtap_open_offline_failure)
+      if (!continue_after_wtap_open_offline_failure)
         exit(1); /* error status */
     }
 
-    if(wth) {
+    if (wth) {
       if ((opt > optind) && (long_report))
         printf("\n");
       status = process_cap_file(wth, argv[opt]);
@@ -1379,3 +1439,16 @@ main(int argc, char *argv[])
 
   return overall_error_status;
 }
+
+/*
+ * Editor modelines  -  http://www.wireshark.org/tools/modelines.html
+ *
+ * Local variables:
+ * c-basic-offset: 2
+ * tab-width: 8
+ * indent-tabs-mode: nil
+ * End:
+ *
+ * vi: set shiftwidth=2 tabstop=8 expandtab:
+ * :indentSize=2:tabSize=8:noTabs=true:
+ */