From 1baaaea48b61b3711fce0fa12e63ff6823c6e81c Mon Sep 17 00:00:00 2001 From: Bart Van Assche Date: Fri, 4 May 2012 13:49:04 +0000 Subject: [PATCH] ib_srpt: Test earlier during login whether a port has been disabled git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@4282 d57e44dd-8a1f-0410-8b47-8ef2f437770f --- srpt/src/ib_srpt.c | 21 +++++++++++---------- 1 file changed, 11 insertions(+), 10 deletions(-) diff --git a/srpt/src/ib_srpt.c b/srpt/src/ib_srpt.c index bb96819d9..f255d7298 100644 --- a/srpt/src/ib_srpt.c +++ b/srpt/src/ib_srpt.c @@ -2551,6 +2551,16 @@ static int srpt_cm_req_recv(struct ib_cm_id *cm_id, spin_lock_irq(&sdev->spinlock); + if (!sdev->enabled) { + rej->reason = cpu_to_be32( + SRP_LOGIN_REJ_INSUFFICIENT_RESOURCES); + PRINT_ERROR("rejected SRP_LOGIN_REQ because the target %s (%s)" + " is not enabled", + sdev->scst_tgt->tgt_name, sdev->device->name); + spin_unlock_irq(&sdev->spinlock); + goto reject; + } + if ((req->req_flags & SRP_MTCH_ACTION) == SRP_MULTICHAN_SINGLE) { struct srpt_rdma_ch *ch2; @@ -2581,16 +2591,6 @@ restart: list_add_tail(&ch->list, &sdev->rch_list); ch->thread = thread; - if (!sdev->enabled) { - rej->reason = cpu_to_be32( - SRP_LOGIN_REJ_INSUFFICIENT_RESOURCES); - PRINT_ERROR("rejected SRP_LOGIN_REQ because the target %s (%s)" - " is not enabled", - sdev->scst_tgt->tgt_name, sdev->device->name); - spin_unlock_irq(&sdev->spinlock); - goto reject; - } - spin_unlock_irq(&sdev->spinlock); ret = srpt_ch_qp_rtr(ch, ch->qp); @@ -2662,6 +2662,7 @@ free_ring: free_ch: cm_id->context = NULL; kfree(ch); + ch = NULL; BUG_ON(ret == 0);