On 02/24/2012 03:32 PM, Roland Dreier wrote:
On Wed, Feb 22, 2012 at 1:43 PM, Steve Wise<[email protected]>  wrote:
@@ -631,6 +631,8 @@ static void cm_conn_req_handler(struct iwcm_id_private 
*listen_id_priv,
        spin_lock_irqsave(&listen_id_priv->lock, flags);
        if (listen_id_priv->state != IW_CM_STATE_LISTEN) {
                spin_unlock_irqrestore(&listen_id_priv->lock, flags);
+               iw_cm_reject(cm_id, NULL, 0);
+               iw_destroy_cm_id(cm_id);
                goto out;
        }
        spin_unlock_irqrestore(&listen_id_priv->lock, flags);
I think I see your bug.  Look at the whole cm_conn_req_handler()
function.  Where is this new code relative to where you initialize cm_id?

  - R.

duh. jeeze. ok, lemme try again.  I'll add printks to make sure I'm really 
hitting this path too.




--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to