summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAl Viro <viro@zeniv.linux.org.uk>2012-08-26 11:01:04 -0400
committerAl Viro <viro@zeniv.linux.org.uk>2012-09-26 21:10:01 -0400
commitc6f3d81115989e274c42a852222b80d2e14ced6f (patch)
tree8a0e17df64381ab883acd76f5190b33fc80126ea
parentddd3e0771bc7b869c550687c204e21f0155d5496 (diff)
downloadop-kernel-dev-c6f3d81115989e274c42a852222b80d2e14ced6f.zip
op-kernel-dev-c6f3d81115989e274c42a852222b80d2e14ced6f.tar.gz
don't leak O_CLOEXEC into ->f_flags
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
-rw-r--r--fs/open.c2
-rw-r--r--fs/proc/fd.c2
2 files changed, 2 insertions, 2 deletions
diff --git a/fs/open.c b/fs/open.c
index 3076001..03028d0 100644
--- a/fs/open.c
+++ b/fs/open.c
@@ -814,7 +814,7 @@ static inline int build_open_flags(int flags, umode_t mode, struct open_flags *o
op->mode = 0;
/* Must never be set by userspace */
- flags &= ~FMODE_NONOTIFY;
+ flags &= ~FMODE_NONOTIFY & ~O_CLOEXEC;
/*
* O_SYNC is implemented as __O_SYNC|O_DSYNC. As many places only
diff --git a/fs/proc/fd.c b/fs/proc/fd.c
index 9cef449..f28a875 100644
--- a/fs/proc/fd.c
+++ b/fs/proc/fd.c
@@ -36,7 +36,7 @@ static int seq_show(struct seq_file *m, void *v)
if (file) {
struct fdtable *fdt = files_fdtable(files);
- f_flags = file->f_flags & ~O_CLOEXEC;
+ f_flags = file->f_flags;
if (close_on_exec(fd, fdt))
f_flags |= O_CLOEXEC;
OpenPOWER on IntegriCloud