Replace "svn" with "git" all over the place.
[metze/wireshark/wip.git] / ui / gtk / summary_dlg.c
index 3722c4f4563b71a6f8407d74e8021e0bea5c7b00..920ece04e843b81df5516384850b18cc9f3e923e 100644 (file)
@@ -37,6 +37,8 @@
 #include "../file.h"
 #include "../summary.h"
 #include "../capture-pcap-util.h"
+#include "../version_info.h"
+
 #ifdef HAVE_LIBPCAP
 #include "../capture.h"
 #include "ui/capture_globals.h"
 #include "ui/gtk/dlg_utils.h"
 #include "ui/gtk/gui_utils.h"
 #include "ui/gtk/help_dlg.h"
+#include "ui/gtk/packet_list.h"
 
-#define SUM_STR_MAX     1024
-#define FILTER_SNIP_LEN 50
-#define SHB_STR_SNIP_LEN 50
+#define SUM_STR_MAX      1024
+#define FILTER_SNIP_LEN    50
+#define SHB_STR_SNIP_LEN   50
 
 static GtkWidget *summary_dlg = NULL;
+static GtkWidget *view_capture_and_pkt_comments_dlg = NULL;
+
 
 static void
-add_string_to_table_sensitive(GtkWidget *list, guint *row, const gchar *title, const gchar *value, gboolean sensitive)
+add_string_to_grid_sensitive(GtkWidget *grid, guint *row, const gchar *title, const gchar *value, gboolean sensitive)
 {
     GtkWidget *label;
     gchar     *indent;
@@ -74,20 +79,21 @@ add_string_to_table_sensitive(GtkWidget *list, guint *row, const gchar *title, c
     g_free(indent);
     gtk_misc_set_alignment(GTK_MISC(label), 0.0f, 0.5f);
     gtk_widget_set_sensitive(label, sensitive);
-    gtk_table_attach_defaults(GTK_TABLE(list), label, 0, 1, *row, *row+1);
+    ws_gtk_grid_attach_defaults(GTK_GRID(grid), label, 0, *row, 1, 1);
 
     label = gtk_label_new(value);
+    gtk_label_set_line_wrap(GTK_LABEL(label), TRUE);
     gtk_misc_set_alignment(GTK_MISC(label), 0.0f, 0.5f);
     gtk_widget_set_sensitive(label, sensitive);
-    gtk_table_attach_defaults(GTK_TABLE(list), label, 1, 2, *row, *row+1);
+    ws_gtk_grid_attach_defaults(GTK_GRID(grid), label, 1, *row, 1, 1);
 
     *row = *row + 1;
 }
 
 static void
-add_string_to_table(GtkWidget *list, guint *row, const gchar *title, const gchar *value)
+add_string_to_grid(GtkWidget *grid, guint *row, const gchar *title, const gchar *value)
 {
-    add_string_to_table_sensitive(list, row, title, value, TRUE);
+    add_string_to_grid_sensitive(grid, row, title, value, TRUE);
 }
 
 
@@ -111,14 +117,6 @@ time_to_string(char *string_buff, gulong string_buff_size, time_t ti_time)
 {
   struct tm *ti_tm;
 
-#ifdef _MSC_VER
-  /* calling localtime() 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 (ti_time > 2000000000) {
-      ti_tm = NULL;
-  } else
-#endif
   ti_tm = localtime(&ti_time);
   if (ti_tm == NULL) {
     g_snprintf(string_buff, string_buff_size, "Not representable");
@@ -142,18 +140,20 @@ summary_ok_cb(GtkWidget *w _U_, GtkWidget *view)
   GtkTextIter end_iter;
   gchar *new_comment = NULL;
 
-  buffer = gtk_text_view_get_buffer (GTK_TEXT_VIEW (view));
-  gtk_text_buffer_get_start_iter (buffer, &start_iter);
-  gtk_text_buffer_get_end_iter (buffer, &end_iter);
+  if (view != NULL) {
+    buffer = gtk_text_view_get_buffer (GTK_TEXT_VIEW (view));
+    gtk_text_buffer_get_start_iter (buffer, &start_iter);
+    gtk_text_buffer_get_end_iter (buffer, &end_iter);
 
-  new_comment = gtk_text_buffer_get_text (buffer, &start_iter, &end_iter, FALSE /* whether to include invisible text */);
+    new_comment = gtk_text_buffer_get_text (buffer, &start_iter, &end_iter, FALSE /* whether to include invisible text */);
 
-  cf_update_capture_comment(&cfile, new_comment);
+    cf_update_capture_comment(&cfile, new_comment);
 
-  /* Update the main window */
-  main_update_for_unsaved_changes(&cfile);
+    /* Update the main window */
+    main_update_for_unsaved_changes(&cfile);
 
-  status_capture_comment_update();
+    status_capture_comment_update();
+  }
 
   window_destroy(summary_dlg);
 }
@@ -168,16 +168,16 @@ summary_destroy_cb(GtkWidget *win _U_, gpointer user_data _U_)
 void
 summary_open_cb(GtkWidget *w _U_, gpointer d _U_)
 {
-  summary_tally summary;
-  GtkWidget     *main_vb, *bbox, *cancel_bt, *ok_bt, *help_bt;
-  GtkWidget     *table, *scrolled_window;
-  GtkWidget     *list, *treeview;
-  GtkWidget     *comment_view, *comment_frame, *comment_vbox;
-  GtkTextBuffer *buffer = NULL;
-  gchar *buf_str;
-  GtkListStore  *store;
-  GtkTreeIter    iter;
-  GtkCellRenderer *renderer;
+  summary_tally      summary;
+  GtkWidget         *main_vb, *bbox, *cancel_bt, *ok_bt, *help_bt;
+  GtkWidget         *grid, *scrolled_window;
+  GtkWidget         *list, *treeview;
+  GtkWidget         *comment_view = NULL, *comment_frame, *comment_vbox;
+  GtkTextBuffer     *buffer = NULL;
+  gchar             *buf_str;
+  GtkListStore      *store;
+  GtkTreeIter        iter;
+  GtkCellRenderer   *renderer;
   GtkTreeViewColumn *column;
   static const char *titles[] = {
     "Traffic", "Captured", "Displayed", "Displayed %", "Marked", "Marked %" };
@@ -191,11 +191,7 @@ summary_open_cb(GtkWidget *w _U_, gpointer d _U_)
   double        seconds;
   double        disp_seconds;
   double        marked_seconds;
-  guint         offset;
-  guint         snip;
   guint         row;
-  gchar        *str_dup;
-  gchar        *str_work;
 
   unsigned int  elapsed_time;
   iface_options iface;
@@ -225,78 +221,89 @@ summary_open_cb(GtkWidget *w _U_, gpointer d _U_)
   marked_seconds = summary.marked_stop - summary.marked_start;
 
   summary_dlg = window_new(GTK_WINDOW_TOPLEVEL, "Wireshark: Summary");
+  gtk_window_set_default_size(GTK_WINDOW(summary_dlg), 400, 750);
+
+  /* Scrolled Window to hold them all */
+  scrolled_window = gtk_scrolled_window_new(NULL, NULL);
+  gtk_scrolled_window_set_policy(GTK_SCROLLED_WINDOW(scrolled_window), GTK_POLICY_NEVER, GTK_POLICY_AUTOMATIC);
+  gtk_container_add(GTK_CONTAINER(summary_dlg), scrolled_window);
 
   /* Container for each row of widgets */
   main_vb = ws_gtk_box_new(GTK_ORIENTATION_VERTICAL, 12, FALSE);
   gtk_container_set_border_width(GTK_CONTAINER(main_vb), 12);
-  gtk_container_add(GTK_CONTAINER(summary_dlg), main_vb);
+#if ! GTK_CHECK_VERSION(3,8,0)
+  gtk_scrolled_window_add_with_viewport(GTK_SCROLLED_WINDOW(scrolled_window), main_vb);
+#else
+  gtk_container_add(GTK_CONTAINER(scrolled_window), main_vb);
+#endif
 
-  /* table */
-  table = gtk_table_new(1, 2, FALSE);
-  gtk_table_set_col_spacings(GTK_TABLE(table), 6);
-  gtk_table_set_row_spacings(GTK_TABLE(table), 3);
-  gtk_box_pack_start(GTK_BOX(main_vb), table, TRUE, TRUE, 0);
+  /* grid */
+  grid = ws_gtk_grid_new();
+  ws_gtk_grid_set_column_spacing(GTK_GRID(grid), 6);
+  ws_gtk_grid_set_row_spacing(GTK_GRID(grid), 3);
+  gtk_box_pack_start(GTK_BOX(main_vb), grid, TRUE, TRUE, 0);
   row = 0;
 
-
   /* File */
-  add_string_to_table(table, &row, "File", "");
+  add_string_to_grid(grid, &row, "File", "");
 
   /* filename */
   g_snprintf(string_buff, SUM_STR_MAX, "%s", summary.filename);
-  add_string_to_table(table, &row, "Name:", string_buff);
+  add_string_to_grid(grid, &row, "Name:", string_buff);
 
   /* length */
   g_snprintf(string_buff, SUM_STR_MAX, "%" G_GINT64_MODIFIER "d bytes",
              summary.file_length);
-  add_string_to_table(table, &row, "Length:", string_buff);
+  add_string_to_grid(grid, &row, "Length:", string_buff);
 
   /* format */
   g_snprintf(string_buff, SUM_STR_MAX, "%s%s",
-             wtap_file_type_string(summary.file_type),
+             wtap_file_type_subtype_string(summary.file_type),
              summary.iscompressed? " (gzip compressed)" : "");
-  add_string_to_table(table, &row, "Format:", string_buff);
+  add_string_to_grid(grid, &row, "Format:", string_buff);
 
   /* encapsulation */
   if (summary.file_encap_type == WTAP_ENCAP_PER_PACKET) {
     for (i = 0; i < summary.packet_encap_types->len; i++) {
       g_snprintf(string_buff, SUM_STR_MAX, "%s",
                  wtap_encap_string(g_array_index(summary.packet_encap_types, int, i)));
-      add_string_to_table(table, &row, (i == 0) ? "Encapsulation:" : "",
+      add_string_to_grid(grid, &row, (i == 0) ? "Encapsulation:" : "",
                           string_buff);
     }
   } else {
     g_snprintf(string_buff, SUM_STR_MAX, "%s", wtap_encap_string(summary.file_encap_type));
-    add_string_to_table(table, &row, "Encapsulation:", string_buff);
+    add_string_to_grid(grid, &row, "Encapsulation:", string_buff);
   }
   if (summary.has_snap) {
     /* snapshot length */
     g_snprintf(string_buff, SUM_STR_MAX, "%u bytes", summary.snap);
-    add_string_to_table(table, &row, "Packet size limit:", string_buff);
+    add_string_to_grid(grid, &row, "Packet size limit:", string_buff);
   }
 
   /* Capture file comment area */
-  comment_frame = gtk_frame_new("Capture file comments");
-  gtk_frame_set_shadow_type(GTK_FRAME(comment_frame), GTK_SHADOW_ETCHED_IN);
-  gtk_box_pack_start(GTK_BOX(main_vb), comment_frame, TRUE, TRUE, 0);
-  gtk_widget_show(comment_frame);
-
-  comment_vbox = ws_gtk_box_new(GTK_ORIENTATION_VERTICAL, 0, FALSE);
-  gtk_container_add(GTK_CONTAINER(comment_frame), comment_vbox);
-  gtk_widget_show(comment_vbox);
-
-  comment_view = gtk_text_view_new();
-  gtk_text_view_set_wrap_mode(GTK_TEXT_VIEW(comment_view), GTK_WRAP_WORD);
-  buffer = gtk_text_view_get_buffer (GTK_TEXT_VIEW (comment_view));
-  if(summary.opt_comment == NULL) {
-    gtk_text_buffer_set_text (buffer, "", -1);
-  } else {
-    buf_str = g_strdup_printf("%s", summary.opt_comment);
-    gtk_text_buffer_set_text (buffer, buf_str, -1);
-    g_free(buf_str);
+  if (wtap_dump_can_write(cfile.linktypes, WTAP_COMMENT_PER_SECTION)) {
+    comment_frame = gtk_frame_new("Capture file comments");
+    gtk_frame_set_shadow_type(GTK_FRAME(comment_frame), GTK_SHADOW_ETCHED_IN);
+    gtk_box_pack_start(GTK_BOX(main_vb), comment_frame, TRUE, TRUE, 0);
+    gtk_widget_show(comment_frame);
+
+    comment_vbox = ws_gtk_box_new(GTK_ORIENTATION_VERTICAL, 0, FALSE);
+    gtk_container_add(GTK_CONTAINER(comment_frame), comment_vbox);
+    gtk_widget_show(comment_vbox);
+
+    comment_view = gtk_text_view_new();
+    gtk_text_view_set_wrap_mode(GTK_TEXT_VIEW(comment_view), GTK_WRAP_WORD);
+    buffer = gtk_text_view_get_buffer (GTK_TEXT_VIEW (comment_view));
+    if(summary.opt_comment == NULL) {
+      gtk_text_buffer_set_text (buffer, "", -1);
+    } else {
+      buf_str = g_strdup_printf("%s", summary.opt_comment);
+      gtk_text_buffer_set_text (buffer, buf_str, -1);
+      g_free(buf_str);
+    }
+    gtk_box_pack_start(GTK_BOX(comment_vbox), comment_view, TRUE, TRUE, 0);
+    gtk_widget_show (comment_view);
   }
-  gtk_box_pack_start(GTK_BOX(comment_vbox), comment_view, TRUE, TRUE, 0);
-  gtk_widget_show (comment_view);
 
   /*
    * We must have no un-time-stamped packets (i.e., the number of
@@ -307,16 +314,16 @@ summary_open_cb(GtkWidget *w _U_, gpointer d _U_)
   if (summary.packet_count_ts == summary.packet_count &&
       summary.packet_count >= 1) {
     /* Time */
-    add_string_to_table(table, &row, "", "");
-    add_string_to_table(table, &row, "Time", "");
+    add_string_to_grid(grid, &row, "", "");
+    add_string_to_grid(grid, &row, "Time", "");
 
     /* start time */
     time_to_string(string_buff, SUM_STR_MAX, (time_t)summary.start_time);
-    add_string_to_table(table, &row, "First packet:", string_buff);
+    add_string_to_grid(grid, &row, "First packet:", string_buff);
 
     /* stop time */
     time_to_string(string_buff, SUM_STR_MAX, (time_t)summary.stop_time);
-    add_string_to_table(table, &row, "Last packet:", string_buff);
+    add_string_to_grid(grid, &row, "Last packet:", string_buff);
 
     /*
      * We must have at least two time-stamped packets for the elapsed time
@@ -332,27 +339,27 @@ summary_open_cb(GtkWidget *w _U_, gpointer d _U_)
           g_snprintf(string_buff, SUM_STR_MAX, "%02u:%02u:%02u",
             elapsed_time%86400/3600, elapsed_time%3600/60, elapsed_time%60);
       }
-      add_string_to_table(table, &row, "Elapsed:", string_buff);
+      add_string_to_grid(grid, &row, "Elapsed:", string_buff);
     }
   }
 
   /* Capture */
-  add_string_to_table(table, &row, "", "");
-  add_string_to_table_sensitive(table, &row, "Capture", "", (summary.ifaces->len > 0));
+  add_string_to_grid(grid, &row, "", "");
+  add_string_to_grid_sensitive(grid, &row, "Capture", "", (summary.ifaces->len > 0));
   if(summary.shb_hardware){
     /* truncate the string to a reasonable length */
     g_snprintf(string_buff, SHB_STR_SNIP_LEN, "%s",summary.shb_hardware);
-    add_string_to_table(table, &row, "Capture HW:",string_buff);
+    add_string_to_grid(grid, &row, "Capture HW:",string_buff);
   }
   if(summary.shb_os){
     /* truncate the strings to a reasonable length */
     g_snprintf(string_buff, SHB_STR_SNIP_LEN, "%s",summary.shb_os);
-    add_string_to_table(table, &row, "OS:", string_buff);
+    add_string_to_grid(grid, &row, "OS:", string_buff);
   }
   if(summary.shb_user_appl){
     /* truncate the string to a reasonable length */
     g_snprintf(string_buff, SHB_STR_SNIP_LEN, "%s",summary.shb_user_appl);
-    add_string_to_table(table, &row, "Capture application:", string_buff);
+    add_string_to_grid(grid, &row, "Capture application:", string_buff);
   }
   scrolled_window = gtk_scrolled_window_new (NULL, NULL);
   gtk_container_set_border_width (GTK_CONTAINER (scrolled_window), 5);
@@ -415,45 +422,29 @@ summary_open_cb(GtkWidget *w _U_, gpointer d _U_)
   gtk_container_add(GTK_CONTAINER(scrolled_window), treeview);
   gtk_box_pack_start(GTK_BOX(main_vb), scrolled_window, TRUE, TRUE, 0);
   gtk_widget_show_all(scrolled_window);
-  table = gtk_table_new(1, 2, FALSE);
-  gtk_table_set_col_spacings(GTK_TABLE(table), 6);
-  gtk_table_set_row_spacings(GTK_TABLE(table), 3);
-  gtk_box_pack_start(GTK_BOX(main_vb), table, TRUE, TRUE, 0);
+  grid = ws_gtk_grid_new();
+  ws_gtk_grid_set_column_spacing(GTK_GRID(grid), 6);
+  ws_gtk_grid_set_row_spacing(GTK_GRID(grid), 3);
+  gtk_box_pack_start(GTK_BOX(main_vb), grid, TRUE, TRUE, 0);
   row = 0;
 
 
   /* Data */
-  add_string_to_table(table, &row, "", "");
-  add_string_to_table(table, &row, "Display", "");
+  add_string_to_grid(grid, &row, "", "");
+  add_string_to_grid(grid, &row, "Display", "");
 
   if (summary.dfilter) {
     /* Display filter */
-    /* limit each row to some reasonable length */
-    str_dup = g_strdup_printf("%s", summary.dfilter);
-    str_work = g_strdup(str_dup);
-    offset = 0;
-    snip = 0;
-    while(strlen(str_work) > FILTER_SNIP_LEN) {
-        str_work[FILTER_SNIP_LEN] = '\0';
-        add_string_to_table(table, &row, (snip == 0) ? "Display filter:" : "", str_work);
-        g_free(str_work);
-        offset+=FILTER_SNIP_LEN;
-        str_work = g_strdup(&str_dup[offset]);
-        snip++;
-    }
-
-    add_string_to_table(table, &row, (snip == 0) ? "Display filter:" : "", str_work);
-    g_free(str_work);
-    g_free(str_dup);
+    add_string_to_grid(grid, &row, "Display filter:", summary.dfilter);
   } else {
     /* Display filter */
-    add_string_to_table(table, &row, "Display filter:", "none");
+    add_string_to_grid(grid, &row, "Display filter:", "none");
   }
 
   /* Ignored packet count */
   g_snprintf(string_buff, SUM_STR_MAX, "%i (%.3f%%)", summary.ignored_count,
              summary.packet_count ? (100.0 * summary.ignored_count)/summary.packet_count : 0.0);
-  add_string_to_table(table, &row, "Ignored packets:", string_buff);
+  add_string_to_grid(grid, &row, "Ignored packets:", string_buff);
 
   /* Traffic */
   list = simple_list_new(6, titles);
@@ -641,15 +632,15 @@ summary_open_cb(GtkWidget *w _U_, gpointer d _U_)
   bbox = dlg_button_row_new(GTK_STOCK_CANCEL, GTK_STOCK_OK, GTK_STOCK_HELP, NULL);
   gtk_box_pack_start(GTK_BOX(main_vb), bbox, TRUE, TRUE, 0);
 
-  cancel_bt = g_object_get_data(G_OBJECT(bbox), GTK_STOCK_CANCEL);
+  cancel_bt = (GtkWidget *)g_object_get_data(G_OBJECT(bbox), GTK_STOCK_CANCEL);
   window_set_cancel_button(summary_dlg, cancel_bt, window_cancel_button_cb);
 
-  ok_bt = g_object_get_data(G_OBJECT(bbox), GTK_STOCK_OK);
+  ok_bt = (GtkWidget *)g_object_get_data(G_OBJECT(bbox), GTK_STOCK_OK);
   g_signal_connect (ok_bt, "clicked",
                     G_CALLBACK(summary_ok_cb), comment_view);
   gtk_widget_grab_focus(ok_bt);
 
-  help_bt = g_object_get_data(G_OBJECT(bbox), GTK_STOCK_HELP);
+  help_bt = (GtkWidget *)g_object_get_data(G_OBJECT(bbox), GTK_STOCK_HELP);
   g_signal_connect(help_bt, "clicked", G_CALLBACK(topic_cb), (gpointer)HELP_STATS_SUMMARY_DIALOG);
 
 
@@ -666,9 +657,12 @@ summary_to_texbuff(GtkTextBuffer *buffer)
 {
   summary_tally summary;
   gchar         string_buff[SUM_STR_MAX];
+  gchar         tmp_buff[SUM_STR_MAX];
   gchar *buf_str;
   unsigned int  i;
   unsigned int  elapsed_time;
+  iface_options iface;
+  double seconds;
 
   /* initial computations */
   summary_fill_in(&cfile, &summary);
@@ -676,36 +670,38 @@ summary_to_texbuff(GtkTextBuffer *buffer)
   summary_fill_in_capture(&cfile, &global_capture_opts, &summary);
 #endif
 
-  /* Add Wireshark version here? TODO */
+  /* Add Wireshark version*/
+  g_snprintf(string_buff, SUM_STR_MAX, "Summary created by Wireshark %s\n\n", wireshark_gitversion);
+  gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
 
   /* Info about file */
   g_snprintf(string_buff, SUM_STR_MAX, "File: \n");
   gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
 
   /* Filename */
-  g_snprintf(string_buff, SUM_STR_MAX, INDENT "Name:             %s\n", summary.filename);
+  g_snprintf(string_buff, SUM_STR_MAX, INDENT "Name: %s\n", summary.filename);
   gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
 
   /* length */
-  g_snprintf(string_buff, SUM_STR_MAX, INDENT "Length:            %" G_GINT64_MODIFIER "d bytes\n",
+  g_snprintf(string_buff, SUM_STR_MAX, INDENT "Length: %" G_GINT64_MODIFIER "d bytes\n",
              summary.file_length);
   gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
 
   /* format */
-  g_snprintf(string_buff, SUM_STR_MAX, INDENT "Format:            %s%s",
-             wtap_file_type_string(summary.file_type),
+  g_snprintf(string_buff, SUM_STR_MAX, INDENT "Format: %s%s",
+             wtap_file_type_subtype_string(summary.file_type),
              summary.iscompressed? " (gzip compressed)\n" : "\n");
   gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
 
   /* encapsulation */
   if (summary.file_encap_type == WTAP_ENCAP_PER_PACKET) {
     for (i = 0; i < summary.packet_encap_types->len; i++) {
-      g_snprintf(string_buff, SUM_STR_MAX, INDENT "Encapsulation:    %s\n",
+      g_snprintf(string_buff, SUM_STR_MAX, INDENT "Encapsulation: %s\n",
                  wtap_encap_string(g_array_index(summary.packet_encap_types, int, i)));
       gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
     }
   } else {
-    g_snprintf(string_buff, SUM_STR_MAX, INDENT "Encapsulation:    %s\n", wtap_encap_string(summary.file_encap_type));
+    g_snprintf(string_buff, SUM_STR_MAX, INDENT "Encapsulation: %s\n", wtap_encap_string(summary.file_encap_type));
     gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
   }
   if (summary.has_snap) {
@@ -726,13 +722,13 @@ summary_to_texbuff(GtkTextBuffer *buffer)
     gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
 
     /* start time */
-    time_to_string(string_buff, SUM_STR_MAX, (time_t)summary.start_time);
-    g_snprintf(string_buff, SUM_STR_MAX, INDENT "First packet: %s\n",string_buff);
+    time_to_string(tmp_buff, SUM_STR_MAX, (time_t)summary.start_time);
+    g_snprintf(string_buff, SUM_STR_MAX, INDENT "First packet: %s\n",tmp_buff);
     gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
 
     /* stop time */
-    time_to_string(string_buff, SUM_STR_MAX, (time_t)summary.stop_time);
-    g_snprintf(string_buff, SUM_STR_MAX, INDENT "Last packet: %s\n", string_buff);
+    time_to_string(tmp_buff, SUM_STR_MAX, (time_t)summary.stop_time);
+    g_snprintf(string_buff, SUM_STR_MAX, INDENT "Last packet: %s\n", tmp_buff);
     gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
 
     /*
@@ -743,14 +739,14 @@ summary_to_texbuff(GtkTextBuffer *buffer)
       /* elapsed seconds */
       elapsed_time = (unsigned int)summary.elapsed_time;
       if(elapsed_time/86400) {
-          g_snprintf(string_buff, SUM_STR_MAX, "%02u days %02u:%02u:%02u",
+          g_snprintf(tmp_buff, SUM_STR_MAX, "%02u days %02u:%02u:%02u",
             elapsed_time/86400, elapsed_time%86400/3600, elapsed_time%3600/60, elapsed_time%60);
       } else {
-          g_snprintf(string_buff, SUM_STR_MAX, "%02u:%02u:%02u",
+          g_snprintf(tmp_buff, SUM_STR_MAX, "%02u:%02u:%02u",
             elapsed_time%86400/3600, elapsed_time%3600/60, elapsed_time%60);
       }
-      g_snprintf(string_buff, SUM_STR_MAX, INDENT "Elapsed: %s\n", string_buff);
-         gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
+      g_snprintf(string_buff, SUM_STR_MAX, INDENT "Elapsed: %s\n", tmp_buff);
+      gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
     }
   }
 
@@ -761,6 +757,132 @@ summary_to_texbuff(GtkTextBuffer *buffer)
   g_snprintf(string_buff, SUM_STR_MAX, "Capture:\n");
   gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
 
+  if(summary.shb_hardware){
+    /* truncate the string to a reasonable length */
+    g_snprintf(string_buff, SUM_STR_MAX, INDENT "Capture HW: %s\n",summary.shb_hardware);
+    gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
+  }
+  if(summary.shb_os){
+    /* truncate the strings to a reasonable length */
+    g_snprintf(string_buff, SUM_STR_MAX, INDENT "OS: %s\n",summary.shb_os);
+    gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
+  }
+  if(summary.shb_user_appl){
+    /* truncate the string to a reasonable length */
+    g_snprintf(string_buff, SUM_STR_MAX, INDENT "Capture application: %s\n",summary.shb_user_appl);
+    gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
+  }
+
+  /* Add empty line */
+  g_snprintf(string_buff, SUM_STR_MAX, "\n");
+  gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
+
+  /* Interface info */
+  for (i = 0; i < summary.ifaces->len; i++) {
+    iface = g_array_index(summary.ifaces, iface_options, i);
+    /* interface */
+    if (iface.descr) {
+      g_snprintf(string_buff, SUM_STR_MAX, INDENT "%s:\n", iface.descr);
+    } else if (iface.name) {
+      g_snprintf(string_buff, SUM_STR_MAX, INDENT "%s:\n", iface.name);
+    } else {
+      g_snprintf(string_buff, SUM_STR_MAX, INDENT "Unknown interface:\n");
+    }
+    gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
+
+    /* Dropped count */
+    if (iface.drops_known) {
+      g_snprintf(string_buff, SUM_STR_MAX, INDENT INDENT "Dropped packets: %" G_GINT64_MODIFIER "u (%.3f%%)\n",
+                 iface.drops, summary.packet_count ? (100.0 * iface.drops)/summary.packet_count : 0.0);
+    } else {
+      g_snprintf(string_buff, SUM_STR_MAX, INDENT INDENT "Dropped packets: unknown\n");
+    }
+    gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
+
+    /* Capture filter */
+    if (iface.cfilter && iface.cfilter[0] != '\0') {
+      g_snprintf(string_buff, SUM_STR_MAX, INDENT INDENT "Capture filter: %s\n", iface.cfilter);
+    } else {
+      if (iface.name) {
+        g_snprintf(string_buff, SUM_STR_MAX, INDENT INDENT "Capture filter: none\n");
+      } else {
+        g_snprintf(string_buff, SUM_STR_MAX, INDENT INDENT "Capture filter: unknown\n");
+      }
+    }
+    gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
+
+    g_snprintf(string_buff, SUM_STR_MAX, INDENT INDENT "Link type: %s\n", wtap_encap_string(iface.encap_type));
+    gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
+
+    g_snprintf(string_buff, SUM_STR_MAX, INDENT INDENT "Packet size limit %u bytes\n", iface.snap);
+    gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
+
+  }
+
+  /* Add empty line */
+  g_snprintf(string_buff, SUM_STR_MAX, "\n");
+  gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
+
+  /* Statistics */
+  g_snprintf(string_buff, SUM_STR_MAX, "Statistics:\n");
+  gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
+
+  /* Packet count */
+  g_snprintf(cap_buf, SUM_STR_MAX, INDENT "Packets: %i\n", summary.packet_count);
+  gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
+
+  /*
+   * Note: the start and stop times are initialized to 0, so if we
+   * have zero or one packets of the type in question that have
+   * time stamps, the elapsed times will be zero, just as if we
+   * have both start and stop time stamps but they're the same.
+   * That means we can avoid some checks for whether we have more
+   * than one packet of the type in question with time stamps.
+   */
+  seconds = summary.stop_time - summary.start_time;
+  /* Time between first and last */
+  if (seconds > 0) {
+    g_snprintf(string_buff, SUM_STR_MAX, INDENT "Between first and last packet:%.3f sec\n", seconds);
+    gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
+  }
+
+  /* Average packets per second */
+  if (seconds > 0) {
+    g_snprintf(string_buff, SUM_STR_MAX, INDENT "Avg. packets/sec: %.3f\n", summary.packet_count/seconds);
+    gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
+  }
+
+  /* Average packet size */
+  if (summary.packet_count > 1) {
+    g_snprintf(string_buff, SUM_STR_MAX, INDENT "Avg packet size: %.3f bytes\n",
+               /* MSVC cannot convert from unsigned __int64 to float, so first convert to signed __int64 */
+               (float) ((gint64) summary.bytes)/summary.packet_count);
+    gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
+  }
+
+  /* Byte count */
+  g_snprintf(string_buff, SUM_STR_MAX, INDENT "Bytes: %" G_GINT64_MODIFIER "u\n", summary.bytes);
+  gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
+
+  /* Bytes per second */
+  if (seconds > 0){
+    /* MSVC cannot convert from unsigned __int64 to float, so first convert to signed __int64 */
+    g_snprintf(string_buff, SUM_STR_MAX, INDENT "Avg bytes/sec: %.3f\n", ((gint64) summary.bytes)/seconds);
+    gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
+  }
+
+  /* MBit per second */
+  if (seconds > 0) {
+    g_snprintf(string_buff, SUM_STR_MAX, INDENT "Avg Mbit/sec: %.3f\n",
+               /* MSVC cannot convert from unsigned __int64 to float, so first convert to signed __int64 */
+               ((gint64) summary.bytes) * 8.0 / (seconds * 1000.0 * 1000.0));
+    gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
+  }
+
+  /* Add empty line */
+  g_snprintf(string_buff, SUM_STR_MAX, "\n");
+  gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
+
   /* Trace file comments from SHB */
   if(summary.opt_comment != NULL) {
     buf_str = g_strdup_printf("%s", summary.opt_comment);
@@ -772,4 +894,98 @@ summary_to_texbuff(GtkTextBuffer *buffer)
   g_snprintf(string_buff, SUM_STR_MAX, "\n\n");
   gtk_text_buffer_insert_at_cursor (buffer, string_buff, -1);
 
-}
\ No newline at end of file
+}
+
+static void
+comment_summary_copy_to_clipboard_cb(GtkWidget *w _U_, GtkWidget *view)
+{
+  GtkTextBuffer *buffer;
+  GtkTextIter start_iter, end_iter;
+  GtkClipboard *clipboard;
+
+  buffer = gtk_text_view_get_buffer (GTK_TEXT_VIEW (view));
+
+  gtk_text_buffer_get_bounds(buffer, &start_iter, &end_iter);
+
+  gtk_text_buffer_select_range(buffer, &start_iter, &end_iter);
+
+  clipboard = gtk_clipboard_get(GDK_SELECTION_CLIPBOARD);     /* Get the default clipboard */
+  gtk_text_buffer_copy_clipboard (buffer, clipboard);
+
+  gtk_text_buffer_select_range(buffer, &end_iter, &end_iter);
+
+}
+
+void
+show_packet_comment_summary_dlg (GtkAction *action _U_, gpointer data _U_)
+{
+
+  GtkWidget *vbox;
+  GtkWidget *view;
+  GtkWidget *scroll;
+  GtkWidget *bbox;
+  GtkWidget *copy_bt, *cancel_bt, *help_bt;
+  GtkTextBuffer *buffer;
+
+  view_capture_and_pkt_comments_dlg = dlg_window_new ("Comments Summary");
+  gtk_widget_set_size_request (view_capture_and_pkt_comments_dlg, 700, 350);
+  gtk_window_set_resizable (GTK_WINDOW (view_capture_and_pkt_comments_dlg), TRUE);
+  gtk_container_set_border_width (GTK_CONTAINER (view_capture_and_pkt_comments_dlg), DLG_OUTER_MARGIN);
+
+  vbox = ws_gtk_box_new(GTK_ORIENTATION_VERTICAL, DLG_UNRELATED_SPACING, FALSE);
+  gtk_container_add (GTK_CONTAINER (view_capture_and_pkt_comments_dlg), vbox);
+  gtk_widget_show (vbox);
+
+  view = gtk_text_view_new ();
+  gtk_text_view_set_wrap_mode(GTK_TEXT_VIEW(view), GTK_WRAP_WORD);
+  buffer = gtk_text_view_get_buffer (GTK_TEXT_VIEW (view));
+  gtk_widget_show (view);
+
+  scroll = gtk_scrolled_window_new(NULL, NULL);
+  gtk_scrolled_window_set_policy(GTK_SCROLLED_WINDOW(scroll),
+                  GTK_POLICY_NEVER, GTK_POLICY_AUTOMATIC);
+  gtk_container_add(GTK_CONTAINER(scroll), view);
+  gtk_widget_show(scroll);
+  gtk_box_pack_start(GTK_BOX (vbox), scroll, TRUE, TRUE, 0);
+
+  /* Add capture summary information */
+  summary_to_texbuff(buffer);
+
+  /* Add all packet comments */
+  packet_list_return_all_comments(buffer);
+
+  /* Button row. */
+  bbox = dlg_button_row_new (GTK_STOCK_COPY, GTK_STOCK_CANCEL, GTK_STOCK_HELP, NULL);
+  gtk_box_pack_end (GTK_BOX(vbox), bbox, FALSE, FALSE, 0);
+
+  copy_bt = (GtkWidget *)g_object_get_data (G_OBJECT(bbox), GTK_STOCK_COPY);
+  g_signal_connect (copy_bt, "clicked", G_CALLBACK(comment_summary_copy_to_clipboard_cb), view);
+  gtk_widget_set_sensitive (copy_bt, TRUE);
+
+  cancel_bt = (GtkWidget *)g_object_get_data (G_OBJECT(bbox), GTK_STOCK_CANCEL);
+  window_set_cancel_button (view_capture_and_pkt_comments_dlg, cancel_bt, window_cancel_button_cb);
+
+  help_bt = (GtkWidget *)g_object_get_data (G_OBJECT(bbox), GTK_STOCK_HELP);
+#if 0
+  g_signal_connect (help_bt, "clicked",/* G_CALLBACK(topic_cb)*/NULL, /*(gpointer)HELP_MANUAL_ADDR_RESOLVE_DIALOG*/NULL);
+#endif
+  gtk_widget_set_sensitive (help_bt, FALSE);
+
+  gtk_widget_grab_default (copy_bt);
+
+
+  gtk_widget_show (view_capture_and_pkt_comments_dlg);
+}
+
+/*
+ * Editor modelines
+ *
+ * Local Variables:
+ * c-basic-offset: 2
+ * tab-width: 8
+ * indent-tabs-mode: nil
+ * End:
+ *
+ * ex: set shiftwidth=2 tabstop=8 expandtab:
+ * :indentSize=2:tabSize=8:noTabs=true:
+ */