Store pointers to previously displayed and captured packet, not nstime_t deltas.
[metze/wireshark/wip.git] / epan / column-utils.c
index d9ea4e39b1d4b63e67412ff288146f8e821fcdd5..7737ec99dc13fc8900604768a994e62c409b57f8 100644 (file)
  *
  * You should have received a copy of the GNU General Public License
  * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
  */
 
-#ifdef HAVE_CONFIG_H
-# include "config.h"
-#endif
+#include "config.h"
 
 #include <string.h>
 #include <time.h>
@@ -48,7 +46,7 @@
 /* Allocate all the data structures for constructing column data, given
    the number of columns. */
 void
-col_setup(column_info *cinfo, gint num_cols)
+col_setup(column_info *cinfo, const gint num_cols)
 {
   int i;
 
@@ -59,6 +57,7 @@ col_setup(column_info *cinfo, gint num_cols)
   cinfo->col_last   = g_new(int, NUM_COL_FMTS);
   cinfo->col_title  = g_new(gchar*, num_cols);
   cinfo->col_custom_field = g_new(gchar*, num_cols);
+  cinfo->col_custom_occurrence = g_new(gint, num_cols);
   cinfo->col_custom_field_id = g_new(int, num_cols);
   cinfo->col_custom_dfilter = g_new(dfilter_t*, num_cols);
   cinfo->col_data   = (const gchar **)g_new(gchar*, num_cols);
@@ -101,7 +100,7 @@ col_get_writable(column_info *cinfo)
 }
 
 void
-col_set_writable(column_info *cinfo, gboolean writable)
+col_set_writable(column_info *cinfo, const gboolean writable)
 {
     if (cinfo)
         cinfo->writable = writable;
@@ -115,14 +114,14 @@ col_set_writable(column_info *cinfo, gboolean writable)
     ((cinfo)->col_first[el] >= 0))
 
 gint
-check_col(column_info *cinfo, gint el)
+check_col(column_info *cinfo, const gint el)
 {
   return CHECK_COL(cinfo, el);
 }
 
 /* Sets the fence for a column to be at the end of the column. */
 void
-col_set_fence(column_info *cinfo, gint el)
+col_set_fence(column_info *cinfo, const gint el)
 {
   int i;
 
@@ -143,7 +142,7 @@ col_set_fence(column_info *cinfo, gint el)
    later append to it, as the later append will cause a string
    copy to be done. */
 void
-col_clear(column_info *cinfo, gint el)
+col_clear(column_info *cinfo, const gint el)
 {
   int    i;
   int    fence;
@@ -170,7 +169,7 @@ col_clear(column_info *cinfo, gint el)
        *      we don't do anything.
        */
       fence = cinfo->col_fence[i];
-      if (fence == 0 || cinfo->col_buf[i] == cinfo->col_data[i]) {
+      if (cinfo->col_buf[i] == cinfo->col_data[i] || fence == 0) {
         /*
          * The fence isn't at the end of the column, or the column wasn't
          * last set with "col_set_str()", so clear the column out.
@@ -192,88 +191,11 @@ col_clear(column_info *cinfo, gint el)
     cinfo->col_data[i] = cinfo->col_buf[i];         \
   }
 
-#define COL_CHECK_REF_TIME(fd, buf) \
+#define COL_CHECK_REF_TIME(fd, buf)         \
   if(fd->flags.ref_time){                   \
     g_strlcpy(buf, "*REF*", COL_MAX_LEN );  \
-    return 0;                           \
-  }
-
-/* Use this if "str" points to something that will stay around (and thus
-   needn't be copied). */
-void
-col_set_str(column_info *cinfo, gint el, const gchar* str)
-{
-  int i;
-  int fence;
-  size_t max_len;
-
-  if (!CHECK_COL(cinfo, el))
-    return;
-
-  if (el == COL_INFO)
-    max_len = COL_MAX_INFO_LEN;
-  else
-    max_len = COL_MAX_LEN;
-
-  for (i = cinfo->col_first[el]; i <= cinfo->col_last[el]; i++) {
-    if (cinfo->fmt_matx[i][el]) {
-      fence = cinfo->col_fence[i];
-      if (fence != 0) {
-        /*
-         * We will append the string after the fence.
-         * First arrange that we can append, if necessary.
-         */
-        COL_CHECK_APPEND(cinfo, i, max_len);
-
-        g_strlcpy(&cinfo->col_buf[i][fence], str, max_len - fence);
-      } else {
-        /*
-         * There's no fence, so we can just set the column to point
-         * to the string.
-         */
-        cinfo->col_data[i] = str;
-      }
-    }
+    return;                                 \
   }
-}
-
-/* Adds a vararg list to a packet info string. */
-void
-col_add_fstr(column_info *cinfo, gint el, const gchar *format, ...) {
-  va_list ap;
-  int     i;
-  int     fence;
-  int     max_len;
-
-  if (!CHECK_COL(cinfo, el))
-    return;
-
-  if (el == COL_INFO)
-    max_len = COL_MAX_INFO_LEN;
-  else
-    max_len = COL_MAX_LEN;
-
-  va_start(ap, format);
-  for (i = cinfo->col_first[el]; i <= cinfo->col_last[el]; i++) {
-    if (cinfo->fmt_matx[i][el]) {
-      fence = cinfo->col_fence[i];
-      if (fence != 0) {
-        /*
-         * We will append the string after the fence.
-         * First arrange that we can append, if necessary.
-         */
-        COL_CHECK_APPEND(cinfo, i, max_len);
-      } else {
-        /*
-         * There's no fence, so we can just write to the string.
-         */
-        cinfo->col_data[i] = cinfo->col_buf[i];
-      }
-      g_vsnprintf(&cinfo->col_buf[i][fence], max_len - fence, format, ap);
-    }
-  }
-  va_end(ap);
-}
 
 /* The same as CHECK_COL(), but without the check to see if the column is writable. */
 #define HAVE_CUSTOM_COLS(cinfo) ((cinfo) && (cinfo)->col_first[COL_CUSTOM] >= 0)
@@ -294,10 +216,12 @@ void col_custom_set_edt(epan_dissect_t *edt, column_info *cinfo)
 
   for (i = cinfo->col_first[COL_CUSTOM];
        i <= cinfo->col_last[COL_CUSTOM]; i++) {
-    if (cinfo->fmt_matx[i][COL_CUSTOM] && cinfo->col_custom_field[i]) {
-      cinfo->col_data[i] = cinfo->col_buf[i];
-
+    if (cinfo->fmt_matx[i][COL_CUSTOM] &&
+        cinfo->col_custom_field[i] &&
+        cinfo->col_custom_field_id[i] != -1) {
+       cinfo->col_data[i] = cinfo->col_buf[i];
        cinfo->col_expr.col_expr[i] = epan_custom_set(edt, cinfo->col_custom_field_id[i],
+                                     cinfo->col_custom_occurrence[i],
                                      cinfo->col_buf[i],
                                      cinfo->col_expr.col_expr_val[i],
                                      COL_MAX_LEN);
@@ -318,45 +242,36 @@ col_custom_prime_edt(epan_dissect_t *edt, column_info *cinfo)
 
     cinfo->col_custom_field_id[i] = -1;
     if (cinfo->fmt_matx[i][COL_CUSTOM] &&
-               cinfo->col_custom_dfilter[i]){
+        cinfo->col_custom_dfilter[i]){
         epan_dissect_prime_dfilter(edt, cinfo->col_custom_dfilter[i]);
         if (cinfo->col_custom_field) {
             header_field_info* hfinfo = proto_registrar_get_byname(cinfo->col_custom_field[i]);
-            /* ASSERT if none ? */
-                       g_assert(hfinfo);
-            cinfo->col_custom_field_id[i] = hfinfo->id;
+            cinfo->col_custom_field_id[i] = hfinfo ? hfinfo->id : -1;
         }
-       }
+    }
   }
 }
 
-gboolean
-col_has_time_fmt(column_info *cinfo, gint col)
-{
-  return ((cinfo->fmt_matx[col][COL_CLS_TIME]) ||
-          (cinfo->fmt_matx[col][COL_ABS_TIME]) ||
-          (cinfo->fmt_matx[col][COL_ABS_DATE_TIME]) ||
-          (cinfo->fmt_matx[col][COL_REL_TIME]) ||
-          (cinfo->fmt_matx[col][COL_DELTA_TIME]) ||
-          (cinfo->fmt_matx[col][COL_DELTA_TIME_DIS]));
-}
-
-static void
-col_do_append_sep_va_fstr(column_info *cinfo, gint el, const gchar *separator,
-              const gchar *format, va_list ap)
+/*  Appends a vararg list to a packet info string.
+ *  This function's code is duplicated in col_append_sep_fstr() below because
+ *  the for() loop below requires us to call va_start/va_end so intermediate
+ *  functions are a problem.
+ */
+void
+col_append_fstr(column_info *cinfo, const gint el, const gchar *format, ...)
 {
   int  i;
-  int  len, max_len, sep_len;
+  int  len, max_len;
+  va_list ap;
+
+  if (!CHECK_COL(cinfo, el))
+    return;
 
   if (el == COL_INFO)
     max_len = COL_MAX_INFO_LEN;
   else
     max_len = COL_MAX_LEN;
 
-  if (separator == NULL)
-    sep_len = 0;
-  else
-    sep_len = (int) strlen(separator);
   for (i = cinfo->col_first[el]; i <= cinfo->col_last[el]; i++) {
     if (cinfo->fmt_matx[i][el]) {
       /*
@@ -366,41 +281,24 @@ col_do_append_sep_va_fstr(column_info *cinfo, gint el, const gchar *separator,
 
       len = (int) strlen(cinfo->col_buf[i]);
 
-      /*
-       * If we have a separator, append it if the column isn't empty.
-       */
-      if (separator != NULL) {
-        if (len != 0) {
-          g_strlcat(cinfo->col_buf[i], separator, max_len);
-          len += sep_len;
-        }
-      }
+      va_start(ap, format);
       g_vsnprintf(&cinfo->col_buf[i][len], max_len - len, format, ap);
-      cinfo->col_buf[i][max_len-1] = 0;
+      va_end(ap);
     }
   }
-}
 
-/* Appends a vararg list to a packet info string. */
-void
-col_append_fstr(column_info *cinfo, gint el, const gchar *format, ...)
-{
-  va_list ap;
-
-  if (!CHECK_COL(cinfo, el))
-    return;
-
-  va_start(ap, format);
-  col_do_append_sep_va_fstr(cinfo, el, NULL, format, ap);
-  va_end(ap);
 }
 
-/* Appends a vararg list to a packet info string.
- * Prefixes it with the given separator if the column is not empty. */
+/*  Appends a vararg list to a packet info string.
+ *  Prefixes it with the given separator if the column is not empty.
+ *  Code is duplicated from col_append_fstr above().
+ */
 void
-col_append_sep_fstr(column_info *cinfo, gint el, const gchar *separator,
-        const gchar *format, ...)
+col_append_sep_fstr(column_info *cinfo, const gint el, const gchar *separator,
+                   const gchar *format, ...)
 {
+  int  i;
+  int  len, max_len, sep_len;
   va_list ap;
 
   if (!CHECK_COL(cinfo, el))
@@ -408,18 +306,44 @@ col_append_sep_fstr(column_info *cinfo, gint el, const gchar *separator,
 
   if (separator == NULL)
     separator = ", ";    /* default */
-  va_start(ap, format);
-  col_do_append_sep_va_fstr(cinfo, el, separator, format, ap);
-  va_end(ap);
-}
 
+  sep_len = (int) strlen(separator);
+
+  if (el == COL_INFO)
+    max_len = COL_MAX_INFO_LEN;
+  else
+    max_len = COL_MAX_LEN;
+
+  for (i = cinfo->col_first[el]; i <= cinfo->col_last[el]; i++) {
+    if (cinfo->fmt_matx[i][el]) {
+      /*
+       * First arrange that we can append, if necessary.
+       */
+      COL_CHECK_APPEND(cinfo, i, max_len);
 
+      len = (int) strlen(cinfo->col_buf[i]);
+
+      /*
+       * If we have a separator, append it if the column isn't empty.
+       */
+      if (sep_len != 0) {
+        if (len != 0) {
+          g_strlcat(cinfo->col_buf[i], separator, max_len);
+          len += sep_len;
+        }
+      }
+      va_start(ap, format);
+      g_vsnprintf(&cinfo->col_buf[i][len], max_len - len, format, ap);
+      va_end(ap);
+    }
+  }
+}
 
 /* Prepends a vararg list to a packet info string. */
 #define COL_BUF_MAX_LEN (((COL_MAX_INFO_LEN) > (COL_MAX_LEN)) ? \
     (COL_MAX_INFO_LEN) : (COL_MAX_LEN))
 void
-col_prepend_fstr(column_info *cinfo, gint el, const gchar *format, ...)
+col_prepend_fstr(column_info *cinfo, const gint el, const gchar *format, ...)
 {
   va_list     ap;
   int         i;
@@ -435,7 +359,6 @@ col_prepend_fstr(column_info *cinfo, gint el, const gchar *format, ...)
   else
     max_len = COL_MAX_LEN;
 
-  va_start(ap, format);
   for (i = cinfo->col_first[el]; i <= cinfo->col_last[el]; i++) {
     if (cinfo->fmt_matx[i][el]) {
       if (cinfo->col_data[i] != cinfo->col_buf[i]) {
@@ -445,8 +368,9 @@ col_prepend_fstr(column_info *cinfo, gint el, const gchar *format, ...)
         g_strlcpy(orig_buf, cinfo->col_buf[i], max_len);
         orig = orig_buf;
       }
+      va_start(ap, format);
       g_vsnprintf(cinfo->col_buf[i], max_len, format, ap);
-      cinfo->col_buf[i][max_len - 1] = '\0';
+      va_end(ap);
 
       /*
        * Move the fence, unless it's at the beginning of the string.
@@ -458,10 +382,9 @@ col_prepend_fstr(column_info *cinfo, gint el, const gchar *format, ...)
       cinfo->col_data[i] = cinfo->col_buf[i];
     }
   }
-  va_end(ap);
 }
 void
-col_prepend_fence_fstr(column_info *cinfo, gint el, const gchar *format, ...)
+col_prepend_fence_fstr(column_info *cinfo, const gint el, const gchar *format, ...)
 {
   va_list     ap;
   int         i;
@@ -477,7 +400,6 @@ col_prepend_fence_fstr(column_info *cinfo, gint el, const gchar *format, ...)
   else
     max_len = COL_MAX_LEN;
 
-  va_start(ap, format);
   for (i = cinfo->col_first[el]; i <= cinfo->col_last[el]; i++) {
     if (cinfo->fmt_matx[i][el]) {
       if (cinfo->col_data[i] != cinfo->col_buf[i]) {
@@ -487,8 +409,9 @@ col_prepend_fence_fstr(column_info *cinfo, gint el, const gchar *format, ...)
         g_strlcpy(orig_buf, cinfo->col_buf[i], max_len);
         orig = orig_buf;
       }
+      va_start(ap, format);
       g_vsnprintf(cinfo->col_buf[i], max_len, format, ap);
-      cinfo->col_buf[i][max_len - 1] = '\0';
+      va_end(ap);
 
       /*
        * Move the fence if it exists, else create a new fence at the
@@ -503,13 +426,12 @@ col_prepend_fence_fstr(column_info *cinfo, gint el, const gchar *format, ...)
       cinfo->col_data[i] = cinfo->col_buf[i];
     }
   }
-  va_end(ap);
 }
 
 /* Use this if "str" points to something that won't stay around (and
    must thus be copied). */
 void
-col_add_str(column_info *cinfo, gint el, const gchar* str)
+col_add_str(column_info *cinfo, const gint el, const gchar* str)
 {
   int    i;
   int    fence;
@@ -543,12 +465,20 @@ col_add_str(column_info *cinfo, gint el, const gchar* str)
   }
 }
 
-static void
-col_do_append_str(column_info *cinfo, gint el, const gchar* separator,
-    const gchar* str)
+/* Use this if "str" points to something that will stay around (and thus
+   needn't be copied). */
+void
+col_set_str(column_info *cinfo, const gint el, const gchar* str)
 {
-  int    i;
-  size_t len, max_len, sep_len;
+  int i;
+  int fence;
+  size_t max_len;
+
+  DISSECTOR_ASSERT(str);
+
+  /* The caller is expected to pass in something that 'will stay around' and
+   * something from the ephemeral pool certainly doesn't fit the bill. */
+  DISSECTOR_ASSERT(!ep_verify_pointer(str));
 
   if (!CHECK_COL(cinfo, el))
     return;
@@ -558,10 +488,77 @@ col_do_append_str(column_info *cinfo, gint el, const gchar* separator,
   else
     max_len = COL_MAX_LEN;
 
-  if (separator == NULL)
-    sep_len = 0;
+  for (i = cinfo->col_first[el]; i <= cinfo->col_last[el]; i++) {
+    if (cinfo->fmt_matx[i][el]) {
+      fence = cinfo->col_fence[i];
+      if (fence != 0) {
+        /*
+         * We will append the string after the fence.
+         * First arrange that we can append, if necessary.
+         */
+        COL_CHECK_APPEND(cinfo, i, max_len);
+
+        g_strlcpy(&cinfo->col_buf[i][fence], str, max_len - fence);
+      } else {
+        /*
+         * There's no fence, so we can just set the column to point
+         * to the string.
+         */
+        cinfo->col_data[i] = str;
+      }
+    }
+  }
+}
+
+/* Adds a vararg list to a packet info string. */
+void
+col_add_fstr(column_info *cinfo, const gint el, const gchar *format, ...) {
+  va_list ap;
+  int     i;
+  int     fence;
+  int     max_len;
+
+  if (!CHECK_COL(cinfo, el))
+    return;
+
+  if (el == COL_INFO)
+    max_len = COL_MAX_INFO_LEN;
   else
-    sep_len = strlen(separator);
+    max_len = COL_MAX_LEN;
+
+  for (i = cinfo->col_first[el]; i <= cinfo->col_last[el]; i++) {
+    if (cinfo->fmt_matx[i][el]) {
+      fence = cinfo->col_fence[i];
+      if (fence != 0) {
+        /*
+         * We will append the string after the fence.
+         * First arrange that we can append, if necessary.
+         */
+        COL_CHECK_APPEND(cinfo, i, max_len);
+      } else {
+        /*
+         * There's no fence, so we can just write to the string.
+         */
+        cinfo->col_data[i] = cinfo->col_buf[i];
+      }
+      va_start(ap, format);
+      g_vsnprintf(&cinfo->col_buf[i][fence], max_len - fence, format, ap);
+      va_end(ap);
+    }
+  }
+}
+
+static void
+col_do_append_str(column_info *cinfo, const gint el, const gchar* separator,
+    const gchar* str)
+{
+  int    i;
+  size_t len, max_len;
+
+  if (el == COL_INFO)
+    max_len = COL_MAX_INFO_LEN;
+  else
+    max_len = COL_MAX_LEN;
 
   for (i = cinfo->col_first[el]; i <= cinfo->col_last[el]; i++) {
     if (cinfo->fmt_matx[i][el]) {
@@ -586,35 +583,59 @@ col_do_append_str(column_info *cinfo, gint el, const gchar* separator,
 }
 
 void
-col_append_str(column_info *cinfo, gint el, const gchar* str)
+col_append_str(column_info *cinfo, const gint el, const gchar* str)
 {
+  if (!CHECK_COL(cinfo, el))
+    return;
+
   col_do_append_str(cinfo, el, NULL, str);
 }
 
 void
-col_append_sep_str(column_info *cinfo, gint el, const gchar* separator,
+col_append_sep_str(column_info *cinfo, const gint el, const gchar* separator,
     const gchar* str)
 {
+  if (!CHECK_COL(cinfo, el))
+    return;
+
   if (separator == NULL)
     separator = ", ";    /* default */
+
   col_do_append_str(cinfo, el, separator, str);
 }
 
 /* --------------------------------- */
-static gint
-set_abs_date_time(frame_data *fd, gchar *buf)
+gboolean
+col_has_time_fmt(column_info *cinfo, const gint col)
+{
+  return ((cinfo->fmt_matx[col][COL_CLS_TIME]) ||
+          (cinfo->fmt_matx[col][COL_ABS_TIME]) ||
+          (cinfo->fmt_matx[col][COL_ABS_DATE_TIME]) ||
+          (cinfo->fmt_matx[col][COL_UTC_TIME]) ||
+          (cinfo->fmt_matx[col][COL_UTC_DATE_TIME]) ||
+          (cinfo->fmt_matx[col][COL_REL_TIME]) ||
+          (cinfo->fmt_matx[col][COL_DELTA_TIME]) ||
+          (cinfo->fmt_matx[col][COL_DELTA_TIME_DIS]));
+}
+
+static void
+set_abs_date_time(const frame_data *fd, gchar *buf, gboolean local)
 {
   struct tm *tmp;
   time_t then;
 
-  COL_CHECK_REF_TIME(fd, buf);
-
-  then = fd->abs_ts.secs;
-  tmp = localtime(&then);
+  if (fd->flags.has_ts) {
+    then = fd->abs_ts.secs;
+    if (local)
+       tmp = localtime(&then);
+    else
+       tmp = gmtime(&then);
+  } else
+    tmp = NULL;
   if (tmp != NULL) {
       switch(timestamp_get_precision()) {
-      case(TS_PREC_FIXED_SEC):
-      case(TS_PREC_AUTO_SEC):
+      case TS_PREC_FIXED_SEC:
+      case TS_PREC_AUTO_SEC:
           g_snprintf(buf, COL_MAX_LEN,"%04d-%02d-%02d %02d:%02d:%02d",
              tmp->tm_year + 1900,
              tmp->tm_mon + 1,
@@ -623,8 +644,8 @@ set_abs_date_time(frame_data *fd, gchar *buf)
              tmp->tm_min,
              tmp->tm_sec);
           break;
-      case(TS_PREC_FIXED_DSEC):
-      case(TS_PREC_AUTO_DSEC):
+      case TS_PREC_FIXED_DSEC:
+      case TS_PREC_AUTO_DSEC:
           g_snprintf(buf, COL_MAX_LEN,"%04d-%02d-%02d %02d:%02d:%02d.%01ld",
              tmp->tm_year + 1900,
              tmp->tm_mon + 1,
@@ -634,8 +655,8 @@ set_abs_date_time(frame_data *fd, gchar *buf)
              tmp->tm_sec,
              (long)fd->abs_ts.nsecs / 100000000);
           break;
-      case(TS_PREC_FIXED_CSEC):
-      case(TS_PREC_AUTO_CSEC):
+      case TS_PREC_FIXED_CSEC:
+      case TS_PREC_AUTO_CSEC:
           g_snprintf(buf, COL_MAX_LEN,"%04d-%02d-%02d %02d:%02d:%02d.%02ld",
              tmp->tm_year + 1900,
              tmp->tm_mon + 1,
@@ -645,8 +666,8 @@ set_abs_date_time(frame_data *fd, gchar *buf)
              tmp->tm_sec,
              (long)fd->abs_ts.nsecs / 10000000);
           break;
-      case(TS_PREC_FIXED_MSEC):
-      case(TS_PREC_AUTO_MSEC):
+      case TS_PREC_FIXED_MSEC:
+      case TS_PREC_AUTO_MSEC:
           g_snprintf(buf, COL_MAX_LEN, "%04d-%02d-%02d %02d:%02d:%02d.%03ld",
              tmp->tm_year + 1900,
              tmp->tm_mon + 1,
@@ -656,8 +677,8 @@ set_abs_date_time(frame_data *fd, gchar *buf)
              tmp->tm_sec,
              (long)fd->abs_ts.nsecs / 1000000);
           break;
-      case(TS_PREC_FIXED_USEC):
-      case(TS_PREC_AUTO_USEC):
+      case TS_PREC_FIXED_USEC:
+      case TS_PREC_AUTO_USEC:
           g_snprintf(buf, COL_MAX_LEN, "%04d-%02d-%02d %02d:%02d:%02d.%06ld",
              tmp->tm_year + 1900,
              tmp->tm_mon + 1,
@@ -667,8 +688,8 @@ set_abs_date_time(frame_data *fd, gchar *buf)
              tmp->tm_sec,
              (long)fd->abs_ts.nsecs / 1000);
           break;
-      case(TS_PREC_FIXED_NSEC):
-      case(TS_PREC_AUTO_NSEC):
+      case TS_PREC_FIXED_NSEC:
+      case TS_PREC_AUTO_NSEC:
           g_snprintf(buf, COL_MAX_LEN, "%04d-%02d-%02d %02d:%02d:%02d.%09ld",
              tmp->tm_year + 1900,
              tmp->tm_mon + 1,
@@ -684,236 +705,354 @@ set_abs_date_time(frame_data *fd, gchar *buf)
   } else {
     buf[0] = '\0';
   }
-  return 1;
 }
 
 static void
-col_set_abs_date_time(frame_data *fd, column_info *cinfo, int col)
+col_set_abs_date_time(const frame_data *fd, column_info *cinfo, const int col)
 {
-  if (set_abs_date_time(fd, cinfo->col_buf[col])) {
-      cinfo->col_expr.col_expr[col] = "frame.time";
-      g_strlcpy(cinfo->col_expr.col_expr_val[col],cinfo->col_buf[col],COL_MAX_LEN);
-  }
+  set_abs_date_time(fd, cinfo->col_buf[col], TRUE);
+  cinfo->col_expr.col_expr[col] = "frame.time";
+  g_strlcpy(cinfo->col_expr.col_expr_val[col],cinfo->col_buf[col],COL_MAX_LEN);
+
   cinfo->col_data[col] = cinfo->col_buf[col];
 }
 
-/* --------------------------------- */
-static gint
-set_rel_time(frame_data *fd, gchar *buf)
+static void
+col_set_utc_date_time(const frame_data *fd, column_info *cinfo, const int col)
 {
-  COL_CHECK_REF_TIME(fd, buf);
+  set_abs_date_time(fd, cinfo->col_buf[col], FALSE);
+  cinfo->col_expr.col_expr[col] = "frame.time";
+  g_strlcpy(cinfo->col_expr.col_expr_val[col],cinfo->col_buf[col],COL_MAX_LEN);
+
+  cinfo->col_data[col] = cinfo->col_buf[col];
+}
 
+static void
+set_time_seconds(const nstime_t *ts, gchar *buf)
+{
   switch(timestamp_get_precision()) {
-      case(TS_PREC_FIXED_SEC):
-      case(TS_PREC_AUTO_SEC):
+      case TS_PREC_FIXED_SEC:
+      case TS_PREC_AUTO_SEC:
           display_signed_time(buf, COL_MAX_LEN,
-            (gint32) fd->rel_ts.secs, fd->rel_ts.nsecs / 1000000000, SECS);
+            (gint32) ts->secs, ts->nsecs / 1000000000, TO_STR_TIME_RES_T_SECS);
           break;
-      case(TS_PREC_FIXED_DSEC):
-      case(TS_PREC_AUTO_DSEC):
+      case TS_PREC_FIXED_DSEC:
+      case TS_PREC_AUTO_DSEC:
           display_signed_time(buf, COL_MAX_LEN,
-            (gint32) fd->rel_ts.secs, fd->rel_ts.nsecs / 100000000, DSECS);
+            (gint32) ts->secs, ts->nsecs / 100000000, TO_STR_TIME_RES_T_DSECS);
           break;
-      case(TS_PREC_FIXED_CSEC):
-      case(TS_PREC_AUTO_CSEC):
+      case TS_PREC_FIXED_CSEC:
+      case TS_PREC_AUTO_CSEC:
           display_signed_time(buf, COL_MAX_LEN,
-            (gint32) fd->rel_ts.secs, fd->rel_ts.nsecs / 10000000, CSECS);
+            (gint32) ts->secs, ts->nsecs / 10000000, TO_STR_TIME_RES_T_CSECS);
           break;
-      case(TS_PREC_FIXED_MSEC):
-      case(TS_PREC_AUTO_MSEC):
+      case TS_PREC_FIXED_MSEC:
+      case TS_PREC_AUTO_MSEC:
           display_signed_time(buf, COL_MAX_LEN,
-            (gint32) fd->rel_ts.secs, fd->rel_ts.nsecs / 1000000, MSECS);
+            (gint32) ts->secs, ts->nsecs / 1000000, TO_STR_TIME_RES_T_MSECS);
           break;
-      case(TS_PREC_FIXED_USEC):
-      case(TS_PREC_AUTO_USEC):
+      case TS_PREC_FIXED_USEC:
+      case TS_PREC_AUTO_USEC:
           display_signed_time(buf, COL_MAX_LEN,
-            (gint32) fd->rel_ts.secs, fd->rel_ts.nsecs / 1000, USECS);
+            (gint32) ts->secs, ts->nsecs / 1000, TO_STR_TIME_RES_T_USECS);
           break;
-      case(TS_PREC_FIXED_NSEC):
-      case(TS_PREC_AUTO_NSEC):
+      case TS_PREC_FIXED_NSEC:
+      case TS_PREC_AUTO_NSEC:
           display_signed_time(buf, COL_MAX_LEN,
-            (gint32) fd->rel_ts.secs, fd->rel_ts.nsecs, NSECS);
+            (gint32) ts->secs, ts->nsecs, TO_STR_TIME_RES_T_NSECS);
           break;
       default:
           g_assert_not_reached();
   }
-  return 1;
 }
 
 static void
-col_set_rel_time(frame_data *fd, column_info *cinfo, int col)
+set_time_hour_min_sec(const nstime_t *ts, gchar *buf)
 {
-  if (set_rel_time(fd, cinfo->col_buf[col])) {
-      cinfo->col_expr.col_expr[col] = "frame.time_relative";
-      g_strlcpy(cinfo->col_expr.col_expr_val[col],cinfo->col_buf[col],COL_MAX_LEN);
-  }
-  cinfo->col_data[col] = cinfo->col_buf[col];
-}
+  time_t secs = ts->secs;
+  long nsecs = (long) ts->nsecs;
+  gboolean negative = FALSE;
 
-/* ------------------------------------------ */
-static gint
-set_delta_time(frame_data *fd, gchar *buf)
-{
-  COL_CHECK_REF_TIME(fd, buf);
+  if (secs < 0) {
+    secs = -secs;
+    negative = TRUE;
+  }
+  if (nsecs < 0) {
+    nsecs = -nsecs;
+    negative = TRUE;
+  }
 
   switch(timestamp_get_precision()) {
-      case(TS_PREC_FIXED_SEC):
-      case(TS_PREC_AUTO_SEC):
-          display_signed_time(buf, COL_MAX_LEN,
-            (gint32) fd->del_cap_ts.secs, fd->del_cap_ts.nsecs / 1000000000, SECS);
-          break;
-      case(TS_PREC_FIXED_DSEC):
-      case(TS_PREC_AUTO_DSEC):
-          display_signed_time(buf, COL_MAX_LEN,
-            (gint32) fd->del_cap_ts.secs, fd->del_cap_ts.nsecs / 100000000, DSECS);
-          break;
-      case(TS_PREC_FIXED_CSEC):
-      case(TS_PREC_AUTO_CSEC):
-          display_signed_time(buf, COL_MAX_LEN,
-            (gint32) fd->del_cap_ts.secs, fd->del_cap_ts.nsecs / 10000000, CSECS);
-          break;
-      case(TS_PREC_FIXED_MSEC):
-      case(TS_PREC_AUTO_MSEC):
-          display_signed_time(buf, COL_MAX_LEN,
-            (gint32) fd->del_cap_ts.secs, fd->del_cap_ts.nsecs / 1000000, MSECS);
-          break;
-      case(TS_PREC_FIXED_USEC):
-      case(TS_PREC_AUTO_USEC):
-          display_signed_time(buf, COL_MAX_LEN,
-            (gint32) fd->del_cap_ts.secs, fd->del_cap_ts.nsecs / 1000, USECS);
-          break;
-      case(TS_PREC_FIXED_NSEC):
-      case(TS_PREC_AUTO_NSEC):
-          display_signed_time(buf, COL_MAX_LEN,
-            (gint32) fd->del_cap_ts.secs, fd->del_cap_ts.nsecs, NSECS);
-          break;
-      default:
-          g_assert_not_reached();
+  case TS_PREC_FIXED_SEC:
+  case TS_PREC_AUTO_SEC:
+    if (secs >= (60*60)) {
+      g_snprintf(buf, COL_MAX_LEN, "%s%dh %2dm %2ds",
+                negative ? "- " : "",
+                (gint32) secs / (60 * 60),
+                (gint32) (secs / 60) % 60,
+                (gint32) secs % 60);
+    } else if (secs >= 60) {
+      g_snprintf(buf, COL_MAX_LEN, "%s%dm %2ds",
+                negative ? "- " : "",
+                (gint32) secs / 60,
+                (gint32) secs % 60);
+    } else {
+      g_snprintf(buf, COL_MAX_LEN, "%s%ds",
+                negative ? "- " : "",
+                (gint32) secs);
+    }
+    break;
+  case TS_PREC_FIXED_DSEC:
+  case TS_PREC_AUTO_DSEC:
+    if (secs >= (60*60)) {
+      g_snprintf(buf, COL_MAX_LEN, "%s%dh %2dm %2d.%01lds",
+                negative ? "- " : "",
+                (gint32) secs / (60 * 60),
+                (gint32) (secs / 60) % 60,
+                (gint32) secs % 60,
+                nsecs / 100000000);
+    } else if (secs >= 60) {
+      g_snprintf(buf, COL_MAX_LEN, "%s%dm %2d.%01lds",
+                negative ? "- " : "",
+                (gint32) secs / 60,
+                (gint32) secs % 60,
+                nsecs / 100000000);
+    } else {
+      g_snprintf(buf, COL_MAX_LEN, "%s%d.%01lds",
+                negative ? "- " : "",
+                (gint32) secs,
+                nsecs / 100000000);
+    }
+    break;
+  case TS_PREC_FIXED_CSEC:
+  case TS_PREC_AUTO_CSEC:
+    if (secs >= (60*60)) {
+      g_snprintf(buf, COL_MAX_LEN, "%s%dh %2dm %2d.%02lds",
+                negative ? "- " : "",
+                (gint32) secs / (60 * 60),
+                (gint32) (secs / 60) % 60,
+                (gint32) secs % 60,
+                nsecs / 10000000);
+    } else if (secs >= 60) {
+      g_snprintf(buf, COL_MAX_LEN, "%s%dm %2d.%02lds",
+                negative ? "- " : "",
+                (gint32) secs / 60,
+                (gint32) secs % 60,
+                nsecs / 10000000);
+    } else {
+      g_snprintf(buf, COL_MAX_LEN, "%s%d.%02lds",
+                negative ? "- " : "",
+                (gint32) secs,
+                nsecs / 10000000);
+    }
+    break;
+  case TS_PREC_FIXED_MSEC:
+  case TS_PREC_AUTO_MSEC:
+    if (secs >= (60*60)) {
+      g_snprintf(buf, COL_MAX_LEN, "%s%dh %2dm %2d.%03lds",
+                negative ? "- " : "",
+                (gint32) secs / (60 * 60),
+                (gint32) (secs / 60) % 60,
+                (gint32) secs % 60,
+                nsecs / 1000000);
+    } else if (secs >= 60) {
+      g_snprintf(buf, COL_MAX_LEN, "%s%dm %2d.%03lds",
+                negative ? "- " : "",
+                (gint32) secs / 60,
+                (gint32) secs % 60,
+                nsecs / 1000000);
+    } else {
+      g_snprintf(buf, COL_MAX_LEN, "%s%d.%03lds",
+                negative ? "- " : "",
+                (gint32) secs,
+                nsecs / 1000000);
+    }
+    break;
+  case TS_PREC_FIXED_USEC:
+  case TS_PREC_AUTO_USEC:
+    if (secs >= (60*60)) {
+      g_snprintf(buf, COL_MAX_LEN, "%s%dh %2dm %2d.%06lds",
+                negative ? "- " : "",
+                (gint32) secs / (60 * 60),
+                (gint32) (secs / 60) % 60,
+                (gint32) secs % 60,
+                nsecs / 1000);
+    } else if (secs >= 60) {
+      g_snprintf(buf, COL_MAX_LEN, "%s%dm %2d.%06lds",
+                negative ? "- " : "",
+                (gint32) secs / 60,
+                (gint32) secs % 60,
+                nsecs / 1000);
+    } else {
+      g_snprintf(buf, COL_MAX_LEN, "%s%d.%06lds",
+                negative ? "- " : "",
+                (gint32) secs,
+                nsecs / 1000);
+    }
+    break;
+  case TS_PREC_FIXED_NSEC:
+  case TS_PREC_AUTO_NSEC:
+    if (secs >= (60*60)) {
+      g_snprintf(buf, COL_MAX_LEN, "%s%dh %2dm %2d.%09lds",
+                negative ? "- " : "",
+                (gint32) secs / (60 * 60),
+                (gint32) (secs / 60) % 60,
+                (gint32) secs % 60,
+                nsecs);
+    } else if (secs >= 60) {
+      g_snprintf(buf, COL_MAX_LEN, "%s%dm %2d.%09lds",
+                negative ? "- " : "",
+                (gint32) secs / 60,
+                (gint32) secs % 60,
+                nsecs);
+    } else {
+      g_snprintf(buf, COL_MAX_LEN, "%s%d.%09lds",
+                negative ? "- " : "",
+                (gint32) secs,
+                nsecs);
+    }
+    break;
+  default:
+    g_assert_not_reached();
   }
-  return 1;
 }
 
 static void
-col_set_delta_time(frame_data *fd, column_info *cinfo, int col)
+col_set_rel_time(const frame_data *fd, column_info *cinfo, const int col)
 {
-  if (set_delta_time(fd, cinfo->col_buf[col])) {
-      cinfo->col_expr.col_expr[col] = "frame.time_delta";
-      g_strlcpy(cinfo->col_expr.col_expr_val[col],cinfo->col_buf[col],COL_MAX_LEN);
+  if (!fd->flags.has_ts) {
+    cinfo->col_buf[col][0] = '\0';
+    return;
+  }
+  switch (timestamp_get_seconds_type()) {
+  case TS_SECONDS_DEFAULT:
+    set_time_seconds(&fd->rel_ts, cinfo->col_buf[col]);
+    cinfo->col_expr.col_expr[col] = "frame.time_relative";
+    g_strlcpy(cinfo->col_expr.col_expr_val[col],cinfo->col_buf[col],COL_MAX_LEN);
+    break;
+  case TS_SECONDS_HOUR_MIN_SEC:
+    set_time_hour_min_sec(&fd->rel_ts, cinfo->col_buf[col]);
+    cinfo->col_expr.col_expr[col] = "frame.time_relative";
+    set_time_seconds(&fd->rel_ts, cinfo->col_expr.col_expr_val[col]);
+    break;
+  default:
+    g_assert_not_reached();
   }
   cinfo->col_data[col] = cinfo->col_buf[col];
 }
 
-/* ------------------------------------------
- To do: Add CHECK_COL checks to the col_add* routines
-*/
-static gint
-set_delta_time_dis(frame_data *fd, gchar *buf)
+static void
+col_set_delta_time(const frame_data *fd, column_info *cinfo, const int col)
 {
-  COL_CHECK_REF_TIME(fd, buf);
+  nstime_t del_cap_ts;
 
-  switch(timestamp_get_precision()) {
-      case(TS_PREC_FIXED_SEC):
-      case(TS_PREC_AUTO_SEC):
-          display_signed_time(buf, COL_MAX_LEN,
-            (gint32) fd->del_dis_ts.secs, fd->del_dis_ts.nsecs / 1000000000, SECS);
-          break;
-      case(TS_PREC_FIXED_DSEC):
-      case(TS_PREC_AUTO_DSEC):
-          display_signed_time(buf, COL_MAX_LEN,
-            (gint32) fd->del_dis_ts.secs, fd->del_dis_ts.nsecs / 100000000, DSECS);
-          break;
-      case(TS_PREC_FIXED_CSEC):
-      case(TS_PREC_AUTO_CSEC):
-          display_signed_time(buf, COL_MAX_LEN,
-            (gint32) fd->del_dis_ts.secs, fd->del_dis_ts.nsecs / 10000000, CSECS);
-          break;
-      case(TS_PREC_FIXED_MSEC):
-      case(TS_PREC_AUTO_MSEC):
-          display_signed_time(buf, COL_MAX_LEN,
-            (gint32) fd->del_dis_ts.secs, fd->del_dis_ts.nsecs / 1000000, MSECS);
-          break;
-      case(TS_PREC_FIXED_USEC):
-      case(TS_PREC_AUTO_USEC):
-          display_signed_time(buf, COL_MAX_LEN,
-            (gint32) fd->del_dis_ts.secs, fd->del_dis_ts.nsecs / 1000, USECS);
-          break;
-      case(TS_PREC_FIXED_NSEC):
-      case(TS_PREC_AUTO_NSEC):
-          display_signed_time(buf, COL_MAX_LEN,
-            (gint32) fd->del_dis_ts.secs, fd->del_dis_ts.nsecs, NSECS);
-          break;
-      default:
-          g_assert_not_reached();
+  frame_delta_abs_time(fd, fd->prev_cap, &del_cap_ts);
+
+  switch (timestamp_get_seconds_type()) {
+  case TS_SECONDS_DEFAULT:
+    set_time_seconds(&del_cap_ts, cinfo->col_buf[col]);
+    cinfo->col_expr.col_expr[col] = "frame.time_delta";
+    g_strlcpy(cinfo->col_expr.col_expr_val[col],cinfo->col_buf[col],COL_MAX_LEN);
+    break;
+  case TS_SECONDS_HOUR_MIN_SEC:
+    set_time_hour_min_sec(&del_cap_ts, cinfo->col_buf[col]);
+    cinfo->col_expr.col_expr[col] = "frame.time_delta";
+    set_time_seconds(&del_cap_ts, cinfo->col_expr.col_expr_val[col]);
+    break;
+  default:
+    g_assert_not_reached();
   }
-  return 1;
+
+  cinfo->col_data[col] = cinfo->col_buf[col];
 }
 
 static void
-col_set_delta_time_dis(frame_data *fd, column_info *cinfo, int col)
+col_set_delta_time_dis(const frame_data *fd, column_info *cinfo, const int col)
 {
-  if (set_delta_time_dis(fd, cinfo->col_buf[col])) {
+  nstime_t del_dis_ts;
+
+  if (!fd->flags.has_ts) {
+    cinfo->col_buf[col][0] = '\0';
+    return;
+  }
+
+  frame_delta_abs_time(fd, fd->prev_dis, &del_dis_ts);
+
+  switch (timestamp_get_seconds_type()) {
+  case TS_SECONDS_DEFAULT:
+    set_time_seconds(&del_dis_ts, cinfo->col_buf[col]);
     cinfo->col_expr.col_expr[col] = "frame.time_delta_displayed";
     g_strlcpy(cinfo->col_expr.col_expr_val[col],cinfo->col_buf[col],COL_MAX_LEN);
+    break;
+  case TS_SECONDS_HOUR_MIN_SEC:
+    set_time_hour_min_sec(&del_dis_ts, cinfo->col_buf[col]);
+    cinfo->col_expr.col_expr[col] = "frame.time_delta_displayed";
+    set_time_seconds(&del_dis_ts, cinfo->col_expr.col_expr_val[col]);
+    break;
+  default:
+    g_assert_not_reached();
   }
+
   cinfo->col_data[col] = cinfo->col_buf[col];
 }
 
-/* ------------------------ */
-/* To do: Add CHECK_COL checks to the col_add* routines */
-
-static gint
-set_abs_time(frame_data *fd, gchar *buf)
+static void
+set_abs_time(const frame_data *fd, gchar *buf, gboolean local)
 {
   struct tm *tmp;
   time_t then;
 
-  COL_CHECK_REF_TIME(fd, buf);
-
-  then = fd->abs_ts.secs;
-  tmp = localtime(&then);
+  if (fd->flags.has_ts) {
+    then = fd->abs_ts.secs;
+    if (local)
+       tmp = localtime(&then);
+    else
+       tmp = gmtime(&then);
+  } else
+    tmp = NULL;
   if (tmp != NULL) {
       switch(timestamp_get_precision()) {
-      case(TS_PREC_FIXED_SEC):
-      case(TS_PREC_AUTO_SEC):
+      case TS_PREC_FIXED_SEC:
+      case TS_PREC_AUTO_SEC:
           g_snprintf(buf, COL_MAX_LEN,"%02d:%02d:%02d",
              tmp->tm_hour,
              tmp->tm_min,
              tmp->tm_sec);
           break;
-      case(TS_PREC_FIXED_DSEC):
-      case(TS_PREC_AUTO_DSEC):
+      case TS_PREC_FIXED_DSEC:
+      case TS_PREC_AUTO_DSEC:
           g_snprintf(buf, COL_MAX_LEN,"%02d:%02d:%02d.%01ld",
              tmp->tm_hour,
              tmp->tm_min,
              tmp->tm_sec,
              (long)fd->abs_ts.nsecs / 100000000);
           break;
-      case(TS_PREC_FIXED_CSEC):
-      case(TS_PREC_AUTO_CSEC):
+      case TS_PREC_FIXED_CSEC:
+      case TS_PREC_AUTO_CSEC:
           g_snprintf(buf, COL_MAX_LEN,"%02d:%02d:%02d.%02ld",
              tmp->tm_hour,
              tmp->tm_min,
              tmp->tm_sec,
              (long)fd->abs_ts.nsecs / 10000000);
           break;
-      case(TS_PREC_FIXED_MSEC):
-      case(TS_PREC_AUTO_MSEC):
+      case TS_PREC_FIXED_MSEC:
+      case TS_PREC_AUTO_MSEC:
           g_snprintf(buf, COL_MAX_LEN,"%02d:%02d:%02d.%03ld",
              tmp->tm_hour,
              tmp->tm_min,
              tmp->tm_sec,
              (long)fd->abs_ts.nsecs / 1000000);
           break;
-      case(TS_PREC_FIXED_USEC):
-      case(TS_PREC_AUTO_USEC):
+      case TS_PREC_FIXED_USEC:
+      case TS_PREC_AUTO_USEC:
           g_snprintf(buf, COL_MAX_LEN,"%02d:%02d:%02d.%06ld",
              tmp->tm_hour,
              tmp->tm_min,
              tmp->tm_sec,
              (long)fd->abs_ts.nsecs / 1000);
           break;
-      case(TS_PREC_FIXED_NSEC):
-      case(TS_PREC_AUTO_NSEC):
+      case TS_PREC_FIXED_NSEC:
+      case TS_PREC_AUTO_NSEC:
           g_snprintf(buf, COL_MAX_LEN, "%02d:%02d:%02d.%09ld",
              tmp->tm_hour,
              tmp->tm_min,
@@ -927,67 +1066,75 @@ set_abs_time(frame_data *fd, gchar *buf)
   } else {
     *buf = '\0';
   }
-  return 1;
 }
 
 static void
-col_set_abs_time(frame_data *fd, column_info *cinfo, int col)
+col_set_abs_time(const frame_data *fd, column_info *cinfo, const int col)
 {
-  if (set_abs_time(fd, cinfo->col_buf[col])) {
-      cinfo->col_expr.col_expr[col] = "frame.time";
-      g_strlcpy(cinfo->col_expr.col_expr_val[col],cinfo->col_buf[col],COL_MAX_LEN);
-  }
+  set_abs_time(fd, cinfo->col_buf[col], TRUE);
+  cinfo->col_expr.col_expr[col] = "frame.time";
+  g_strlcpy(cinfo->col_expr.col_expr_val[col],cinfo->col_buf[col],COL_MAX_LEN);
+
   cinfo->col_data[col] = cinfo->col_buf[col];
 }
 
-/* ------------------------ */
-static gint
-set_epoch_time(frame_data *fd, gchar *buf)
+static void
+col_set_utc_time(const frame_data *fd, column_info *cinfo, const int col)
 {
+  set_abs_time(fd, cinfo->col_buf[col], FALSE);
+  cinfo->col_expr.col_expr[col] = "frame.time";
+  g_strlcpy(cinfo->col_expr.col_expr_val[col],cinfo->col_buf[col],COL_MAX_LEN);
 
-  COL_CHECK_REF_TIME(fd, buf);
+  cinfo->col_data[col] = cinfo->col_buf[col];
+}
 
+static gboolean
+set_epoch_time(const frame_data *fd, gchar *buf)
+{
+  if (!fd->flags.has_ts) {
+    buf[0] = '\0';
+    return FALSE;
+  }
   switch(timestamp_get_precision()) {
-      case(TS_PREC_FIXED_SEC):
-      case(TS_PREC_AUTO_SEC):
+      case TS_PREC_FIXED_SEC:
+      case TS_PREC_AUTO_SEC:
           display_epoch_time(buf, COL_MAX_LEN,
-            fd->abs_ts.secs, fd->abs_ts.nsecs / 1000000000, SECS);
+            fd->abs_ts.secs, fd->abs_ts.nsecs / 1000000000, TO_STR_TIME_RES_T_SECS);
           break;
-      case(TS_PREC_FIXED_DSEC):
-      case(TS_PREC_AUTO_DSEC):
+      case TS_PREC_FIXED_DSEC:
+      case TS_PREC_AUTO_DSEC:
           display_epoch_time(buf, COL_MAX_LEN,
-            fd->abs_ts.secs, fd->abs_ts.nsecs / 100000000, DSECS);
+            fd->abs_ts.secs, fd->abs_ts.nsecs / 100000000, TO_STR_TIME_RES_T_DSECS);
           break;
-      case(TS_PREC_FIXED_CSEC):
-      case(TS_PREC_AUTO_CSEC):
+      case TS_PREC_FIXED_CSEC:
+      case TS_PREC_AUTO_CSEC:
           display_epoch_time(buf, COL_MAX_LEN,
-            fd->abs_ts.secs, fd->abs_ts.nsecs / 10000000, CSECS);
+            fd->abs_ts.secs, fd->abs_ts.nsecs / 10000000, TO_STR_TIME_RES_T_CSECS);
           break;
-      case(TS_PREC_FIXED_MSEC):
-      case(TS_PREC_AUTO_MSEC):
+      case TS_PREC_FIXED_MSEC:
+      case TS_PREC_AUTO_MSEC:
           display_epoch_time(buf, COL_MAX_LEN,
-            fd->abs_ts.secs, fd->abs_ts.nsecs / 1000000, MSECS);
+            fd->abs_ts.secs, fd->abs_ts.nsecs / 1000000, TO_STR_TIME_RES_T_MSECS);
           break;
-      case(TS_PREC_FIXED_USEC):
-      case(TS_PREC_AUTO_USEC):
+      case TS_PREC_FIXED_USEC:
+      case TS_PREC_AUTO_USEC:
           display_epoch_time(buf, COL_MAX_LEN,
-            fd->abs_ts.secs, fd->abs_ts.nsecs / 1000, USECS);
+            fd->abs_ts.secs, fd->abs_ts.nsecs / 1000, TO_STR_TIME_RES_T_USECS);
           break;
-      case(TS_PREC_FIXED_NSEC):
-      case(TS_PREC_AUTO_NSEC):
+      case TS_PREC_FIXED_NSEC:
+      case TS_PREC_AUTO_NSEC:
           display_epoch_time(buf, COL_MAX_LEN,
-            fd->abs_ts.secs, fd->abs_ts.nsecs, NSECS);
+            fd->abs_ts.secs, fd->abs_ts.nsecs, TO_STR_TIME_RES_T_NSECS);
           break;
       default:
           g_assert_not_reached();
   }
-  return 1;
+  return TRUE;
 }
 
 static void
-col_set_epoch_time(frame_data *fd, column_info *cinfo, int col)
+col_set_epoch_time(const frame_data *fd, column_info *cinfo, const int col)
 {
-
   if (set_epoch_time(fd, cinfo->col_buf[col])) {
     cinfo->col_expr.col_expr[col] = "frame.time_delta";
     g_strlcpy(cinfo->col_expr.col_expr_val[col],cinfo->col_buf[col],COL_MAX_LEN);
@@ -995,51 +1142,99 @@ col_set_epoch_time(frame_data *fd, column_info *cinfo, int col)
   cinfo->col_data[col] = cinfo->col_buf[col];
 }
 
-/* Set the format of the variable time format.
-   XXX - this is called from "file.c" when the user changes the time
-   format they want for "command-line-specified" time; it's a bit ugly
-   that we have to export it, but if we go to a CList-like widget that
-   invokes callbacks to get the text for the columns rather than
-   requiring us to stuff the text into the widget from outside, we
-   might be able to clean this up. */
 void
-set_cls_time(frame_data *fd, gchar *buf)
+set_fd_time(frame_data *fd, gchar *buf)
 {
+
   switch (timestamp_get_type()) {
     case TS_ABSOLUTE:
-      set_abs_time(fd, buf);
+      set_abs_time(fd, buf, TRUE);
       break;
 
     case TS_ABSOLUTE_WITH_DATE:
-      set_abs_date_time(fd, buf);
+      set_abs_date_time(fd, buf, TRUE);
       break;
 
     case TS_RELATIVE:
-      set_rel_time(fd, buf);
+      if (fd->flags.has_ts) {
+        switch (timestamp_get_seconds_type()) {
+        case TS_SECONDS_DEFAULT:
+          set_time_seconds(&fd->rel_ts, buf);
+          break;
+        case TS_SECONDS_HOUR_MIN_SEC:
+          set_time_seconds(&fd->rel_ts, buf);
+          break;
+        default:
+          g_assert_not_reached();
+        }
+      } else {
+        buf[0] = '\0';
+      }
       break;
 
     case TS_DELTA:
-      set_delta_time(fd, buf);
+      if (fd->flags.has_ts) {
+        nstime_t del_cap_ts;
+
+        frame_delta_abs_time(fd, fd->prev_cap, &del_cap_ts);
+
+        switch (timestamp_get_seconds_type()) {
+        case TS_SECONDS_DEFAULT:
+          set_time_seconds(&del_cap_ts, buf);
+          break;
+        case TS_SECONDS_HOUR_MIN_SEC:
+          set_time_hour_min_sec(&del_cap_ts, buf);
+          break;
+        default:
+          g_assert_not_reached();
+        }
+      } else {
+        buf[0] = '\0';
+      }
       break;
 
     case TS_DELTA_DIS:
-      set_delta_time_dis(fd, buf);
+      if (fd->flags.has_ts) {
+        nstime_t del_dis_ts;
+
+        frame_delta_abs_time(fd, fd->prev_dis, &del_dis_ts);
+
+        switch (timestamp_get_seconds_type()) {
+        case TS_SECONDS_DEFAULT:
+          set_time_seconds(&del_dis_ts, buf);
+          break;
+        case TS_SECONDS_HOUR_MIN_SEC:
+          set_time_hour_min_sec(&del_dis_ts, buf);
+          break;
+        default:
+          g_assert_not_reached();
+        }
+      } else {
+        buf[0] = '\0';
+      }
       break;
 
     case TS_EPOCH:
       set_epoch_time(fd, buf);
       break;
 
+    case TS_UTC:
+      set_abs_time(fd, buf, FALSE);
+      break;
+
+    case TS_UTC_WITH_DATE:
+      set_abs_date_time(fd, buf, FALSE);
+      break;
+
     case TS_NOT_SET:
-    /* code is missing for this case, but I don't know which [jmayer20051219] */
-    g_assert(FALSE);
-        break;
+      /* code is missing for this case, but I don't know which [jmayer20051219] */
+      g_assert(FALSE);
+      break;
   }
 }
 
-
 static void
-col_set_cls_time(frame_data *fd, column_info *cinfo, gint col)
+col_set_cls_time(const frame_data *fd, column_info *cinfo, const gint col)
 {
   switch (timestamp_get_type()) {
     case TS_ABSOLUTE:
@@ -1066,6 +1261,14 @@ col_set_cls_time(frame_data *fd, column_info *cinfo, gint col)
       col_set_epoch_time(fd, cinfo, col);
       break;
 
+    case TS_UTC:
+      col_set_utc_time(fd, cinfo, col);
+      break;
+
+    case TS_UTC_WITH_DATE:
+      col_set_utc_date_time(fd, cinfo, col);
+      break;
+
     case TS_NOT_SET:
       /* code is missing for this case, but I don't know which [jmayer20051219] */
       g_assert_not_reached();
@@ -1073,19 +1276,15 @@ col_set_cls_time(frame_data *fd, column_info *cinfo, gint col)
   }
 }
 
-/* Set the format of the variable time format.
-   XXX - this is called from "file.c" when the user changes the time
-   format they want for "command-line-specified" time; it's a bit ugly
-   that we have to export it, but if we go to a CList-like widget that
-   invokes callbacks to get the text for the columns rather than
-   requiring us to stuff the text into the widget from outside, we
-   might be able to clean this up. */
-void
-col_set_fmt_time(frame_data *fd, column_info *cinfo, gint fmt, gint col)
+/* Set the format of the variable time format. */
+static void
+col_set_fmt_time(const frame_data *fd, column_info *cinfo, const gint fmt, const gint col)
 {
+  COL_CHECK_REF_TIME(fd, cinfo->col_buf[col]);
+
   switch (fmt) {
     case COL_CLS_TIME:
-       col_set_cls_time(fd, cinfo, col);
+      col_set_cls_time(fd, cinfo, col);
       break;
 
     case COL_ABS_TIME:
@@ -1108,6 +1307,14 @@ col_set_fmt_time(frame_data *fd, column_info *cinfo, gint fmt, gint col)
       col_set_delta_time_dis(fd, cinfo, col);
       break;
 
+    case COL_UTC_TIME:
+      col_set_utc_time(fd, cinfo, col);
+      break;
+
+    case COL_UTC_DATE_TIME:
+      col_set_utc_date_time(fd, cinfo, col);
+      break;
+
     default:
       g_assert_not_reached();
       break;
@@ -1115,46 +1322,61 @@ col_set_fmt_time(frame_data *fd, column_info *cinfo, gint fmt, gint col)
 }
 
 /* --------------------------- */
+/* Set the given (relative) time to a column element.
+ *
+ * Used by multiple dissectors to set the time in the column
+ * COL_DELTA_CONV_TIME
+ *
+ * @param cinfo                the current packet row
+ * @param col          the column to use, e.g. COL_INFO
+ * @param ts           the time to set in the column
+ * @param fieldname    the fieldname to use for creating a filter (when
+ *                       applying/preparing/copying as filter)
+ */
 void
-col_set_time(column_info *cinfo, gint el, nstime_t *ts, char *fieldname)
+col_set_time(column_info *cinfo, const gint el, const nstime_t *ts, char *fieldname)
 {
-  int   col;
+  int col;
 
   if (!CHECK_COL(cinfo, el))
     return;
 
+  /** @todo TODO: We don't respect fd->flags.ref_time (no way to access 'fd')
+  COL_CHECK_REF_TIME(fd, buf);
+  */
+
   for (col = cinfo->col_first[el]; col <= cinfo->col_last[el]; col++) {
     if (cinfo->fmt_matx[col][el]) {
       switch(timestamp_get_precision()) {
-    case(TS_PREC_FIXED_SEC):
-    case(TS_PREC_AUTO_SEC):
+    case TS_PREC_FIXED_SEC:
+    case TS_PREC_AUTO_SEC:
       display_signed_time(cinfo->col_buf[col], COL_MAX_LEN,
-        (gint32) ts->secs, ts->nsecs / 1000000000, SECS);
+        (gint32) ts->secs, ts->nsecs / 1000000000, TO_STR_TIME_RES_T_SECS);
       break;
-    case(TS_PREC_FIXED_DSEC):
-    case(TS_PREC_AUTO_DSEC):
+    case TS_PREC_FIXED_DSEC:
+    case TS_PREC_AUTO_DSEC:
       display_signed_time(cinfo->col_buf[col], COL_MAX_LEN,
-        (gint32) ts->secs, ts->nsecs / 100000000, DSECS);
+        (gint32) ts->secs, ts->nsecs / 100000000, TO_STR_TIME_RES_T_DSECS);
       break;
-    case(TS_PREC_FIXED_CSEC):
-    case(TS_PREC_AUTO_CSEC):
+    case TS_PREC_FIXED_CSEC:
+    case TS_PREC_AUTO_CSEC:
       display_signed_time(cinfo->col_buf[col], COL_MAX_LEN,
-        (gint32) ts->secs, ts->nsecs / 10000000, CSECS);
+        (gint32) ts->secs, ts->nsecs / 10000000, TO_STR_TIME_RES_T_CSECS);
       break;
-    case(TS_PREC_FIXED_MSEC):
-    case(TS_PREC_AUTO_MSEC):
+    case TS_PREC_FIXED_MSEC:
+    case TS_PREC_AUTO_MSEC:
       display_signed_time(cinfo->col_buf[col], COL_MAX_LEN,
-        (gint32) ts->secs, ts->nsecs / 1000000, MSECS);
+        (gint32) ts->secs, ts->nsecs / 1000000, TO_STR_TIME_RES_T_MSECS);
       break;
-    case(TS_PREC_FIXED_USEC):
-    case(TS_PREC_AUTO_USEC):
+    case TS_PREC_FIXED_USEC:
+    case TS_PREC_AUTO_USEC:
       display_signed_time(cinfo->col_buf[col], COL_MAX_LEN,
-        (gint32) ts->secs, ts->nsecs / 1000, USECS);
+        (gint32) ts->secs, ts->nsecs / 1000, TO_STR_TIME_RES_T_USECS);
       break;
-    case(TS_PREC_FIXED_NSEC):
-    case(TS_PREC_AUTO_NSEC):
+    case TS_PREC_FIXED_NSEC:
+    case TS_PREC_AUTO_NSEC:
       display_signed_time(cinfo->col_buf[col], COL_MAX_LEN,
-        (gint32) ts->secs, ts->nsecs, NSECS);
+        (gint32) ts->secs, ts->nsecs, TO_STR_TIME_RES_T_NSECS);
       break;
     default:
       g_assert_not_reached();
@@ -1167,29 +1389,45 @@ col_set_time(column_info *cinfo, gint el, nstime_t *ts, char *fieldname)
 }
 
 static void
-col_set_addr(packet_info *pinfo, int col, address *addr, gboolean is_src)
+col_set_addr(packet_info *pinfo, const int col, const address *addr, const gboolean is_src, const gboolean fill_col_exprs)
 {
-  struct e_in6_addr ipv6_addr;
-
-  if (addr->type == AT_NONE)
+  if (addr->type == AT_NONE) {
     /* No address, nothing to do */
     return;
+  }
 
-#ifdef NEW_PACKET_LIST
   pinfo->cinfo->col_data[col] = se_get_addr_name(addr);
-#else
-  get_addr_name_buf(addr, pinfo->cinfo->col_buf[col], COL_MAX_LEN);
-  pinfo->cinfo->col_data[col] = pinfo->cinfo->col_buf[col];
-#endif
+
+  if (!fill_col_exprs)
+    return;
 
   switch (addr->type) {
 
-  case AT_ETHER:
+  case AT_AX25:
     if (is_src)
-      pinfo->cinfo->col_expr.col_expr[col] = "eth.src";
+      pinfo->cinfo->col_expr.col_expr[col] = "ax25.src";
     else
-      pinfo->cinfo->col_expr.col_expr[col] = "eth.dst";
-    g_strlcpy(pinfo->cinfo->col_expr.col_expr_val[col], ether_to_str(addr->data), COL_MAX_LEN);
+      pinfo->cinfo->col_expr.col_expr[col] = "ax25.dst";
+    g_strlcpy(pinfo->cinfo->col_expr.col_expr_val[col], ax25_to_str(addr->data), COL_MAX_LEN);
+    break;
+
+  case AT_ETHER:
+    switch(addr->subtype) {
+    default:
+      if (is_src)
+        pinfo->cinfo->col_expr.col_expr[col] = "eth.src";
+      else
+        pinfo->cinfo->col_expr.col_expr[col] = "eth.dst";
+      address_to_str_buf(addr, pinfo->cinfo->col_expr.col_expr_val[col], COL_MAX_LEN);
+      break;
+    case AT_SUB_IEEE80211:
+      if (is_src)
+        pinfo->cinfo->col_expr.col_expr[col] = "wlan.sa";
+      else
+        pinfo->cinfo->col_expr.col_expr[col] = "wlan.da";
+      address_to_str_buf(addr, pinfo->cinfo->col_expr.col_expr_val[col], COL_MAX_LEN);
+      break;
+    }
     break;
 
   case AT_IPv4:
@@ -1197,7 +1435,7 @@ col_set_addr(packet_info *pinfo, int col, address *addr, gboolean is_src)
       pinfo->cinfo->col_expr.col_expr[col] = "ip.src";
     else
       pinfo->cinfo->col_expr.col_expr[col] = "ip.dst";
-    g_strlcpy(pinfo->cinfo->col_expr.col_expr_val[col], ip_to_str(addr->data), COL_MAX_LEN);
+    ip_to_str_buf(addr->data, pinfo->cinfo->col_expr.col_expr_val[col], COL_MAX_LEN);
     break;
 
   case AT_IPv6:
@@ -1205,8 +1443,7 @@ col_set_addr(packet_info *pinfo, int col, address *addr, gboolean is_src)
       pinfo->cinfo->col_expr.col_expr[col] = "ipv6.src";
     else
       pinfo->cinfo->col_expr.col_expr[col] = "ipv6.dst";
-    memcpy(&ipv6_addr.bytes, addr->data, sizeof ipv6_addr.bytes);
-    g_strlcpy(pinfo->cinfo->col_expr.col_expr_val[col], ip6_to_str(&ipv6_addr), COL_MAX_LEN);
+    address_to_str_buf(addr, pinfo->cinfo->col_expr.col_expr_val[col], COL_MAX_LEN);
     break;
 
   case AT_ATALK:
@@ -1240,7 +1477,7 @@ col_set_addr(packet_info *pinfo, int col, address *addr, gboolean is_src)
 
 /* ------------------------ */
 static void
-col_set_port(packet_info *pinfo, int col, gboolean is_res, gboolean is_src)
+col_set_port(packet_info *pinfo, const int col, const gboolean is_res, const gboolean is_src, const gboolean fill_col_exprs _U_)
 {
   guint32 port;
 
@@ -1249,16 +1486,18 @@ col_set_port(packet_info *pinfo, int col, gboolean is_res, gboolean is_src)
   else
     port = pinfo->destport;
 
+  /* TODO: Use fill_col_exprs */
+
   switch (pinfo->ptype) {
   case PT_SCTP:
     if (is_res)
       g_strlcpy(pinfo->cinfo->col_buf[col], get_sctp_port(port), COL_MAX_LEN);
     else
-      g_snprintf(pinfo->cinfo->col_buf[col], COL_MAX_LEN, "%u", port);
+      guint32_to_str_buf(port, pinfo->cinfo->col_buf[col], COL_MAX_LEN);
     break;
 
   case PT_TCP:
-    g_snprintf(pinfo->cinfo->col_expr.col_expr_val[col], COL_MAX_LEN, "%u", port);
+    guint32_to_str_buf(port, pinfo->cinfo->col_expr.col_expr_val[col], COL_MAX_LEN);
     if (is_res)
       g_strlcpy(pinfo->cinfo->col_buf[col], get_tcp_port(port), COL_MAX_LEN);
     else
@@ -1270,7 +1509,7 @@ col_set_port(packet_info *pinfo, int col, gboolean is_res, gboolean is_src)
     break;
 
   case PT_UDP:
-    g_snprintf(pinfo->cinfo->col_expr.col_expr_val[col], COL_MAX_LEN, "%u", port);
+    guint32_to_str_buf(port, pinfo->cinfo->col_expr.col_expr_val[col], COL_MAX_LEN);
     if (is_res)
       g_strlcpy(pinfo->cinfo->col_buf[col], get_udp_port(port), COL_MAX_LEN);
     else
@@ -1286,7 +1525,7 @@ col_set_port(packet_info *pinfo, int col, gboolean is_res, gboolean is_src)
       pinfo->cinfo->col_expr.col_expr[col] = "ddp.src_socket";
     else
       pinfo->cinfo->col_expr.col_expr[col] = "ddp.dst_socket";
-    g_snprintf(pinfo->cinfo->col_expr.col_expr_val[col], COL_MAX_LEN, "%u", port);
+    guint32_to_str_buf(port, pinfo->cinfo->col_expr.col_expr_val[col], COL_MAX_LEN);
     g_strlcpy(pinfo->cinfo->col_buf[col], pinfo->cinfo->col_expr.col_expr_val[col], COL_MAX_LEN);
     break;
 
@@ -1327,14 +1566,19 @@ col_set_port(packet_info *pinfo, int col, gboolean is_res, gboolean is_src)
 }
 
 gboolean
-col_based_on_frame_data(column_info *cinfo, gint col)
+col_based_on_frame_data(column_info *cinfo, const gint col)
 {
+    g_assert(cinfo);
+    g_assert(col < cinfo->num_cols);
+
     switch (cinfo->col_fmt[col]) {
 
     case COL_NUMBER:
     case COL_CLS_TIME:
     case COL_ABS_TIME:
     case COL_ABS_DATE_TIME:
+    case COL_UTC_TIME:
+    case COL_UTC_DATE_TIME:
     case COL_REL_TIME:
     case COL_DELTA_TIME:
     case COL_DELTA_TIME_DIS:
@@ -1348,36 +1592,68 @@ col_based_on_frame_data(column_info *cinfo, gint col)
 }
 
 void
-col_fill_in_frame_data(frame_data *fd, column_info *cinfo, gint col)
+col_fill_in_frame_data(const frame_data *fd, column_info *cinfo, const gint col, const gboolean fill_col_exprs)
 {
     switch (cinfo->col_fmt[col]) {
 
     case COL_NUMBER:
-      g_snprintf(cinfo->col_buf[col], COL_MAX_LEN, "%u", fd->num);
+      guint32_to_str_buf(fd->num, cinfo->col_buf[col], COL_MAX_LEN);
       cinfo->col_data[col] = cinfo->col_buf[col];
-      cinfo->col_expr.col_expr[col] = "frame.number";
-      g_strlcpy(cinfo->col_expr.col_expr_val[col], cinfo->col_buf[col], COL_MAX_LEN);
       break;
 
     case COL_CLS_TIME:
     case COL_ABS_TIME:
     case COL_ABS_DATE_TIME:
+    case COL_UTC_TIME:
+    case COL_UTC_DATE_TIME:
     case COL_REL_TIME:
     case COL_DELTA_TIME:
     case COL_DELTA_TIME_DIS:
+       /* TODO: Pass on fill_col_exprs */
       col_set_fmt_time(fd, cinfo, cinfo->col_fmt[col], col);
       break;
 
     case COL_PACKET_LENGTH:
-      g_snprintf(cinfo->col_buf[col], COL_MAX_LEN, "%u", fd->pkt_len);
+      guint32_to_str_buf(fd->pkt_len, cinfo->col_buf[col], COL_MAX_LEN);
       cinfo->col_data[col] = cinfo->col_buf[col];
+      break;
+
+    case COL_CUMULATIVE_BYTES:
+      guint32_to_str_buf(fd->cum_bytes, cinfo->col_buf[col], COL_MAX_LEN);
+      cinfo->col_data[col] = cinfo->col_buf[col];
+      break;
+
+    default:
+      break;
+    }
+
+    if (!fill_col_exprs)
+        return;
+
+    switch (cinfo->col_fmt[col]) {
+
+    case COL_NUMBER:
+      cinfo->col_expr.col_expr[col] = "frame.number";
+      g_strlcpy(cinfo->col_expr.col_expr_val[col], cinfo->col_buf[col], COL_MAX_LEN);
+      break;
+
+    case COL_CLS_TIME:
+    case COL_ABS_TIME:
+    case COL_ABS_DATE_TIME:
+    case COL_UTC_TIME:
+    case COL_UTC_DATE_TIME:
+    case COL_REL_TIME:
+    case COL_DELTA_TIME:
+    case COL_DELTA_TIME_DIS:
+      /* Already handled above */
+      break;
+
+    case COL_PACKET_LENGTH:
       cinfo->col_expr.col_expr[col] = "frame.len";
       g_strlcpy(cinfo->col_expr.col_expr_val[col], cinfo->col_buf[col], COL_MAX_LEN);
       break;
 
     case COL_CUMULATIVE_BYTES:
-      g_snprintf(cinfo->col_buf[col], COL_MAX_LEN, "%u", fd->cum_bytes);
-      cinfo->col_data[col] = cinfo->col_buf[col];
       break;
 
     default:
@@ -1386,7 +1662,7 @@ col_fill_in_frame_data(frame_data *fd, column_info *cinfo, gint col)
 }
 
 void
-col_fill_in(packet_info *pinfo, gboolean fill_fd_colums)
+col_fill_in(packet_info *pinfo, const gboolean fill_col_exprs, const gboolean fill_fd_colums)
 {
   int i;
 
@@ -1400,121 +1676,159 @@ col_fill_in(packet_info *pinfo, gboolean fill_fd_colums)
     case COL_CLS_TIME:
     case COL_ABS_TIME:
     case COL_ABS_DATE_TIME:
+    case COL_UTC_TIME:
+    case COL_UTC_DATE_TIME:
     case COL_REL_TIME:
     case COL_DELTA_TIME:
     case COL_DELTA_TIME_DIS:
     case COL_PACKET_LENGTH:
     case COL_CUMULATIVE_BYTES:
       if (fill_fd_colums)
-        col_fill_in_frame_data(pinfo->fd, pinfo->cinfo, i);
+        col_fill_in_frame_data(pinfo->fd, pinfo->cinfo, i, fill_col_exprs);
       break;
 
     case COL_DEF_SRC:
     case COL_RES_SRC:   /* COL_DEF_SRC is currently just like COL_RES_SRC */
-      col_set_addr(pinfo, i, &pinfo->src, TRUE);
+      col_set_addr(pinfo, i, &pinfo->src, TRUE, fill_col_exprs);
       break;
 
     case COL_UNRES_SRC:
-      col_set_addr(pinfo, i, &pinfo->src, TRUE);
+      col_set_addr(pinfo, i, &pinfo->src, TRUE, fill_col_exprs);
       break;
 
     case COL_DEF_DL_SRC:
     case COL_RES_DL_SRC:
-      col_set_addr(pinfo, i, &pinfo->dl_src, TRUE);
+      col_set_addr(pinfo, i, &pinfo->dl_src, TRUE, fill_col_exprs);
       break;
 
     case COL_UNRES_DL_SRC:
-      col_set_addr(pinfo, i, &pinfo->dl_src, TRUE);
+      col_set_addr(pinfo, i, &pinfo->dl_src, TRUE, fill_col_exprs);
       break;
 
     case COL_DEF_NET_SRC:
     case COL_RES_NET_SRC:
-      col_set_addr(pinfo, i, &pinfo->net_src, TRUE);
+      col_set_addr(pinfo, i, &pinfo->net_src, TRUE, fill_col_exprs);
       break;
 
     case COL_UNRES_NET_SRC:
-      col_set_addr(pinfo, i, &pinfo->net_src, TRUE);
+      col_set_addr(pinfo, i, &pinfo->net_src, TRUE, fill_col_exprs);
       break;
 
     case COL_DEF_DST:
     case COL_RES_DST:   /* COL_DEF_DST is currently just like COL_RES_DST */
-      col_set_addr(pinfo, i, &pinfo->dst, FALSE);
+      col_set_addr(pinfo, i, &pinfo->dst, FALSE, fill_col_exprs);
       break;
 
     case COL_UNRES_DST:
-      col_set_addr(pinfo, i, &pinfo->dst, FALSE);
+      col_set_addr(pinfo, i, &pinfo->dst, FALSE, fill_col_exprs);
       break;
 
     case COL_DEF_DL_DST:
     case COL_RES_DL_DST:
-      col_set_addr(pinfo, i, &pinfo->dl_dst, FALSE);
+      col_set_addr(pinfo, i, &pinfo->dl_dst, FALSE, fill_col_exprs);
       break;
 
     case COL_UNRES_DL_DST:
-      col_set_addr(pinfo, i, &pinfo->dl_dst, FALSE);
+      col_set_addr(pinfo, i, &pinfo->dl_dst, FALSE, fill_col_exprs);
       break;
 
     case COL_DEF_NET_DST:
     case COL_RES_NET_DST:
-      col_set_addr(pinfo, i, &pinfo->net_dst, FALSE);
+      col_set_addr(pinfo, i, &pinfo->net_dst, FALSE, fill_col_exprs);
       break;
 
     case COL_UNRES_NET_DST:
-      col_set_addr(pinfo, i, &pinfo->net_dst, FALSE);
+      col_set_addr(pinfo, i, &pinfo->net_dst, FALSE, fill_col_exprs);
       break;
 
     case COL_DEF_SRC_PORT:
     case COL_RES_SRC_PORT:  /* COL_DEF_SRC_PORT is currently just like COL_RES_SRC_PORT */
-      col_set_port(pinfo, i, TRUE, TRUE);
+      col_set_port(pinfo, i, TRUE, TRUE, fill_col_exprs);
       break;
 
     case COL_UNRES_SRC_PORT:
-      col_set_port(pinfo, i, FALSE, TRUE);
+      col_set_port(pinfo, i, FALSE, TRUE, fill_col_exprs);
       break;
 
     case COL_DEF_DST_PORT:
     case COL_RES_DST_PORT:  /* COL_DEF_DST_PORT is currently just like COL_RES_DST_PORT */
-      col_set_port(pinfo, i, TRUE, FALSE);
+      col_set_port(pinfo, i, TRUE, FALSE, fill_col_exprs);
       break;
 
     case COL_UNRES_DST_PORT:
-      col_set_port(pinfo, i, FALSE, FALSE);
+      col_set_port(pinfo, i, FALSE, FALSE, fill_col_exprs);
       break;
 
     case COL_VSAN:
-      g_snprintf(pinfo->cinfo->col_buf[i], COL_MAX_LEN, "%u", pinfo->vsan);
+      guint32_to_str_buf(pinfo->vsan, pinfo->cinfo->col_buf[i], COL_MAX_LEN);
       pinfo->cinfo->col_data[i] = pinfo->cinfo->col_buf[i];
       break;
 
-    case COL_CUSTOM:    /* done by col_custom_set_edt() */
-      break;
-
-    case COL_PROTOCOL:  /* currently done by dissectors */
-    case COL_INFO:      /* currently done by dissectors */
+    case NUM_COL_FMTS:  /* keep compiler happy - shouldn't get here */
+      g_assert_not_reached();
       break;
-
-    case COL_IF_DIR:    /* currently done by dissectors */
+    default:
+      if (pinfo->cinfo->col_fmt[i] >= NUM_COL_FMTS) {
+        g_assert_not_reached();
+      }
+      /*
+       * Formatting handled by col_custom_set_edt() (COL_CUSTOM), expert.c
+       * (COL_EXPERT), or individual dissectors.
+       */
       break;
+    }
+  }
+}
 
-    case COL_DCE_CALL:  /* done by dcerpc */
-      break;
+/*
+ * Fill in columns if we got an error reading the packet.
+ * We set most columns to "???", and set the Info column to an error
+ * message.
+ */
+void
+col_fill_in_error(column_info *cinfo, frame_data *fdata, const gboolean fill_col_exprs, const gboolean fill_fd_colums)
+{
+  int i;
 
-    case COL_DCE_CTX:   /* done by dcerpc */
-      break;
+  if (!cinfo)
+    return;
 
-    case COL_8021Q_VLAN_ID: /* done by packet-nstrace.c and packet-vlan.c */
-      break;
+  for (i = 0; i < cinfo->num_cols; i++) {
+    switch (cinfo->col_fmt[i]) {
 
-    case COL_EXPERT:    /* done by expert.c */
+    case COL_NUMBER:
+    case COL_CLS_TIME:
+    case COL_ABS_TIME:
+    case COL_ABS_DATE_TIME:
+    case COL_UTC_TIME:
+    case COL_UTC_DATE_TIME:
+    case COL_REL_TIME:
+    case COL_DELTA_TIME:
+    case COL_DELTA_TIME_DIS:
+    case COL_PACKET_LENGTH:
+    case COL_CUMULATIVE_BYTES:
+      if (fill_fd_colums)
+        col_fill_in_frame_data(fdata, cinfo, i, fill_col_exprs);
       break;
 
-    case COL_FREQ_CHAN: /* done by radio dissectors */
+    case COL_INFO:
+      /* XXX - say more than this */
+      cinfo->col_data[i] = "Read error";
       break;
 
     case NUM_COL_FMTS:  /* keep compiler happy - shouldn't get here */
-    default:
       g_assert_not_reached();
       break;
+    default:
+      if (cinfo->col_fmt[i] >= NUM_COL_FMTS) {
+        g_assert_not_reached();
+      }
+      /*
+       * No dissection was done, and these columns are set as the
+       * result of the dissection, so....
+       */
+      cinfo->col_data[i] = "???";
+      break;
     }
   }
 }
@@ -1558,7 +1872,9 @@ col_fill_fdata(packet_info *pinfo)
     case COL_CUMULATIVE_BYTES: /* fd->cum_bytes */
     case COL_CLS_TIME:
     case COL_ABS_TIME:
-    case COL_ABS_DATE_TIME:    /* from fd structures */
+    case COL_ABS_DATE_TIME:
+    case COL_UTC_TIME:
+    case COL_UTC_DATE_TIME:  /* from fd structures */
     case COL_REL_TIME:
     case COL_DELTA_TIME:
     case COL_DELTA_TIME_DIS:
@@ -1672,7 +1988,7 @@ col_fill_fdata(packet_info *pinfo)
   }
 }
 
-/* XXX Gets/creates the text fro col_text in frame data */
+/* XXX Gets/creates the text from col_text in frame data */
 /* --------------------- */
 gchar *
 col_get_text(frame_data *fd, column_info *cinfo, gint col)
@@ -1689,17 +2005,23 @@ gchar  *ptr;
 
     switch (cinfo->col_fmt[col]) {
     case COL_NUMBER: /* frame number */
-      g_snprintf(buf, COL_MAX_LEN, "%u", fd->num);
+      guint32_to_str_buf(fd->num, buf, COL_MAX_LEN);
       break;
 
     case COL_CLS_TIME:
       set_cls_time(fd, buf);
       break;
     case COL_ABS_TIME:
-      set_abs_time(fd, buf);
+      set_abs_time(fd, buf, TRUE);
+      break;
+    case COL_UTC_TIME:
+      set_abs_time(fd, buf, FALSE);
       break;
     case COL_ABS_DATE_TIME:
-      set_abs_date_time(fd, buf);
+      set_abs_date_time(fd, buf, TRUE);
+      break;
+    case COL_UTC_DATE_TIME:
+      set_abs_date_time(fd, buf, FALSE);
       break;
     case COL_REL_TIME:
       set_rel_time(fd, buf);
@@ -1712,11 +2034,11 @@ gchar  *ptr;
       break;
 
     case COL_PACKET_LENGTH: /* fd->pkt_len */
-      g_snprintf(buf, COL_MAX_LEN, "%u", fd->pkt_len);
+      guint32_to_str_buf(fd->pkt_len, buf, COL_MAX_LEN);
       break;
 
     case COL_CUMULATIVE_BYTES: /* fd->cum_bytes */
-      g_snprintf(buf, COL_MAX_LEN, "%u", fd->cum_bytes);
+      guint32_to_str_buf(fd->cum_bytes, buf, COL_MAX_LEN);
       break;
 
     case COL_DEF_SRC:
@@ -1764,7 +2086,7 @@ gchar  *ptr;
     case COL_UNRES_DST_PORT:
       /* hack */
       if (GPOINTER_TO_UINT(fd->col_text[col]) <= 65536)
-          g_snprintf(buf, COL_MAX_LEN, "%u", GPOINTER_TO_UINT(fd->col_text[col]));
+          guint32_to_str_buf(GPOINTER_TO_UINT(fd->col_text[col], buf, COL_MAX_LEN));
       else
           ptr = fd->col_text[col];
       break;
@@ -1777,7 +2099,7 @@ gchar  *ptr;
       break;
 
     case COL_VSAN:
-      g_snprintf(buf, COL_MAX_LEN, "%u", GPOINTER_TO_UINT(fd->col_text[col]));
+      guint32_to_str_buf(GPOINTER_TO_UINT(fd->col_text[col]), buf, COL_MAX_LEN);
       break;
 
     case NUM_COL_FMTS:  /* keep compiler happy - shouldn't get here */