Slightly simplify logic in process_lockingX()

The "else" is pointless here, we did a "return True" in the if branch.

diff --git a/source3/smbd/blocking.c b/source3/smbd/blocking.c
index 414ea9fc5cc14809f28d505fcc58314df2173737..6428c3684baa5dd23f1fa4df3e947e888c2e4261 100644 (file)
--- a/source3/smbd/blocking.c
+++ b/source3/smbd/blocking.c
@@ -470,14 +470,15 @@ static bool process_lockingX(blocking_lock_record *blr)
 
                reply_lockingX_success(blr);
                return True;
-       } else if (!NT_STATUS_EQUAL(status,NT_STATUS_LOCK_NOT_GRANTED) &&
-                       !NT_STATUS_EQUAL(status,NT_STATUS_FILE_LOCK_CONFLICT)) {
-                       /*
-                        * We have other than a "can't get lock"
-                        * error. Free any locks we had and return an error.
-                        * Return True so we get dequeued.
-                        */
+       }
 
+       if (!NT_STATUS_EQUAL(status,NT_STATUS_LOCK_NOT_GRANTED) &&
+           !NT_STATUS_EQUAL(status,NT_STATUS_FILE_LOCK_CONFLICT)) {
+               /*
+                * We have other than a "can't get lock"
+                * error. Free any locks we had and return an error.
+                * Return True so we get dequeued.
+                */
                blocking_lock_reply_error(blr, status);
                return True;
        }