From: Xiubo Li <xiubli@xxxxxxxxxx>
There is no reason that the con->private will be NULL for mon client,
once it is here in dispatch() routine the con->monc->private should
already correctly set done. And also just before the dispatch() in
try_read() it will also reference the con->monc->private to allocate
memory for in_msg.
Signed-off-by: Xiubo Li <xiubli@xxxxxxxxxx>
---
net/ceph/mon_client.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/net/ceph/mon_client.c b/net/ceph/mon_client.c
index 7256c402ebaa..9d9e4e4ea600 100644
--- a/net/ceph/mon_client.c
+++ b/net/ceph/mon_client.c
@@ -1233,9 +1233,6 @@ static void dispatch(struct ceph_connection *con, struct ceph_msg *msg)
struct ceph_mon_client *monc = con->private;
int type = le16_to_cpu(msg->hdr.type);
- if (!monc)
- return;
-
switch (type) {
case CEPH_MSG_AUTH_REPLY:
handle_auth_reply(monc, msg);
--
2.21.0
