-//===-- ArchiveWriter.cpp - LLVM archive writing --------------------------===//
-//
+//===-- ArchiveWriter.cpp - Write LLVM archive files ----------------------===//
+//
// The LLVM Compiler Infrastructure
//
-// This file was developed by Reid Spencerand is distributed under the
-// University of Illinois Open Source License. See LICENSE.TXT for details.
-//
+// This file is distributed under the University of Illinois Open Source
+// License. See LICENSE.TXT for details.
+//
//===----------------------------------------------------------------------===//
//
-// Builds up standard unix archive files (.a) containing LLVM bytecode.
+// Builds up an LLVM archive file (.a) containing LLVM bitcode.
//
//===----------------------------------------------------------------------===//
#include "ArchiveInternals.h"
-#include "llvm/Module.h"
-#include "llvm/Bytecode/Reader.h"
-#include "llvm/Support/FileUtilities.h"
-#include "llvm/ADT/StringExtras.h"
-#include "llvm/System/MappedFile.h"
+#include "llvm/Bitcode/ReaderWriter.h"
+#include "llvm/System/Signals.h"
+#include "llvm/System/Process.h"
+#include "llvm/ModuleProvider.h"
#include <fstream>
-#include <iostream>
-
+#include <ostream>
+#include <iomanip>
using namespace llvm;
-Archive*
-Archive::CreateEmpty(const sys::Path& Filename) {
- Archive* result = new Archive;
- Archive::ArchiveInternals* impl = result->impl = new Archive::ArchiveInternals;
- impl->fname = Filename;
- return result;
+// Write an integer using variable bit rate encoding. This saves a few bytes
+// per entry in the symbol table.
+inline void writeInteger(unsigned num, std::ofstream& ARFile) {
+ while (1) {
+ if (num < 0x80) { // done?
+ ARFile << (unsigned char)num;
+ return;
+ }
+
+ // Nope, we are bigger than a character, output the next 7 bits and set the
+ // high bit to say that there is more coming...
+ ARFile << (unsigned char)(0x80 | ((unsigned char)num & 0x7F));
+ num >>= 7; // Shift out 7 bits now...
+ }
+}
+
+// Compute how many bytes are taken by a given VBR encoded value. This is needed
+// to pre-compute the size of the symbol table.
+inline unsigned numVbrBytes(unsigned num) {
+
+ // Note that the following nested ifs are somewhat equivalent to a binary
+ // search. We split it in half by comparing against 2^14 first. This allows
+ // most reasonable values to be done in 2 comparisons instead of 1 for
+ // small ones and four for large ones. We expect this to access file offsets
+ // in the 2^10 to 2^24 range and symbol lengths in the 2^0 to 2^8 range,
+ // so this approach is reasonable.
+ if (num < 1<<14) {
+ if (num < 1<<7)
+ return 1;
+ else
+ return 2;
+ }
+ if (num < 1<<21)
+ return 3;
+
+ if (num < 1<<28)
+ return 4;
+ return 5; // anything >= 2^28 takes 5 bytes
}
+// Create an empty archive.
Archive*
-Archive::CreateFromFiles(
- const sys::Path& Filename,
- const PathList& Files,
- const std::string& StripName
-) {
- Archive* result = new Archive;
- Archive::ArchiveInternals* impl = result->impl = new Archive::ArchiveInternals;
- impl->fname = Filename;
-
- try {
- size_t strip_len = StripName.length();
- for (PathList::const_iterator P = Files.begin(), E = Files.end(); P != E ;++P)
- {
- if (P->readable()) {
- std::string name(P->get());
- if (strip_len > 0 && StripName == name.substr(0,strip_len)) {
- name.erase(0,strip_len);
- }
- if (P->isBytecodeFile()) {
- std::vector<std::string> syms;
- if (!GetBytecodeSymbols(*P, syms))
- throw std::string("Can not get symbols from: ") + P->get();
- impl->addFileMember(*P, name, &syms);
- } else {
- impl->addFileMember(*P, name);
- }
- }
- else
- throw std::string("Can not read: ") + P->get();
+Archive::CreateEmpty(const sys::Path& FilePath ) {
+ Archive* result = new Archive(FilePath);
+ return result;
+}
+
+// Fill the ArchiveMemberHeader with the information from a member. If
+// TruncateNames is true, names are flattened to 15 chars or less. The sz field
+// is provided here instead of coming from the mbr because the member might be
+// stored compressed and the compressed size is not the ArchiveMember's size.
+// Furthermore compressed files have negative size fields to identify them as
+// compressed.
+bool
+Archive::fillHeader(const ArchiveMember &mbr, ArchiveMemberHeader& hdr,
+ int sz, bool TruncateNames) const {
+
+ // Set the permissions mode, uid and gid
+ hdr.init();
+ char buffer[32];
+ sprintf(buffer, "%-8o", mbr.getMode());
+ memcpy(hdr.mode,buffer,8);
+ sprintf(buffer, "%-6u", mbr.getUser());
+ memcpy(hdr.uid,buffer,6);
+ sprintf(buffer, "%-6u", mbr.getGroup());
+ memcpy(hdr.gid,buffer,6);
+
+ // Set the last modification date
+ uint64_t secondsSinceEpoch = mbr.getModTime().toEpochTime();
+ sprintf(buffer,"%-12u", unsigned(secondsSinceEpoch));
+ memcpy(hdr.date,buffer,12);
+
+ // Get rid of trailing blanks in the name
+ std::string mbrPath = mbr.getPath().toString();
+ size_t mbrLen = mbrPath.length();
+ while (mbrLen > 0 && mbrPath[mbrLen-1] == ' ') {
+ mbrPath.erase(mbrLen-1,1);
+ mbrLen--;
+ }
+
+ // Set the name field in one of its various flavors.
+ bool writeLongName = false;
+ if (mbr.isStringTable()) {
+ memcpy(hdr.name,ARFILE_STRTAB_NAME,16);
+ } else if (mbr.isSVR4SymbolTable()) {
+ memcpy(hdr.name,ARFILE_SVR4_SYMTAB_NAME,16);
+ } else if (mbr.isBSD4SymbolTable()) {
+ memcpy(hdr.name,ARFILE_BSD4_SYMTAB_NAME,16);
+ } else if (mbr.isLLVMSymbolTable()) {
+ memcpy(hdr.name,ARFILE_LLVM_SYMTAB_NAME,16);
+ } else if (TruncateNames) {
+ const char* nm = mbrPath.c_str();
+ unsigned len = mbrPath.length();
+ size_t slashpos = mbrPath.rfind('/');
+ if (slashpos != std::string::npos) {
+ nm += slashpos + 1;
+ len -= slashpos +1;
}
+ if (len > 15)
+ len = 15;
+ memcpy(hdr.name,nm,len);
+ hdr.name[len] = '/';
+ } else if (mbrPath.length() < 16 && mbrPath.find('/') == std::string::npos) {
+ memcpy(hdr.name,mbrPath.c_str(),mbrPath.length());
+ hdr.name[mbrPath.length()] = '/';
+ } else {
+ std::string nm = "#1/";
+ nm += utostr(mbrPath.length());
+ memcpy(hdr.name,nm.data(),nm.length());
+ if (sz < 0)
+ sz -= mbrPath.length();
+ else
+ sz += mbrPath.length();
+ writeLongName = true;
+ }
- // Now that we've collected everything, write the archive
- impl->writeArchive();
+ // Set the size field
+ if (sz < 0) {
+ buffer[0] = '-';
+ sprintf(&buffer[1],"%-9u",(unsigned)-sz);
+ } else {
+ sprintf(buffer, "%-10u", (unsigned)sz);
+ }
+ memcpy(hdr.size,buffer,10);
- } catch(...) {
- delete impl;
- result->impl = 0;
- delete result;
- throw;
+ return writeLongName;
+}
+
+// Insert a file into the archive before some other member. This also takes care
+// of extracting the necessary flags and information from the file.
+bool
+Archive::addFileBefore(const sys::Path& filePath, iterator where,
+ std::string* ErrMsg) {
+ if (!filePath.exists()) {
+ if (ErrMsg)
+ *ErrMsg = "Can not add a non-existent file to archive";
+ return true;
}
- return result;
+ ArchiveMember* mbr = new ArchiveMember(this);
+
+ mbr->data = 0;
+ mbr->path = filePath;
+ const sys::FileStatus *FSInfo = mbr->path.getFileStatus(false, ErrMsg);
+ if (FSInfo)
+ mbr->info = *FSInfo;
+ else
+ return true;
+
+ unsigned flags = 0;
+ bool hasSlash = filePath.toString().find('/') != std::string::npos;
+ if (hasSlash)
+ flags |= ArchiveMember::HasPathFlag;
+ if (hasSlash || filePath.toString().length() > 15)
+ flags |= ArchiveMember::HasLongFilenameFlag;
+ std::string magic;
+ mbr->path.getMagicNumber(magic,4);
+ switch (sys::IdentifyFileType(magic.c_str(),4)) {
+ case sys::Bitcode_FileType:
+ flags |= ArchiveMember::BitcodeFlag;
+ break;
+ default:
+ break;
+ }
+ mbr->flags = flags;
+ members.insert(where,mbr);
+ return false;
}
-void
-Archive::ArchiveInternals::addFileMember(
- const sys::Path& filePath,
- const std::string& memberName,
- const StrTab* symbols
+// Write one member out to the file.
+bool
+Archive::writeMember(
+ const ArchiveMember& member,
+ std::ofstream& ARFile,
+ bool CreateSymbolTable,
+ bool TruncateNames,
+ bool ShouldCompress,
+ std::string* ErrMsg
) {
- MemberInfo info;
- info.path = filePath;
- info.name = memberName;
- filePath.getStatusInfo(info.status);
- if (symbols)
- info.symbols = *symbols;
- info.offset = 0;
- members.push_back(info);
-}
-void
-Archive::ArchiveInternals::writeInteger(int num, std::ofstream& ARFile) {
- char buff[4];
- buff[0] = (num >> 24) & 255;
- buff[1] = (num >> 16) & 255;
- buff[2] = (num >> 8) & 255;
- buff[3] = num & 255;
- ARFile.write(buff, sizeof(buff));
-}
+ unsigned filepos = ARFile.tellp();
+ filepos -= 8;
+
+ // Get the data and its size either from the
+ // member's in-memory data or directly from the file.
+ size_t fSize = member.getSize();
+ const char* data = (const char*)member.getData();
+ sys::MappedFile* mFile = 0;
+ if (!data) {
+ mFile = new sys::MappedFile();
+ if (mFile->open(member.getPath(), sys::MappedFile::READ_ACCESS, ErrMsg))
+ return true;
+ if (!(data = (const char*) mFile->map(ErrMsg)))
+ return true;
+ fSize = mFile->size();
+ }
-void
-Archive::ArchiveInternals::writeSymbolTable( std::ofstream& ARFile ) {
-
- // Compute the number of symbols in the symbol table and the
- // total byte size of the string pool. While we're traversing,
- // build the string pool for supporting long file names. Also,
- // build the table of file offsets for the symbol table and
- // the
- typedef std::map<std::string,unsigned> SymbolMap;
- StrTab stringPool;
- SymbolMap symbolTable;
- std::vector<unsigned> fileOffsets;
- std::string symTabStrings;
- unsigned fileOffset = 0;
- unsigned spOffset = 0;
- unsigned numSymbols = 0;
- unsigned numSymBytes = 0;
- for (unsigned i = 0; i < members.size(); i++ ) {
- MemberInfo& mi = members[i];
- StrTab& syms = mi.symbols;
- size_t numSym = syms.size();
- numSymbols += numSym;
- for (unsigned j = 0; j < numSym; j++ ) {
- numSymBytes += syms[j].size() + 1;
- symbolTable[syms[i]] = i;
- }
- if (mi.name.length() > 15 || std::string::npos != mi.name.find('/')) {
- stringPool.push_back(mi.name + "/\n");
- mi.name = std::string("/") + utostr(spOffset);
- spOffset += mi.name.length() + 2;
- } else if (mi.name[mi.name.length()-1] != '/') {
- mi.name += "/";
+ // Now that we have the data in memory, update the
+ // symbol table if its a bitcode file.
+ if (CreateSymbolTable && member.isBitcode()) {
+ std::vector<std::string> symbols;
+ std::string FullMemberName = archPath.toString() + "(" +
+ member.getPath().toString()
+ + ")";
+ ModuleProvider* MP =
+ GetBitcodeSymbols((const unsigned char*)data,fSize,
+ FullMemberName, symbols, ErrMsg);
+
+ // If the bitcode parsed successfully
+ if ( MP ) {
+ for (std::vector<std::string>::iterator SI = symbols.begin(),
+ SE = symbols.end(); SI != SE; ++SI) {
+
+ std::pair<SymTabType::iterator,bool> Res =
+ symTab.insert(std::make_pair(*SI,filepos));
+
+ if (Res.second) {
+ symTabSize += SI->length() +
+ numVbrBytes(SI->length()) +
+ numVbrBytes(filepos);
+ }
+ }
+ // We don't need this module any more.
+ delete MP;
+ } else {
+ if (mFile != 0) {
+ mFile->close();
+ delete mFile;
+ }
+ if (ErrMsg)
+ *ErrMsg = "Can't parse bitcode member: " + member.getPath().toString()
+ + ": " + *ErrMsg;
+ return true;
}
- fileOffsets.push_back(fileOffset);
- fileOffset += sizeof(ArchiveMemberHeader) + mi.status.fileSize;
}
+ int hdrSize = fSize;
- // Compute the size of the symbol table file member
- unsigned symTabSize = 0;
- if (numSymbols != 0)
- symTabSize =
- sizeof(ArchiveMemberHeader) + // Size of the file header
- 4 + // Size of "number of entries"
- (4 * numSymbols) + // Size of member file indices
- numSymBytes; // Size of the string table
-
- // Compute the size of the string pool
- unsigned strPoolSize = 0;
- if (spOffset != 0 )
- strPoolSize =
- sizeof(ArchiveMemberHeader) + // Size of the file header
- spOffset; // Number of bytes in the string pool
-
- // Compute the byte index offset created by symbol table and string pool
- unsigned firstFileOffset = symTabSize + strPoolSize;
-
- // Create header for symbol table. This must be first if there is
- // a symbol table and must have a special name.
- if ( symTabSize > 0 ) {
- ArchiveMemberHeader Hdr;
- Hdr.init();
-
- // Name of symbol table is '/ ' but "" is passed in
- // because the setName method always terminates with a /
- Hdr.setName(ARFILE_SYMTAB_NAME);
- Hdr.setDate();
- Hdr.setSize(symTabSize - sizeof(ArchiveMemberHeader));
- Hdr.setMode(0);
- Hdr.setUid(0);
- Hdr.setGid(0);
-
- // Write header to archive file
- ARFile.write((char*)&Hdr, sizeof(Hdr));
-
- // Write the number of entries in the symbol table
- this->writeInteger(numSymbols, ARFile);
-
- // Write the file offset indices for each symbol and build the
- // symbol table string pool
- std::string symTabStrPool;
- symTabStrPool.reserve(256 * 1024); // Reserve 256KBytes for symbols
- for (SymbolMap::iterator I = symbolTable.begin(), E = symbolTable.end();
- I != E; ++I ) {
- this->writeInteger(firstFileOffset + fileOffsets[I->second], ARFile);
- symTabStrPool += I->first;
- symTabStrPool += "\0";
- }
+ // Compute the fields of the header
+ ArchiveMemberHeader Hdr;
+ bool writeLongName = fillHeader(member,Hdr,hdrSize,TruncateNames);
- // Write the symbol table's string pool
- ARFile.write(symTabStrPool.data(), symTabStrPool.size());
+ // Write header to archive file
+ ARFile.write((char*)&Hdr, sizeof(Hdr));
+
+ // Write the long filename if its long
+ if (writeLongName) {
+ ARFile.write(member.getPath().toString().data(),
+ member.getPath().toString().length());
}
- //============== DONE WITH SYMBOL TABLE
-
- if (strPoolSize > 0) {
- // Initialize the header for the string pool
- ArchiveMemberHeader Hdr;
- Hdr.init();
- Hdr.setName(ARFILE_STRTAB_NAME);
- Hdr.setDate();
- Hdr.setSize(spOffset);
- Hdr.setMode(0);
- Hdr.setUid(0);
- Hdr.setGid(0);
-
- // Write the string pool header
- ARFile.write((char*)&Hdr, sizeof(Hdr));
-
- // Write the string pool
- for (unsigned i = 0; i < stringPool.size(); i++) {
- ARFile.write(stringPool[i].data(), stringPool[i].size());
- }
+ // Write the (possibly compressed) member's content to the file.
+ ARFile.write(data,fSize);
+
+ // Make sure the member is an even length
+ if ((ARFile.tellp() & 1) == 1)
+ ARFile << ARFILE_PAD;
+
+ // Close the mapped file if it was opened
+ if (mFile != 0) {
+ mFile->close();
+ delete mFile;
}
+ return false;
}
+// Write out the LLVM symbol table as an archive member to the file.
void
-Archive::ArchiveInternals::writeMember(
- const MemberInfo& member,
- std::ofstream& ARFile
-) {
-
- // Map the file into memory. We do this early for two reasons. First,
- // if there's any kind of error, we want to know about it. Second, we
- // want to ensure we're using the most recent size for this file.
- sys::MappedFile mFile(member.path);
- mFile.map();
+Archive::writeSymbolTable(std::ofstream& ARFile) {
- // Header for the archive member
+ // Construct the symbol table's header
ArchiveMemberHeader Hdr;
Hdr.init();
+ memcpy(Hdr.name,ARFILE_LLVM_SYMTAB_NAME,16);
+ uint64_t secondsSinceEpoch = sys::TimeValue::now().toEpochTime();
+ char buffer[32];
+ sprintf(buffer, "%-8o", 0644);
+ memcpy(Hdr.mode,buffer,8);
+ sprintf(buffer, "%-6u", sys::Process::GetCurrentUserId());
+ memcpy(Hdr.uid,buffer,6);
+ sprintf(buffer, "%-6u", sys::Process::GetCurrentGroupId());
+ memcpy(Hdr.gid,buffer,6);
+ sprintf(buffer,"%-12u", unsigned(secondsSinceEpoch));
+ memcpy(Hdr.date,buffer,12);
+ sprintf(buffer,"%-10u",symTabSize);
+ memcpy(Hdr.size,buffer,10);
+
+ // Write the header
+ ARFile.write((char*)&Hdr, sizeof(Hdr));
- // Set the name. If its longer than 15 chars, it will have already
- // been reduced by the writeSymbolTable.
- Hdr.setName(member.name);
+ // Save the starting position of the symbol tables data content.
+ unsigned startpos = ARFile.tellp();
+
+ // Write out the symbols sequentially
+ for ( Archive::SymTabType::iterator I = symTab.begin(), E = symTab.end();
+ I != E; ++I)
+ {
+ // Write out the file index
+ writeInteger(I->second, ARFile);
+ // Write out the length of the symbol
+ writeInteger(I->first.length(), ARFile);
+ // Write out the symbol
+ ARFile.write(I->first.data(), I->first.length());
+ }
- // Set the other header members
- Hdr.setSize( mFile.size() );
- Hdr.setMode( member.status.mode);
- Hdr.setUid ( member.status.user);
- Hdr.setGid ( member.status.group);
- Hdr.setDate( member.status.modTime.ToPosixTime() );
+ // Now that we're done with the symbol table, get the ending file position
+ unsigned endpos = ARFile.tellp();
- // Write header to archive file
- ARFile.write((char*)&Hdr, sizeof(Hdr));
-
- //write to archive file
- ARFile.write(mFile.charBase(),mFile.size());
+ // Make sure that the amount we wrote is what we pre-computed. This is
+ // critical for file integrity purposes.
+ assert(endpos - startpos == symTabSize && "Invalid symTabSize computation");
- mFile.unmap();
+ // Make sure the symbol table is even sized
+ if (symTabSize % 2 != 0 )
+ ARFile << ARFILE_PAD;
}
-void
-Archive::ArchiveInternals::writeArchive() {
-
+// Write the entire archive to the file specified when the archive was created.
+// This writes to a temporary file first. Options are for creating a symbol
+// table, flattening the file names (no directories, 15 chars max) and
+// compressing each archive member.
+bool
+Archive::writeToDisk(bool CreateSymbolTable, bool TruncateNames, bool Compress,
+ std::string* ErrMsg)
+{
+ // Make sure they haven't opened up the file, not loaded it,
+ // but are now trying to write it which would wipe out the file.
+ if (members.empty() && mapfile->size() > 8) {
+ if (ErrMsg)
+ *ErrMsg = "Can't write an archive not opened for writing";
+ return true;
+ }
+
+ // Create a temporary file to store the archive in
+ sys::Path TmpArchive = archPath;
+ if (TmpArchive.createTemporaryFileOnDisk(ErrMsg))
+ return true;
+
+ // Make sure the temporary gets removed if we crash
+ sys::RemoveFileOnSignal(TmpArchive);
+
// Create archive file for output.
- std::ofstream ArchiveFile(fname.get().c_str());
-
+ std::ios::openmode io_mode = std::ios::out | std::ios::trunc |
+ std::ios::binary;
+ std::ofstream ArchiveFile(TmpArchive.c_str(), io_mode);
+
// Check for errors opening or creating archive file.
- if ( !ArchiveFile.is_open() || ArchiveFile.bad() ) {
- throw std::string("Error opening archive file: ") + fname.get();
+ if (!ArchiveFile.is_open() || ArchiveFile.bad()) {
+ if (TmpArchive.exists())
+ TmpArchive.eraseFromDisk();
+ if (ErrMsg)
+ *ErrMsg = "Error opening archive file: " + archPath.toString();
+ return true;
+ }
+
+ // If we're creating a symbol table, reset it now
+ if (CreateSymbolTable) {
+ symTabSize = 0;
+ symTab.clear();
}
// Write magic string to archive.
ArchiveFile << ARFILE_MAGIC;
- // Write the symbol table and string pool
- writeSymbolTable(ArchiveFile);
-
- //Loop over all member files, and add to the archive.
- for ( unsigned i = 0; i < members.size(); ++i) {
- if(ArchiveFile.tellp() % 2 != 0)
- ArchiveFile << ARFILE_PAD;
- writeMember(members[i],ArchiveFile);
+ // Loop over all member files, and write them out. Note that this also
+ // builds the symbol table, symTab.
+ for (MembersList::iterator I = begin(), E = end(); I != E; ++I) {
+ if (writeMember(*I, ArchiveFile, CreateSymbolTable,
+ TruncateNames, Compress, ErrMsg)) {
+ if (TmpArchive.exists())
+ TmpArchive.eraseFromDisk();
+ ArchiveFile.close();
+ return true;
+ }
}
- //Close archive file.
+ // Close archive file.
ArchiveFile.close();
-}
-// vim: sw=2 ai
+ // Write the symbol table
+ if (CreateSymbolTable) {
+ // At this point we have written a file that is a legal archive but it
+ // doesn't have a symbol table in it. To aid in faster reading and to
+ // ensure compatibility with other archivers we need to put the symbol
+ // table first in the file. Unfortunately, this means mapping the file
+ // we just wrote back in and copying it to the destination file.
+
+ // Map in the archive we just wrote.
+ sys::MappedFile arch;
+ if (arch.open(TmpArchive, sys::MappedFile::READ_ACCESS, ErrMsg))
+ return true;
+ const char* base;
+ if (!(base = (const char*) arch.map(ErrMsg)))
+ return true;
+
+ // Open another temporary file in order to avoid invalidating the
+ // mmapped data
+ sys::Path FinalFilePath = archPath;
+ if (FinalFilePath.createTemporaryFileOnDisk(ErrMsg))
+ return true;
+ sys::RemoveFileOnSignal(FinalFilePath);
+
+ std::ofstream FinalFile(FinalFilePath.c_str(), io_mode);
+ if (!FinalFile.is_open() || FinalFile.bad()) {
+ if (TmpArchive.exists())
+ TmpArchive.eraseFromDisk();
+ if (ErrMsg)
+ *ErrMsg = "Error opening archive file: " + FinalFilePath.toString();
+ return true;
+ }
+
+ // Write the file magic number
+ FinalFile << ARFILE_MAGIC;
+
+ // If there is a foreign symbol table, put it into the file now. Most
+ // ar(1) implementations require the symbol table to be first but llvm-ar
+ // can deal with it being after a foreign symbol table. This ensures
+ // compatibility with other ar(1) implementations as well as allowing the
+ // archive to store both native .o and LLVM .bc files, both indexed.
+ if (foreignST) {
+ if (writeMember(*foreignST, FinalFile, false, false, false, ErrMsg)) {
+ FinalFile.close();
+ if (TmpArchive.exists())
+ TmpArchive.eraseFromDisk();
+ return true;
+ }
+ }
+
+ // Put out the LLVM symbol table now.
+ writeSymbolTable(FinalFile);
+
+ // Copy the temporary file contents being sure to skip the file's magic
+ // number.
+ FinalFile.write(base + sizeof(ARFILE_MAGIC)-1,
+ arch.size()-sizeof(ARFILE_MAGIC)+1);
+
+ // Close up shop
+ FinalFile.close();
+ arch.close();
+
+ // Move the final file over top of TmpArchive
+ if (FinalFilePath.renamePathOnDisk(TmpArchive, ErrMsg))
+ return true;
+ }
+
+ // Before we replace the actual archive, we need to forget all the
+ // members, since they point to data in that old archive. We need to do
+ // this because we cannot replace an open file on Windows.
+ cleanUpMemory();
+
+ if (TmpArchive.renamePathOnDisk(archPath, ErrMsg))
+ return true;
+
+ return false;
+}