diff options
author | wm4 <wm4@nowhere> | 2013-11-30 22:40:51 +0100 |
---|---|---|
committer | wm4 <wm4@nowhere> | 2013-11-30 22:40:51 +0100 |
commit | 95cfe58e3db9d939abe7a9a26116c1d576eed60b (patch) | |
tree | 98a4738f2f989c900702d84ef8a257f9413389af /mpvcore/player/configfiles.c | |
parent | eea69682a6a874d540f9fc576c937466970713f6 (diff) | |
download | mpv-95cfe58e3db9d939abe7a9a26116c1d576eed60b.tar.bz2 mpv-95cfe58e3db9d939abe7a9a26116c1d576eed60b.tar.xz |
Use O_CLOEXEC when creating FDs
This is needed so that new processes (created with fork+exec) don't
inherit open files, which can be important for a number of reasons.
Since O_CLOEXEC is relatively new (POSIX.1-2008, before that Linux
specific), we #define it to 0 in io.h to prevent compilation errors on
older/crappy systems. At least this is the plan.
input.c creates a pipe. For that, add a mp_set_cloexec() function (which
is based on Weston's code in vo_wayland.c, but more correct). We could
use pipe2() instead, but that is Linux specific. Technically, we have a
race condition, but it won't matter.
Diffstat (limited to 'mpvcore/player/configfiles.c')
-rw-r--r-- | mpvcore/player/configfiles.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/mpvcore/player/configfiles.c b/mpvcore/player/configfiles.c index 29350c0aed..31e3d0e75d 100644 --- a/mpvcore/player/configfiles.c +++ b/mpvcore/player/configfiles.c @@ -51,7 +51,6 @@ bool mp_parse_cfgfiles(struct MPContext *mpctx) struct MPOpts *opts = mpctx->opts; m_config_t *conf = mpctx->mconfig; char *conffile; - int conffile_fd; if (!opts->load_config) return true; if (!m_config_parse_config_file(conf, MPLAYER_CONFDIR "/mpv.conf", 0) < 0) @@ -60,11 +59,11 @@ bool mp_parse_cfgfiles(struct MPContext *mpctx) if ((conffile = mp_find_user_config_file("config")) == NULL) MP_ERR(mpctx, "mp_find_user_config_file(\"config\") problem\n"); else { - if ((conffile_fd = open(conffile, O_CREAT | O_EXCL | O_WRONLY, - 0666)) != -1) { + int fd = open(conffile, O_CREAT | O_EXCL | O_WRONLY | O_CLOEXEC, 0666); + if (fd != -1) { MP_INFO(mpctx, "Creating config file: %s\n", conffile); - write(conffile_fd, DEF_CONFIG, sizeof(DEF_CONFIG) - 1); - close(conffile_fd); + write(fd, DEF_CONFIG, sizeof(DEF_CONFIG) - 1); + close(fd); } if (m_config_parse_config_file(conf, conffile, 0) < 0) return false; |