On Saturday, August 25, 2018 at 10:16:52 AM UTC-7, Christophe JAILLET wrote:
>
> All error handling paths in this function go through 
> 'iscsi_host_dec_session_cnt()' except this one. 
> Fix it and properly decrement the number of active sessions in such a 
> case. 
>
> Signed-off-by: Christophe JAILLET <christophe.jail...@wanadoo.fr> 
> --- 
>  drivers/scsi/libiscsi.c | 2 +- 
>  1 file changed, 1 insertion(+), 1 deletion(-) 
>
> diff --git a/drivers/scsi/libiscsi.c b/drivers/scsi/libiscsi.c 
> index 93c66ebad907..1fa7d6b77761 100644 
> --- a/drivers/scsi/libiscsi.c 
> +++ b/drivers/scsi/libiscsi.c 
> @@ -2803,7 +2803,7 @@ iscsi_session_setup(struct iscsi_transport *iscsit, 
> struct Scsi_Host *shost, 
>                         "must be a power of 2.\n", total_cmds); 
>                  total_cmds = rounddown_pow_of_two(total_cmds); 
>                  if (total_cmds < ISCSI_TOTAL_CMDS_MIN) 
> -                        return NULL; 
> +                        goto dec_session_count; 
>                  printk(KERN_INFO "iscsi: Rounding can_queue to %d.\n", 
>                         total_cmds); 
>          } 
> -- 
> 2.17.1 
>
>
Signed-off-by: Lee Duncan <ldun...@suse.com> 

-- 
You received this message because you are subscribed to the Google Groups 
"open-iscsi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to open-iscsi+unsubscr...@googlegroups.com.
To post to this group, send email to open-iscsi@googlegroups.com.
Visit this group at https://groups.google.com/group/open-iscsi.
For more options, visit https://groups.google.com/d/optout.

Reply via email to