Smatch gets confused because we sometimes refer to "server->srv_mutex" and
sometimes to "sess->server->srv_mutex".  They refer to the same lock so
let's just make this consistent.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
 
        rc = allocate_mid(ses, in_buf, &midQ);
        if (rc) {
-               mutex_unlock(&ses->server->srv_mutex);
+               mutex_unlock(&server->srv_mutex);
                /* Update # of requests on wire to server */
                add_credits(server, &credits, 0);
                return rc;