Change fd_close_posix() to return int instead of NTSTATUS.

The errno is handed up through the VFS layer to the callers.

Michael
(This used to be commit d928e6648d61cf2d3c1b77db440efb835b729a84)

1 files changed, 4 insertions, 12 deletions

diff --git a/source3/locking/posix.c b/source3/locking/posix.c
index 844a86e863..aedc12dede 100644
--- a/source3/locking/posix.c
+++ b/source3/locking/posix.c
@@ -607,7 +607,7 @@ static size_t get_posix_pending_close_entries(TALLOC_CTX *mem_ctx,
  to delete all locks on this fsp before this function is called.
 ****************************************************************************/
 
-NTSTATUS fd_close_posix(struct files_struct *fsp)
+int fd_close_posix(struct files_struct *fsp)
 {
 	int saved_errno = 0;
 	int ret;
@@ -620,11 +620,7 @@ NTSTATUS fd_close_posix(struct files_struct *fsp)
 		 * which will lose all locks on all fd's open on this dev/inode,
 		 * just close.
 		 */
-		ret = close(fsp->fh->fd);
-		if (ret == -1) {
-			return map_nt_error_from_unix(errno);
-		}
-		return NT_STATUS_OK;
+		return close(fsp->fh->fd);
 	}
 
 	if (get_windows_lock_ref_count(fsp)) {
@@ -635,7 +631,7 @@ NTSTATUS fd_close_posix(struct files_struct *fsp)
 		 */
 
 		add_fd_to_close_entry(fsp);
-		return NT_STATUS_OK;
+		return 0;
 	}
 
 	/*
@@ -678,11 +674,7 @@ NTSTATUS fd_close_posix(struct files_struct *fsp)
 		ret = -1;
 	} 
 
-	if (ret == -1) {
-		return map_nt_error_from_unix(errno);
-	}
-
-	return NT_STATUS_OK;
+	return ret;
 }
 
 /****************************************************************************