Re: [PATCH] scsi: lpfc: prevent potential null pointer dereference

2017-05-31 Thread Martin K. Petersen
Gustavo A., > Null check at line 966: if (ndlp) {, implies that ndlp might be NULL. > Functions lpfc_nlp_set_state() and lpfc_issue_els_prli() dereference > pointer ndlp. Include these function calls inside the IF block that > tests pointer ndlp. Applied to 4.12/scsi-fixes. Thank you! --

Re: [PATCH] scsi: lpfc: prevent potential null pointer dereference

2017-05-29 Thread James Smart
Looks good Signed-off-by: James Smart -- james On 5/23/2017 8:09 AM, Gustavo A. R. Silva wrote: Null check at line 966: if (ndlp) {, implies that ndlp might be NULL. Functions lpfc_nlp_set_state() and lpfc_issue_els_prli() dereference pointer ndlp. Include these

[PATCH] scsi: lpfc: prevent potential null pointer dereference

2017-05-23 Thread Gustavo A. R. Silva
Null check at line 966: if (ndlp) {, implies that ndlp might be NULL. Functions lpfc_nlp_set_state() and lpfc_issue_els_prli() dereference pointer ndlp. Include these function calls inside the IF block that tests pointer ndlp. Addresses-Coverity-ID: 1401856 Signed-off-by: Gustavo A. R. Silva