public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Jan Hudec <bulb@ucw.cz>
To: linux-fsdevel@vger.kerner.org, linux-kernel@vger.kernel.org
Subject: Race in open(O_CREAT|O_EXCL) and network filesystem
Date: Sun, 28 Jul 2002 18:52:56 +0200	[thread overview]
Message-ID: <20020728165256.GA4631@vagabond> (raw)

Hi all,

maybe I'm blind, but I think there is a race featuring
open(O_CREAT|O_EXCL) and nfs or any other network fs.

What may happen is:

client A: open_namei looks up the inode
	driver queries server and gets ENOENT
client B: open_namei looks up the inode
	driver queries server and gets ENOENT
client A: open_namei calls create method
	driver requests file to be created and is successful
client B: open_namei calls create method
	dirver requests file to be created and since it does not know,
	cant specify exclusion, thus is succesful
client A: open_namei does no more checks and thus open succeeds
client B: open succeeds too here - and it shouldn't

Since many applications rely on this working correctly (especialy
mailboxes are locked using exclusive creates and mounting them over NFS
is quite common).

So, can someone please answer:

1) Is there some reason this can't happen that I overlooked?
2) If it is a problem (comment in NFS suggest so), I can see two ways of
handling this. Either pass the flags to the create method, or restart
the open when create returns EEXISTS. Which one would be prefered?
3) How to fix NFS to add exclude flag to the NFSv3 request?

-------------------------------------------------------------------------------
						 Jan 'Bulb' Hudec <bulb@ucw.cz>

             reply	other threads:[~2002-07-29 11:14 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-07-28 16:52 Jan Hudec [this message]
2002-07-29 11:45 ` Race in open(O_CREAT|O_EXCL) and network filesystem Trond Myklebust
2002-07-29 11:50 ` Neil Brown
2002-07-29 15:02   ` Andreas Dilger
2002-07-29 23:04     ` Neil Brown
2002-07-29 23:44       ` Andreas Dilger

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20020728165256.GA4631@vagabond \
    --to=bulb@ucw.cz \
    --cc=linux-fsdevel@vger.kerner.org \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox