Add support for getting OS error information for creating temporary directories

runtime/bin/directory_posix.cc

 // Copyright (c) 2011, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "bin/directory.h"
 
 #include <dirent.h>
 #include <errno.h>
 #include <sys/param.h>
 #include <sys/stat.h>
 #include <unistd.h>
 
 #include "bin/file.h"
 #include "bin/platform.h"
 
+
+static char* SafeStrNCpy(char* dest, const char* src, size_t n) {
+  strncpy(dest, src, n);
+  dest[n - 1] = '\0';
+  return dest;
+}
+
+
+static void SetOsErrorMessage(char* os_error_message,
+                              int os_error_message_len) {
+  SafeStrNCpy(os_error_message, strerror(errno), os_error_message_len);
+}
+
+
 // Forward declaration.
 static bool ListRecursively(const char* dir_name,
                             bool recursive,
                             Dart_Port dir_port,
                             Dart_Port file_port,
                             Dart_Port done_port,
                             Dart_Port error_port);
 
 
 static void ComputeFullPath(const char* dir_name,
@@ skipping 220 matching lines @@
 }
 
 
 bool Directory::Create(const char* dir_name) {
   // Create the directory with the permissions specified by the
   // process umask.
   return (mkdir(dir_name, 0777) == 0);
 }
 
 
-char* Directory::CreateTemp(const char* const_template, int64_t number) {
+int Directory::CreateTemp(const char* const_template,
+                          int64_t number,
+                          char** path,
+                          char* os_error_message,
+                          int os_error_message_len) {
   // Returns a new, unused directory name, modifying the contents of
   // dir_template.  Creates the directory with the permissions specified
   // by the process umask.
   // The return value must be freed by the caller.
-  char* path = static_cast<char*>(malloc(PATH_MAX + 1));
-  strncpy(path, const_template, PATH_MAX + 1);
-  path[PATH_MAX] = '\0';
-  int path_length = strlen(path);
+  *path = static_cast<char*>(malloc(PATH_MAX + 1));
+  strncpy(*path, const_template, PATH_MAX + 1);

[Bill Hesse, 2011/12/13 13:51:03]

Use SafeStrNCpy here, or remove SafeStrNCpy.

[Søren Gjesse, 2011/12/13 14:07:40]

Done.

+  (*path)[PATH_MAX] = '\0';
+  int path_length = strlen(*path);
   if (path_length > 0) {
-    if (path[path_length - 1] == '/') {
-      snprintf(path + path_length, PATH_MAX - path_length, "temp_dir_XXXXXX");
+    if ((*path)[path_length - 1] == '/') {
+      snprintf(*path + path_length, PATH_MAX - path_length, "temp_dir_XXXXXX");
     } else {
-      snprintf(path + path_length, PATH_MAX - path_length, "XXXXXX");
+      snprintf(*path + path_length, PATH_MAX - path_length, "XXXXXX");
     }
   } else {
-    snprintf(path, PATH_MAX, "/tmp/temp_dir1_XXXXXX");
+    snprintf(*path, PATH_MAX, "/tmp/temp_dir1_XXXXXX");
   }
-  char* result = mkdtemp(path);
+  char* result = mkdtemp(*path);
   if (result == NULL) {
-    // Return the empty string, but as a freeable array.
-    path[0] = '\0';
+    SetOsErrorMessage(os_error_message, os_error_message_len);
+    free(*path);
+    *path = NULL;
+    return errno;
   }
-  return path;
+  return 0;
 }
 
 
 bool Directory::Delete(const char* dir_name) {
   return (rmdir(dir_name) == 0);
 }