Re: [PATCH] scsi: lpfc: Avoid NULL pointer dereference in lpfc_els_abort()

2017-05-31 Thread Martin K. Petersen
Guilherme, > We might have a NULL pring in lpfc_els_abort(), for example on error > recovery path, since queues are destroyed during error recovery > mechanism. > > In this case, we should just drop the abort since the queues will be > recreated anyway. This patch just verifies for NULL pointer

Re: [PATCH] scsi: lpfc: Avoid NULL pointer dereference in lpfc_els_abort()

2017-05-29 Thread James Smart
Change looks good Signed-off-by: James Smart -- james On 5/24/2017 2:48 PM, Guilherme G. Piccoli wrote: We might have a NULL pring in lpfc_els_abort(), for example on error recovery path, since queues are destroyed during error recovery mechanism. In this case,

Re: [PATCH] scsi: lpfc: Avoid NULL pointer dereference in lpfc_els_abort()

2017-05-29 Thread Johannes Thumshirn
On 05/24/2017 11:48 PM, Guilherme G. Piccoli wrote: > We might have a NULL pring in lpfc_els_abort(), for example on > error recovery path, since queues are destroyed during error > recovery mechanism. > > In this case, we should just drop the abort since the queues will > be recreated anyway.

[PATCH] scsi: lpfc: Avoid NULL pointer dereference in lpfc_els_abort()

2017-05-24 Thread Guilherme G. Piccoli
We might have a NULL pring in lpfc_els_abort(), for example on error recovery path, since queues are destroyed during error recovery mechanism. In this case, we should just drop the abort since the queues will be recreated anyway. This patch just verifies for NULL pointer and stop the abortion of