bug 452217 nsLocalFileWin should return errors consistent with other platforms, r=neil, sr=bsmedberg

2009-01-27 16:50:14 -08:00
parent ff9deea53a
commit 5fb451ef46
1 changed files with 8 additions and 0 deletions
--- a/xpcom/io/nsLocalFileWin.cpp
+++ b/xpcom/io/nsLocalFileWin.cpp
@@ -1038,6 +1038,14 @@ nsLocalFile::Create(PRUint32 type, PRUint32 attributes)
                      &file);
        if (file)
            PR_Close(file);
+
+        if (rv == NS_ERROR_FILE_ACCESS_DENIED)
+        {
+            // need to return already-exists for directories (bug 452217)
+            PRBool isdir;
+            if (NS_SUCCEEDED(IsDirectory(&isdir)) && isdir)
+                rv = NS_ERROR_FILE_ALREADY_EXISTS;
+        }
        return rv;
    }