ceph: return proper bool type to caller instead of pointer
authorChengguang Xu <cgxu519@gmx.com>
Tue, 13 Mar 2018 02:43:45 +0000 (10:43 +0800)
committerIlya Dryomov <idryomov@gmail.com>
Mon, 2 Apr 2018 08:12:49 +0000 (10:12 +0200)
Change to return true/false only for bool type return code.

Signed-off-by: Chengguang Xu <cgxu519@gmx.com>
Reviewed-by: "Yan, Zheng" <zyan@redhat.com>
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
fs/ceph/mds_client.c

index a511a777ffb997fbc45e3dda7ceb6c6a73aada37..3a555b60444190d5edd786a2b4c9afac562a3752 100644 (file)
@@ -419,9 +419,10 @@ struct ceph_mds_session *__ceph_lookup_mds_session(struct ceph_mds_client *mdsc,
 
 static bool __have_session(struct ceph_mds_client *mdsc, int mds)
 {
-       if (mds >= mdsc->max_sessions)
+       if (mds >= mdsc->max_sessions || !mdsc->sessions[mds])
                return false;
-       return mdsc->sessions[mds];
+       else
+               return true;
 }
 
 static int __verify_registered_session(struct ceph_mds_client *mdsc,