diff options
| author | Jeff Cohen <jeffc@jolt-lang.org> | 2005-01-22 16:28:33 +0000 |
|---|---|---|
| committer | Jeff Cohen <jeffc@jolt-lang.org> | 2005-01-22 16:28:33 +0000 |
| commit | ccbd3f0d57cb5192cb5850ee447d2d81fa492086 (patch) | |
| tree | dc5dad8783c5419ed851c20ffa4f661cabe0c527 /llvm/lib/System/Win32/Path.inc | |
| parent | 52c97fbea95149fd6c4ecfa3f095b0a7fdfed835 (diff) | |
| download | bcm5719-llvm-ccbd3f0d57cb5192cb5850ee447d2d81fa492086.tar.gz bcm5719-llvm-ccbd3f0d57cb5192cb5850ee447d2d81fa492086.zip | |
Fix destroyDirectory bug
llvm-svn: 19746
Diffstat (limited to 'llvm/lib/System/Win32/Path.inc')
| -rw-r--r-- | llvm/lib/System/Win32/Path.inc | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/llvm/lib/System/Win32/Path.inc b/llvm/lib/System/Win32/Path.inc index 7f891f43f9f..a28fd82d38a 100644 --- a/llvm/lib/System/Win32/Path.inc +++ b/llvm/lib/System/Win32/Path.inc @@ -562,15 +562,17 @@ Path::destroyDirectory(bool remove_contents) const { // If it doesn't exist, we're done. if (!exists()) return true; - char *pathname = reinterpret_cast<char *>(_alloca(path.length()+1)); - path.copy(pathname,path.length()+1); + char *pathname = reinterpret_cast<char *>(_alloca(path.length()+2)); int lastchar = path.length() - 1 ; - if (pathname[lastchar] == '/') - pathname[lastchar] = 0; + path.copy(pathname,lastchar+2); + + // Make path end with '/*'. + pathname[lastchar+1] = '*'; + pathname[lastchar+2] = 0; if (remove_contents) { WIN32_FIND_DATA fd; - HANDLE h = FindFirstFile(path.c_str(), &fd); + HANDLE h = FindFirstFile(pathname, &fd); // It's a bad idea to alter the contents of a directory while enumerating // its contents. So build a list of its contents first, then destroy them. @@ -610,6 +612,7 @@ Path::destroyDirectory(bool remove_contents) const { } } + pathname[lastchar] = 0; if (!RemoveDirectory(pathname)) ThrowError(std::string(pathname) + ": Can't destroy directory: "); return true; |

