Skip to content

M
mpd
Commit bbea6564 authored by Max Kellermann's avatar Max Kellermann
Browse files
Options

fs/output/FileOutputStream: use open_cloexec()

Support operating systems that don't have O_CLOEXEC.
parent 1ca8d7ad
Hide whitespace changes
Inline Side-by-side
Showing with 4 additions and 3 deletions
src/fs/output/FileOutputStream.cxx
View file @ bbea6564
...@@ -20,6 +20,7 @@ ...@@ -20,6 +20,7 @@
#include "config.h" #include "config.h"
#include "FileOutputStream.hxx" #include "FileOutputStream.hxx"
#include "fs/FileSystem.hxx" #include "fs/FileSystem.hxx"
#include "system/fd_util.h"
#include "util/Error.hxx" #include "util/Error.hxx"
#ifdef WIN32 #ifdef WIN32
...@@ -81,9 +82,9 @@ FileOutputStream::Cancel() ...@@ -81,9 +82,9 @@ FileOutputStream::Cancel()
FileOutputStream::FileOutputStream(Path _path, Error &error) FileOutputStream::FileOutputStream(Path _path, Error &error)
:path(_path), :path(_path),
fd(open(path.c_str(), fd(open_cloexec(path.c_str(),
O_WRONLY|O_CLOEXEC|O_CREAT|O_TRUNC|O_NOCTTY, O_WRONLY|O_CREAT|O_TRUNC,
0666)) 0666))
{ {
if (fd < 0) if (fd < 0)
error.FormatErrno("Failed to create %s", path.c_str()); error.FormatErrno("Failed to create %s", path.c_str());
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment