diff --git a/test/Object/archive-error-tmp.txt b/test/Object/archive-error-tmp.txt new file mode 100644 index 00000000000..061898655b6 --- /dev/null +++ b/test/Object/archive-error-tmp.txt @@ -0,0 +1,9 @@ +REQUIRES: shell + +Test that no temporary file is left behind on error. + +RUN: rm -rf %t +RUN: mkdir %t +RUN: not llvm-ar rc %t/foo.a . +RUN: rmdir %t + diff --git a/tools/llvm-ar/llvm-ar.cpp b/tools/llvm-ar/llvm-ar.cpp index 32bcb49a1db..d65f9ecc9fe 100644 --- a/tools/llvm-ar/llvm-ar.cpp +++ b/tools/llvm-ar/llvm-ar.cpp @@ -41,10 +41,13 @@ using namespace llvm; static StringRef ToolName; static const char *TemporaryOutput; +static int TmpArchiveFD = -1; // fail - Show the error message and exit. LLVM_ATTRIBUTE_NORETURN static void fail(Twine Error) { outs() << ToolName << ": " << Error << ".\n"; + if (TmpArchiveFD != -1) + close(TmpArchiveFD); if (TemporaryOutput) sys::fs::remove(TemporaryOutput); exit(1); @@ -526,7 +529,6 @@ static void printWithSpacePadding(raw_ostream &OS, T Data, unsigned Size) { static void performWriteOperation(ArchiveOperation Operation, object::Archive *OldArchive) { - int TmpArchiveFD; SmallString<128> TmpArchive; failIfError(sys::fs::createUniqueFile(ArchiveName + ".temp-archive-%%%%%%%.a", TmpArchiveFD, TmpArchive));