Updates to create_tempfile:

[obnox/wireshark/wip.git] / gtk / print_dlg.c

diff --git a/gtk/print_dlg.c b/gtk/print_dlg.c
index 0a6f5b0224796ac8d96e665bd5f25efb26c9c4f1..55a2ffd34370fe75c58ead6437d2a518dc5db93d 100644 (file)
--- a/gtk/print_dlg.c
+++ b/gtk/print_dlg.c
@@ -908,7 +908,7 @@ print_ok_cb(GtkWidget *ok_bt, gpointer parent_w)
 #ifdef _WIN32
   gboolean          win_printer = FALSE;
   int               tmp_fd;
-  char              tmp_namebuf[128+1];  /* XX: see create_tmpfile which says [128+1]; why ? */
+  char              *tmp_namebuf;
   char              *tmp_oldfile;
 #endif
   cf_print_status_t status;
@@ -951,10 +951,10 @@ print_ok_cb(GtkWidget *ok_bt, gpointer parent_w)
     /* Don't use tmpnam() or such, as this will fail under some ACL           */
     /* circumstances: http://bugs.wireshark.org/bugzilla/show_bug.cgi?id=358  */
     /* Also: tmpnam is "insecure" and should not be used.                     */
-    tmp_fd = create_tempfile(tmp_namebuf, sizeof(tmp_namebuf), "wshprint");
+    tmp_fd = create_tempfile(&tmp_namebuf, "wshprint");
     if(tmp_fd == -1) {
         simple_dialog(ESD_TYPE_ERROR, ESD_BTN_OK,
-            "Couldn't create a temporary file for printing.");
+            "Couldn't create a temporary file for printing:\n%s", tmp_namebuf);
         return;
     }
     /* remember to restore these values later! */