6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2024-12-26 21:32:10 +00:00
Code Issues Projects Releases Wiki Activity

On error, close the temporary file descriptor.

Browse Source 
With this change llvm-ar can remove the temporary file on windows too.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@186423 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Rafael Espindola 2013-07-16 16:00:32 +00:00
parent 21508bf853
commit 95779b65cf
2 changed files with 12 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
test/Object/archive-error-tmp.txt Normal file
View File

@ -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

4
tools/llvm-ar/llvm-ar.cpp
View File

@ -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));