From df264bf3e00d7d77afcf55e54d2669b9ffa9af4a Mon Sep 17 00:00:00 2001 From: Michael Adam Date: Fri, 11 Jan 2008 13:03:16 +0100 Subject: [PATCH] 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 --- source/locking/posix.c | 4 ---- 1 file changed, 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); } -- 2.47.3