diff options
author | Jaswinder Singh Rajput <jaswinder@kernel.org> | 2009-06-20 03:59:21 -0400 |
---|---|---|
committer | James Bottomley <James.Bottomley@HansenPartnership.com> | 2009-06-21 12:58:11 -0400 |
commit | 24add1c4326ce5ca22e7af8b84bb113cd48efac9 (patch) | |
tree | 11869f940027d928d514865667e90f2a9e956f65 /drivers/scsi/scsi_transport_iscsi.c | |
parent | 47e7e89ed029780adf2cc0cf506fcd4c2d5ca1e2 (diff) |
scsi_transport_iscsi: return -EOVERFLOW for Too many iscsi targets
setting err as -EOVERFLOW for Too many iscsi targets.
Also fixes a spurious compiler warning for gcc 4.3.3 and gcc 4.4 :
CC drivers/scsi/scsi_transport_iscsi.o
drivers/scsi/scsi_transport_iscsi.c: In function ‘iscsi_add_session’:
drivers/scsi/scsi_transport_iscsi.c:678: warning: ‘err’ may be used uninitialized in this function
Signed-off-by: Jaswinder Singh Rajput <jaswinderrajput@gmail.com>
Acked-by: Mike Christie <michaelc@cs.wisc.edu>
Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com>
Diffstat (limited to 'drivers/scsi/scsi_transport_iscsi.c')
-rw-r--r-- | drivers/scsi/scsi_transport_iscsi.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/scsi/scsi_transport_iscsi.c b/drivers/scsi/scsi_transport_iscsi.c index f3e664628d7a..783e33c65eb7 100644 --- a/drivers/scsi/scsi_transport_iscsi.c +++ b/drivers/scsi/scsi_transport_iscsi.c | |||
@@ -692,6 +692,7 @@ int iscsi_add_session(struct iscsi_cls_session *session, unsigned int target_id) | |||
692 | "Too many iscsi targets. Max " | 692 | "Too many iscsi targets. Max " |
693 | "number of targets is %d.\n", | 693 | "number of targets is %d.\n", |
694 | ISCSI_MAX_TARGET - 1); | 694 | ISCSI_MAX_TARGET - 1); |
695 | err = -EOVERFLOW; | ||
695 | goto release_host; | 696 | goto release_host; |
696 | } | 697 | } |
697 | } | 698 | } |