lineedit: do not hang on error, but return error indicator.

[oweals/busybox.git] / coreutils / touch.c

diff --git a/coreutils/touch.c b/coreutils/touch.c
index 7d1bf0d9ef7cf3d1047692c6b78c0c4d53c1b4e4..03aac3aa2f8f9a01cc8ceadb29138cf3d46c889f 100644 (file)
--- a/coreutils/touch.c
+++ b/coreutils/touch.c
@@ -4,7 +4,7 @@
  *
  * Copyright (C) 1999-2004 by Erik Andersen <andersen@codepoet.org>
  *
- * Licensed under GPLv2 or later, see file LICENSE in this tarball for details.
+ * Licensed under GPLv2 or later, see file LICENSE in this source tree.
  */
 
 /* BB_AUDIT SUSv3 _NOT_ compliant -- options -a, -m, -r, -t not supported. */
@@ -40,6 +40,9 @@
 int touch_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;
 int touch_main(int argc UNUSED_PARAM, char **argv)
 {
+       int fd;
+       int status = EXIT_SUCCESS;
+       int opts;
 #if ENABLE_DESKTOP
 # if ENABLE_LONG_OPTS
        static const char touch_longopts[] ALIGN1 =
@@ -49,17 +52,15 @@ int touch_main(int argc UNUSED_PARAM, char **argv)
                "date\0"              Required_argument "d"
        ;
 # endif
-       struct timeval timebuf = {.tv_usec = 0};
        char *reference_file = NULL;
        char *date_str = NULL;
+       struct timeval timebuf[2];
+       timebuf[1].tv_usec = timebuf[0].tv_usec = 0;
 #else
 # define reference_file NULL
 # define date_str       NULL
-# define timebuf        (*(struct timeval*)NULL)
+# define timebuf        ((struct timeval*)NULL)
 #endif
-       int fd;
-       int status = EXIT_SUCCESS;
-       int opts;
 
 #if ENABLE_DESKTOP && ENABLE_LONG_OPTS
        applet_long_options = touch_longopts;
@@ -83,7 +84,7 @@ int touch_main(int argc UNUSED_PARAM, char **argv)
        if (reference_file) {
                struct stat stbuf;
                xstat(reference_file, &stbuf);
-               timebuf.tv_sec = stbuf.st_mtime;
+               timebuf[1].tv_sec = timebuf[0].tv_sec = stbuf.st_mtime;
        }
 
        if (date_str) {
@@ -99,22 +100,21 @@ int touch_main(int argc UNUSED_PARAM, char **argv)
                tm_time.tm_isdst = -1;  /* Be sure to recheck dst */
                t = validate_tm_time(date_str, &tm_time);
 
-               timebuf.tv_sec = t;
+               timebuf[1].tv_sec = timebuf[0].tv_sec = t;
        }
 
        do {
-               if (utimes(*argv, reference_file ? &timebuf : NULL)) {
+               if (utimes(*argv, (reference_file || date_str) ? timebuf : NULL) != 0) {
                        if (errno == ENOENT) { /* no such file */
                                if (opts) { /* creation is disabled, so ignore */
                                        continue;
                                }
-                               /* Try to create the file. */
-                               fd = open(*argv, O_RDWR | O_CREAT,
-                                                 S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH | S_IWOTH
-                                                 );
-                               if ((fd >= 0) && !close(fd)) {
-                                       if (reference_file)
-                                               utimes(*argv, &timebuf);
+                               /* Try to create the file */
+                               fd = open(*argv, O_RDWR | O_CREAT, 0666);
+                               if (fd >= 0) {
+                                       xclose(fd);
+                                       if (reference_file || date_str)
+                                               utimes(*argv, timebuf);
                                        continue;
                                }
                        }