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

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);
 	}