Now I'm confused about this patch for another reason: > @@ -1429,6 +1431,8 @@ static int srp_reset_device(struct scsi_ > return FAILED; > if (req->tsk_status) > return FAILED; > + if (!req->scmnd->device) > + return FAILED; > > spin_lock_irq(target->scsi_host->host_lock);
This adds the check *after* the call to srp_send_tsk_mgmt() -- which is where scmnd->device will be dereferenced. So how does this fix the bug? - R. _______________________________________________ general mailing list general@lists.openfabrics.org http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general
