Simplify allocate_mid() slightly: Remove some unnecessary "else" branches

Simplify allocate_mid() slightly: Remove some unnecessary "else" branches

Signed-off-by: Volker Lendecke <vl@samba.org>
Acked-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>

1 files changed, 6 insertions, 2 deletions

diff --git a/fs/cifs/transport.c b/fs/cifs/transport.c
index e80210693ff7..c98f929315f2 100644
--- a/fs/cifs/transport.c
+++ b/fs/cifs/transport.c
@@ -385,10 +385,14 @@ static int allocate_mid(struct cifsSesInfo *ses, struct smb_hdr *in_buf,
 {
         if (ses->server->tcpStatus == CifsExiting) {
                 return -ENOENT;
-        } else if (ses->server->tcpStatus == CifsNeedReconnect) {
+        }
+
+        if (ses->server->tcpStatus == CifsNeedReconnect) {
                 cFYI(1, ("tcp session dead - return to caller to retry"));
                 return -EAGAIN;
-        } else if (ses->status != CifsGood) {
+        }
+
+        if (ses->status != CifsGood) {
                 /* check if SMB session is bad because we are setting it up */
                 if ((in_buf->Command != SMB_COM_SESSION_SETUP_ANDX) &&
                         (in_buf->Command != SMB_COM_NEGOTIATE))