diff options
| author | Michael Adam <obnox@samba.org> | 2008-01-11 13:03:16 +0100 |
|---|---|---|
| committer | Karolin Seeger <kseeger@samba.org> | 2008-05-07 16:09:21 +0200 |
| commit | e1210bcd54525737296d2d7e4b3e6f4c332376f7 (patch) | |
| tree | 4c6182b1152323e6c6f2e6ed7121ecc16eadec82 /source/locking | |
| parent | 341544230f7f8415140cee95ed2066f49f609aae (diff) | |
| download | samba-e1210bcd54525737296d2d7e4b3e6f4c332376f7.tar.gz samba-e1210bcd54525737296d2d7e4b3e6f4c332376f7.tar.xz samba-e1210bcd54525737296d2d7e4b3e6f4c332376f7.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
(cherry picked from commit df264bf3e00d7d77afcf55e54d2669b9ffa9af4a)
Diffstat (limited to 'source/locking')
| -rw-r--r-- | source/locking/posix.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/source/locking/posix.c b/source/locking/posix.c index f42d1ec6f8a..844a86e8636 100644 --- a/source/locking/posix.c +++ b/source/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); } |