Change looks good Signed-off-by: James Smart <james.smart@xxxxxxxxxxxx> -- 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, 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 the queue in case of a NULL pring. Also, this patch converts return type of lpfc_els_abort() from int to void, since it's not checked anywhere. Reported-by: Harsha Thyagaraja <hathyaga@xxxxxxxxxx> Reported-by: Naresh Bannoth <nbannoth@xxxxxxxxxx> Tested-by: Raphael Silva <raphasil@xxxxxxxxxxxxxxxxxx> Signed-off-by: Guilherme G. Piccoli <gpiccoli@xxxxxxxxxxxxxxxxxx>
