diff options
| author | David Kalnischkies <david@kalnischkies.de> | 2019-01-23 22:50:45 +0100 |
|---|---|---|
| committer | David Kalnischkies <david@kalnischkies.de> | 2019-01-24 00:33:16 +0100 |
| commit | 73e3459689c05cd62f15c29d2faddb0fc215ef5e (patch) | |
| tree | 5cdddd899253499de6f9b43b171ee27a8674a0f7 /apt-pkg/deb | |
| parent | e2965b0b6bdd68ffcad0e06d11755412a7e16e50 (diff) | |
Merge and reuse tmp file handling across the board
Having many rather similar implementations especially if one is exported
while others aren't (and the rest of it not factored out at all) seems
suboptimal.
Diffstat (limited to 'apt-pkg/deb')
| -rw-r--r-- | apt-pkg/deb/debindexfile.cc | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/apt-pkg/deb/debindexfile.cc b/apt-pkg/deb/debindexfile.cc index f7e3c7a5c..25e0a3312 100644 --- a/apt-pkg/deb/debindexfile.cc +++ b/apt-pkg/deb/debindexfile.cc @@ -315,13 +315,10 @@ const pkgIndexFile::Type *debStringPackageIndex::GetType() const debStringPackageIndex::debStringPackageIndex(std::string const &content) : pkgDebianIndexRealFile("", false), d(NULL) { - char fn[1024]; - std::string const tempdir = GetTempDir(); - snprintf(fn, sizeof(fn), "%s/%s.XXXXXX", tempdir.c_str(), "apt-tmp-index"); - int const fd = mkstemp(fn); - File = fn; - FileFd::Write(fd, content.data(), content.length()); - close(fd); + FileFd fd; + GetTempFile("apt-tmp-index", false, &fd); + fd.Write(content.data(), content.length()); + File = fd.Name(); } debStringPackageIndex::~debStringPackageIndex() { |