From 9644b5e1be38a8eddaa70531891cc9da98c93394 Mon Sep 17 00:00:00 2001
From: Martin Pärtel <martin.partel@gmail.com>
Date: Fri, 14 Sep 2012 17:35:58 +0300
Subject: Added a note about a race condition when creating new files.

---
 src/bindfs.c | 2 ++
 1 file changed, 2 insertions(+)

(limited to 'src')

diff --git a/src/bindfs.c b/src/bindfs.c
index df0746c..dbbb588 100644
--- a/src/bindfs.c
+++ b/src/bindfs.c
@@ -290,6 +290,8 @@ static int getattr_common(const char *procpath, struct stat *stbuf)
     return 0;
 }
 
+/* FIXME: another thread may race to see the old owner before the chown is done.
+          Is there a scenario where this compromises security? Or application correctness? */
 static void chown_new_file(const char *path, struct fuse_context *fc, int (*chown_func)(const char*, uid_t, gid_t))
 {
     uid_t file_owner;
-- 
cgit v1.2.3