diff options
| author | Volker Lendecke <vl@samba.org> | 2012-07-17 22:43:06 +0200 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 2012-07-19 03:40:17 +0200 |
| commit | 8ef968a23dffb2e1c1518f7489d4ab0b14a71118 (patch) | |
| tree | 9e9cd5237df264028fb1752ffd82a5e1ea358c62 /source3/smbd/close.c | |
| parent | a6b58367410abc85fc079910f3f193eaa386f9f1 (diff) | |
| download | samba-8ef968a23dffb2e1c1518f7489d4ab0b14a71118.tar.gz samba-8ef968a23dffb2e1c1518f7489d4ab0b14a71118.tar.bz2 samba-8ef968a23dffb2e1c1518f7489d4ab0b14a71118.zip | |
s3-aio: Panic if we try to close a fsp with outstanding aio requests
The core smbd must have taken care of this. If we don't do this properly,
we have a race of the close(2) against a pwrite(2). We might end up
writing to the wrong file.
Signed-off-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Thu Jul 19 03:40:17 CEST 2012 on sn-devel-104
Diffstat (limited to 'source3/smbd/close.c')
| -rw-r--r-- | source3/smbd/close.c | 16 |
1 files changed, 15 insertions, 1 deletions
diff --git a/source3/smbd/close.c b/source3/smbd/close.c index 720ffa7b64..3c5b6d74eb 100644 --- a/source3/smbd/close.c +++ b/source3/smbd/close.c @@ -707,7 +707,21 @@ static NTSTATUS close_normal_file(struct smb_request *req, files_struct *fsp, NTSTATUS tmp; connection_struct *conn = fsp->conn; - aio_fsp_close(fsp); + if (fsp->num_aio_requests != 0) { + char *str; + /* + * reply_close and the smb2 close must have taken care of + * this. No other callers of close_file should ever have + * created async I/O. + * + * We need to panic here because if we close() the fd while we + * have outstanding async I/O requests, in the worst case we + * could end up writing to the wrong file. + */ + DEBUG(0, ("fsp->num_aio_requests=%u\n", + fsp->num_aio_requests)); + smb_panic("can not close with outstanding aio requests"); + } /* * If we're flushing on a close we can get a write |