ceph: fix session check on mds reply
authorSage Weil <sage@newdream.net>
Sun, 21 Mar 2010 03:50:58 +0000 (20:50 -0700)
committerSage Weil <sage@newdream.net>
Tue, 23 Mar 2010 14:47:05 +0000 (07:47 -0700)
Fix a broken check that a reply came back from the same MDS we sent the
request to.  I don't think a case that actually triggers this would ever
come up in practice, but it's clearly wrong and easy to fix.

Reported-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Sage Weil <sage@newdream.net>
fs/ceph/mds_client.c

index ad0fbc3128d3b48c985d8fdd6f0070495c83fe60..5268d404963c8eca8bf994ffb652bc2ea8bd4845 100644 (file)
@@ -1778,7 +1778,7 @@ static void handle_reply(struct ceph_mds_session *session, struct ceph_msg *msg)
        dout("handle_reply %p\n", req);
 
        /* correct session? */
-       if (!req->r_session && req->r_session != session) {
+       if (req->r_session != session) {
                pr_err("mdsc_handle_reply got %llu on session mds%d"
                       " not mds%d\n", tid, session->s_mds,
                       req->r_session ? req->r_session->s_mds : -1);