smbd: Simplify reopen_from_fsp() with an early return

Review with git show -b Signed-off-by: Volker Lendecke <[email protected]> Reviewed-by: Jeremy Allison <[email protected]>
Roo4L · Feb 1, 2022 · e93f463 · e93f463
1 parent 1d6762d
commit e93f463
Showing 1 changed file with 16 additions and 16 deletions.
diff --git a/source3/smbd/open.c b/source3/smbd/open.c
@@ -1227,25 +1227,25 @@ static NTSTATUS reopen_from_fsp(struct files_struct *fsp,
 	status = reopen_from_procfd(fsp,
 				    flags,
 				    mode);
-	if (NT_STATUS_EQUAL(status, NT_STATUS_MORE_PROCESSING_REQUIRED)) {
-		/*
-		 * Close the existing pathref fd and set the fsp flag
-		 * is_pathref to false so we get a "normal" fd this
-		 * time.
-		 */
-		status = fd_close(fsp);
-		if (!NT_STATUS_IS_OK(status)) {
-			return status;
-		}
-
-		fsp->fsp_flags.is_pathref = false;
+	if (!NT_STATUS_EQUAL(status, NT_STATUS_MORE_PROCESSING_REQUIRED)) {
+		return status;
+	}
 
-		status = fd_open_atomic(fsp,
-					flags,
-					mode,
-					p_file_created);
+	/*
+	 * Close the existing pathref fd and set the fsp flag
+	 * is_pathref to false so we get a "normal" fd this time.
+	 */
+	status = fd_close(fsp);
+	if (!NT_STATUS_IS_OK(status)) {
+		return status;
 	}
 
+	fsp->fsp_flags.is_pathref = false;
+
+	status = fd_open_atomic(fsp,
+				flags,
+				mode,
+				p_file_created);
 	return status;
 }