X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=src%2Fdata%2Fmake-file.c;h=c8f3238a80cdc2f7a7c5d9479cc72c2bbf191df5;hb=232b19c50410590a705a9f4917c20500fb15c100;hp=f5cda498a89358662965a01c2b5e44a39c5a707b;hpb=cb586666724d5fcbdb658ce471b85484f0a7babe;p=pspp
diff --git a/src/data/make-file.c b/src/data/make-file.c
index f5cda498a8..c8f3238a80 100644
--- a/src/data/make-file.c
+++ b/src/data/make-file.c
@@ -1,5 +1,5 @@
/* PSPP - a program for statistical analysis.
- Copyright (C) 2004 Free Software Foundation, Inc.
+ Copyright (C) 2004, 2010 Free Software Foundation, Inc.
This program is free software: you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
@@ -15,123 +15,55 @@
along with this program. If not, see . */
#include
+
+#include "data/make-file.h"
+#include "libpspp/i18n.h"
+
#include
+#include
#include
+#include
#include
#include
-#include
-#include
#include
+#include
-#include
-#include
-#include
-#include
+#include "data/file-name.h"
+#include "data/file-handle-def.h"
+#include "libpspp/ll.h"
+#include "libpspp/message.h"
-#include "fatal-signal.h"
-#include "tempname.h"
-#include "xalloc.h"
+#include "gl/fatal-signal.h"
+#include "gl/tempname.h"
+#include "gl/xalloc.h"
+#include "gl/xvasprintf.h"
#include "gettext.h"
#define _(msgid) gettext (msgid)
-/* Non ansi compilers may set this */
-#ifndef P_tmpdir
-#define P_tmpdir "/tmp"
-#endif
-
-/* Creates a temporary file and stores its name in *FILE_NAME and
- a file descriptor for it in *FD. Returns success. Caller is
- responsible for freeing *FILE_NAME. */
-int
-make_temp_file (int *fd, char **file_name)
-{
- const char *parent_dir;
-
- assert (file_name != NULL);
- assert (fd != NULL);
-
- if (getenv ("TMPDIR") != NULL)
- parent_dir = getenv ("TMPDIR");
- else
- parent_dir = P_tmpdir;
-
- *file_name = xmalloc (strlen (parent_dir) + 32);
- sprintf (*file_name, "%s/psppXXXXXX", parent_dir);
- *fd = mkstemp (*file_name);
- if (*fd < 0)
- {
- msg (ME, _("%s: Creating temporary file: %s."),
- *file_name, strerror (errno));
- free (*file_name);
- *file_name = NULL;
- return 0;
- }
- return 1;
-}
-
-
-/* Creates a temporary file and stores its name in *FILE_NAME and
- a file stream for it in *FP. Returns success. Caller is
- responsible for freeing *FILE_NAME and for closing *FP */
-int
-make_unique_file_stream (FILE **fp, char **file_name)
-{
- static int serial = 0;
- const char *parent_dir;
-
-
- /* FIXME:
- Need to check for pre-existing file name.
- Need also to pass in the directory instead of using /tmp
- */
-
- assert (file_name != NULL);
- assert (fp != NULL);
-
- if (getenv ("TMPDIR") != NULL)
- parent_dir = getenv ("TMPDIR");
- else
- parent_dir = P_tmpdir;
-
- *file_name = xmalloc (strlen (parent_dir) + 32);
-
-
- sprintf (*file_name, "%s/pspp%d.png", parent_dir, serial++);
-
- *fp = fopen(*file_name, "w");
-
- if (! *fp )
- {
- msg (ME, _("%s: Creating file: %s."), *file_name, strerror (errno));
- free (*file_name);
- *file_name = NULL;
- return 0;
- }
-
- return 1;
-}
-
struct replace_file
{
struct ll ll;
char *file_name;
char *tmp_name;
};
-
+
static struct ll_list all_files = LL_INITIALIZER (all_files);
static void free_replace_file (struct replace_file *);
static void unlink_replace_files (void);
struct replace_file *
-replace_file_start (const char *file_name, const char *mode,
- mode_t permissions, FILE **fp, char **tmp_name)
+replace_file_start (const struct file_handle *fh, const char *mode,
+ mode_t permissions, FILE **fp)
{
static bool registered;
struct stat s;
struct replace_file *rf;
int fd;
+ int saved_errno = errno;
+
+ const char *file_name = fh_get_file_name (fh);
/* If FILE_NAME represents a special file, write to it directly
instead of trying to replace it. */
@@ -141,8 +73,9 @@ replace_file_start (const char *file_name, const char *mode,
fd = open (file_name, O_WRONLY);
if (fd < 0)
{
+ saved_errno = errno;
msg (ME, _("Opening %s for writing: %s."),
- file_name, strerror (errno));
+ file_name, strerror (saved_errno));
return NULL;
}
@@ -150,8 +83,9 @@ replace_file_start (const char *file_name, const char *mode,
*fp = fdopen (fd, mode);
if (*fp == NULL)
{
- msg (ME, _("Opening stream for %s: %s."),
- file_name, strerror (errno));
+ saved_errno = errno;
+ msg (ME, _("Opening stream for %s: %s."),
+ file_name, strerror (saved_errno));
close (fd);
return NULL;
}
@@ -159,8 +93,6 @@ replace_file_start (const char *file_name, const char *mode,
rf = xmalloc (sizeof *rf);
rf->file_name = NULL;
rf->tmp_name = xstrdup (file_name);
- if (tmp_name != NULL)
- *tmp_name = rf->tmp_name;
return rf;
}
@@ -176,11 +108,12 @@ replace_file_start (const char *file_name, const char *mode,
for (;;)
{
/* Generate unique temporary file name. */
- rf->tmp_name = xasprintf ("%s.tmpXXXXXX", file_name);
- if (gen_tempname (rf->tmp_name, 0600, GT_NOCREATE) < 0)
+ rf->tmp_name = xasprintf ("%stmpXXXXXX", file_name);
+ if (gen_tempname (rf->tmp_name, 0, 0600, GT_NOCREATE) < 0)
{
+ saved_errno = errno;
msg (ME, _("Creating temporary file to replace %s: %s."),
- rf->file_name, strerror (errno));
+ rf->file_name, strerror (saved_errno));
goto error;
}
@@ -190,8 +123,9 @@ replace_file_start (const char *file_name, const char *mode,
break;
if (errno != EEXIST)
{
+ saved_errno = errno;
msg (ME, _("Creating temporary file %s: %s."),
- rf->tmp_name, strerror (errno));
+ rf->tmp_name, strerror (saved_errno));
goto error;
}
free (rf->tmp_name);
@@ -202,8 +136,9 @@ replace_file_start (const char *file_name, const char *mode,
*fp = fdopen (fd, mode);
if (*fp == NULL)
{
+ saved_errno = errno;
msg (ME, _("Opening stream for temporary file %s: %s."),
- rf->tmp_name, strerror (errno));
+ rf->tmp_name, strerror (saved_errno));
close (fd);
unlink (rf->tmp_name);
goto error;
@@ -213,17 +148,13 @@ replace_file_start (const char *file_name, const char *mode,
ll_push_head (&all_files, &rf->ll);
unblock_fatal_signals ();
- if (tmp_name != NULL)
- *tmp_name = rf->tmp_name;
-
return rf;
error:
unblock_fatal_signals ();
free_replace_file (rf);
*fp = NULL;
- if (tmp_name != NULL)
- *tmp_name = NULL;
+ errno = saved_errno;
return NULL;
}
@@ -301,7 +232,6 @@ unlink_replace_files (void)
/* We don't free_replace_file(RF) because calling free is unsafe
from an asynchronous signal handler. */
unlink (rf->tmp_name);
- fflush (stdout);
}
unblock_fatal_signals ();
}