Roland Dreier wrote:
 >   if (status < 0) {
 > -         ep->com.cm_id->rem_ref(ep->com.cm_id);
 > +         if (ep->com.cm_id)
 > +                 ep->com.cm_id->rem_ref(ep->com.cm_id);

Steve, does this make sense?
I actually think that ep->com.cm_id is always valid when connect_reply_upcall() is called. But I'd have to test it more. I'd rather not add this change unless someone convinces me that there actually is a path where the cm_id null...
--
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