diff options
| author | Michael Adam <obnox@samba.org> | 2008-01-11 13:03:16 +0100 |
|---|---|---|
| committer | Michael Adam <obnox@samba.org> | 2008-04-21 00:21:24 +0200 |
| commit | 96e9e83ee021db69179fe924144e6ba3dea1b73d (patch) | |
| tree | 6f2cae740d0d6440ed97e689071b8850f2ec81a1 /source3/locking | |
| parent | 3756467db6a661be91f4aeb484000e993e4a9a4c (diff) | |
| download | samba-96e9e83ee021db69179fe924144e6ba3dea1b73d.tar.gz samba-96e9e83ee021db69179fe924144e6ba3dea1b73d.tar.bz2 samba-96e9e83ee021db69179fe924144e6ba3dea1b73d.zip | |
Don't modify the fsp in fd_close_posix() anymore.
Now that it is inside the vfs layer, this function should
not alter the fsp (i.e. set fsp->fh->fd = -1) anymore.
That belongs above the vfs layer.
Michael
(This used to be commit df264bf3e00d7d77afcf55e54d2669b9ffa9af4a)
Diffstat (limited to 'source3/locking')
| -rw-r--r-- | source3/locking/posix.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/source3/locking/posix.c b/source3/locking/posix.c index f42d1ec6f8..844a86e863 100644 --- a/source3/locking/posix.c +++ b/source3/locking/posix.c @@ -621,7 +621,6 @@ NTSTATUS fd_close_posix(struct files_struct *fsp) * just close. */ ret = close(fsp->fh->fd); - fsp->fh->fd = -1; if (ret == -1) { return map_nt_error_from_unix(errno); } @@ -636,7 +635,6 @@ NTSTATUS fd_close_posix(struct files_struct *fsp) */ add_fd_to_close_entry(fsp); - fsp->fh->fd = -1; return NT_STATUS_OK; } @@ -680,8 +678,6 @@ NTSTATUS fd_close_posix(struct files_struct *fsp) ret = -1; } - fsp->fh->fd = -1; - if (ret == -1) { return map_nt_error_from_unix(errno); } |