diff options
| author | eldritch horrors <pennae@lix.systems> | 2024-03-18 14:52:04 +0100 |
|---|---|---|
| committer | eldritch horrors <pennae@lix.systems> | 2024-03-18 15:42:52 -0600 |
| commit | f38ae92a38a66b597dbd6975219d6ddb4f52fa4f (patch) | |
| tree | be6e1ed225e3d77a3c1fbcef1778169878326cbe /src/libstore/binary-cache-store.cc | |
| parent | 0f518f44e289114eb4828c8232bc0b79c7a85ac7 (diff) | |
libutil: make AutoCloseFD a better resource
add a reset() method to close the wrapped fd instead of assigning magic
constants. also make the from-fd constructor explicit so you can't
accidentally assign the *wrong* magic constant, or even an unrelated
integer that also just happens to be an fd by pure chance.
Change-Id: I51311b0f6e040240886b5103d39d1794a6acc325
Diffstat (limited to 'src/libstore/binary-cache-store.cc')
| -rw-r--r-- | src/libstore/binary-cache-store.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/libstore/binary-cache-store.cc b/src/libstore/binary-cache-store.cc index 2e1f84859..502fac05f 100644 --- a/src/libstore/binary-cache-store.cc +++ b/src/libstore/binary-cache-store.cc @@ -124,7 +124,7 @@ void BinaryCacheStore::writeNarInfo(ref<NarInfo> narInfo) AutoCloseFD openFile(const Path & path) { - auto fd = open(path.c_str(), O_RDONLY | O_CLOEXEC); + AutoCloseFD fd{open(path.c_str(), O_RDONLY | O_CLOEXEC)}; if (!fd) throw SysError("opening file '%1%'", path); return fd; |