qla4xxx: Fix smatch warning in func qla4xxx_get_ep_param
authorAdheer Chandravanshi <adheer.chandravanshi@qlogic.com>
Thu, 10 Apr 2014 01:12:27 +0000 (21:12 -0400)
committerChristoph Hellwig <hch@lst.de>
Mon, 19 May 2014 17:12:20 +0000 (19:12 +0200)
Fix following smatch warning:-
 drivers/scsi/qla4xxx/ql4_os.c:1752 qla4xxx_get_ep_param() warn: variable dereferenced before check 'qla_ep' (see line 1745)

Signed-off-by: Vikas Chaudhary <vikas.chaudhary@qlogic.com>
Reviewed-by: Mike Christie <michaelc@cs.wisc.edu>
Signed-off-by: Christoph Hellwig <hch@lst.de>
drivers/scsi/qla4xxx/ql4_os.c

index 9be829bb90dba795d12ff3be871bd833aae62412..592f29375a89e460987d4028c19e6e95661d864f 100644 (file)
@@ -1742,6 +1742,9 @@ static int qla4xxx_get_ep_param(struct iscsi_endpoint *ep,
        struct sockaddr *dst_addr;
        struct scsi_qla_host *ha;
 
+       if (!qla_ep)
+               return -ENOTCONN;
+
        ha = to_qla_host(qla_ep->host);
        DEBUG2(ql4_printk(KERN_INFO, ha, "%s: host: %ld\n", __func__,
                          ha->host_no));
@@ -1749,9 +1752,6 @@ static int qla4xxx_get_ep_param(struct iscsi_endpoint *ep,
        switch (param) {
        case ISCSI_PARAM_CONN_PORT:
        case ISCSI_PARAM_CONN_ADDRESS:
-               if (!qla_ep)
-                       return -ENOTCONN;
-
                dst_addr = (struct sockaddr *)&qla_ep->dst_addr;
                if (!dst_addr)
                        return -ENOTCONN;