Signed-off-by: Joe Carnuccio <joe.carnuccio@qlogic.com>
Signed-off-by: Saurav Kashyap <saurav.kashyap@qlogic.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
        fc_port_t       *fcport;
 
        /* Check for matching device in remote port list. */
-       fcport = NULL;
        list_for_each_entry(fcport, &vha->vp_fcports, list) {
                if (fcport->tgt_id == tgt_id) {
                        ql_dbg(ql_dbg_async, vha, 0x5072,
                            "Matching fcport(%p) found with TGT-ID: 0x%x "
                            "and Remote TGT_ID: 0x%x\n",
                            fcport, fcport->tgt_id, tgt_id);
-                       break;
+                       return fcport;
                }
        }
-       return fcport;
+       return NULL;
 }
 
 static void