summaryrefslogtreecommitdiff
path: root/source3
diff options
context:
space:
mode:
authorMichael Adam <obnox@samba.org>2008-01-11 13:03:16 +0100
committerMichael Adam <obnox@samba.org>2008-04-21 00:21:24 +0200
commit96e9e83ee021db69179fe924144e6ba3dea1b73d (patch)
tree6f2cae740d0d6440ed97e689071b8850f2ec81a1 /source3
parent3756467db6a661be91f4aeb484000e993e4a9a4c (diff)
downloadsamba-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')
-rw-r--r--source3/locking/posix.c4
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);
}