Clean up. AFAICT, there is no way to reach the out_free_conn label
with @old set to a non-NULL value, so the expire_client(old) call
is never reached and can be removed.
Reviewed-by: Jeff Layton <jlayton@kernel.org>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
        return status;
 
 out_expired_error:
-       old = NULL;
        /*
         * Revert the slot seq_nr change so the server will process
         * the client's resend instead of returning a cached response.
 out_free_conn:
        spin_unlock(&nn->client_lock);
        free_conn(conn);
-       if (old)
-               expire_client(old);
 out_free_session:
        __free_session(new);
 out_release_drc_mem: