CreateDirectory() should check if an existing path is actually a directory...

base/file_util_posix.cc

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/file_util.h"
 
 #include <fcntl.h>
 #include <fnmatch.h>
 #include <fts.h>
 #include <libgen.h>
@@ skipping 93 matching lines @@
 bool CopyTree(const std::wstring& from_path, const std::wstring& to_path) {
   // TODO(erikkay): implement
   return false;
 }
 
 bool PathExists(const std::wstring& path) {
   struct stat64 file_info;
   return (stat64(WideToUTF8(path).c_str(), &file_info) == 0);
 }
 
+bool DirectoryExists(const std::wstring& path) {
+  struct stat64 file_info;
+  if (stat64(WideToUTF8(path).c_str(), &file_info) == 0)
+    return S_ISDIR(file_info.st_mode);
+  return false;
+}
+
 // TODO(erikkay): implement
 #if 0
 bool GetFileCreationLocalTimeFromHandle(int fd,
                                         LPSYSTEMTIME creation_time) {
   if (!file_handle)
     return false;
   
   FILETIME utc_filetime;
   if (!GetFileTime(file_handle, &utc_filetime, NULL, NULL))
     return false;
@@ skipping 48 matching lines @@
 bool CreateDirectory(const std::wstring& full_path) {
   std::vector<std::wstring> components;
   PathComponents(full_path, &components);
   std::wstring path;
   std::vector<std::wstring>::iterator i = components.begin();
   for (; i != components.end(); ++i) {
     if (path.length() == 0)
       path = *i;
     else
       AppendToPath(&path, *i);
-    if (!PathExists(path)) {
+    if (!DirectoryExists(path)) {
       if (mkdir(WideToUTF8(path).c_str(), 0777) != 0)
         return false;
     }
   }
   return true;
 }
 
 bool GetFileSize(const std::wstring& file_path, int64* file_size) {
   struct stat64 file_info;
   if (stat64(WideToUTF8(file_path).c_str(), &file_info) != 0)
@@ skipping 124 matching lines @@
     return (file_type_ & FileEnumerator::DIRECTORIES) ? cur_file : Next();
   } else if (fts_ent->fts_info == FTS_F) {
     return (file_type_ & FileEnumerator::FILES) ? cur_file : Next();
   }
   // TODO(erikkay) - verify that the other fts_info types aren't interesting
   return Next();
 }
   
   
 } // namespace file_util
-
-