cifs: refcount only the selected iface during interface update

Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=2151418
commit 7246210

When the server interface for a channel is not active anymore,
we have the logic to select an alternative interface. However
this was not breaking out of the loop as soon as a new alternative
was found. As a result, some interfaces may get refcounted unintentionally.

There was also a bug in checking if we found an alternate iface.
Fixed that too.

    Fixes: b54034a ("cifs: during reconnect, update interface if necessary")
    Cc: stable@vger.kernel.org # 5.19+
    Signed-off-by: Shyam Prasad N <sprasad@microsoft.com>
    Reviewed-by: Paulo Alcantara (SUSE) <pc@cjr.nz>
    Signed-off-by: Steve French <stfrench@microsoft.com>
    (cherry picked from commit 7246210)
Signed-off-by: Ronnie Sahlberg <lsahlber@redhat.com>

Author: Ronnie Sahlberg

fs/cifs/sess.c

@@ -292,9 +292,10 @@ cifs_chan_update_iface(struct cifs_ses *ses, struct TCP_Server_Info *server)
 			continue;
 		}
 		kref_get(&iface->refcount);
+		break;
 	}
 
-	if (!list_entry_is_head(iface, &ses->iface_list, iface_head)) {
+	if (list_entry_is_head(iface, &ses->iface_list, iface_head)) {
 		rc = 1;
 		iface = NULL;
 		cifs_dbg(FYI, "unable to find a suitable iface\n");