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. 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> > --- Looks good, Reviewed-by: Johannes Thumshirn <jthumshirn@xxxxxxx> -- Johannes Thumshirn Storage jthumshirn@xxxxxxx +49 911 74053 689 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850
