Dan, We should have another patch set coming shortly - will address it in that patch. -- james s On 12/16/2011 3:27 PM, Dan Carpenter wrote:
Hello James Smart, This is a semi-automatic email about new static checker warnings. The patch cb69f7decc37: "[SCSI] lpfc 8.3.28: Add support for ABTS failure handling" from Dec 13, 2011, leads to the following Smatch complaint: drivers/scsi/lpfc/lpfc_sli.c +8391 lpfc_sli4_abts_err_handler() error: we previously assumed 'ndlp' could be null (see line 8387) drivers/scsi/lpfc/lpfc_sli.c 8386 8387 if (!ndlp || !NLP_CHK_NODE_ACT(ndlp)) ^^^^^ New check. 8388 lpfc_printf_log(phba, KERN_INFO, LOG_SLI, 8389 "3115 Node Context not found, driver " 8390 "ignoring abts err event\n"); 8391 vport = ndlp->vport; ^^^^^^^^^^^ New dereference. Maybe the intent was to return after the print statement? 8392 lpfc_printf_log(phba, KERN_WARNING, LOG_SLI, 8393 "3116 Port generated FCP XRI ABORT event on " regards, dan carpenter
-- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html