s3: libsmb: Cleanup - change to early continue in internal_resolve_name() for KDC...

No logic change.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Isaac Boukris <iboukris@samba.org>

diff --git a/source3/libsmb/namequery.c b/source3/libsmb/namequery.c
index c964d90fc4ba091a64d06504963081335126e4c7..ea6aec5290afabbbf131ad6e2cda1d021a687b5c 100644 (file)
--- a/source3/libsmb/namequery.c
+++ b/source3/libsmb/namequery.c
@@ -2692,12 +2692,13 @@ NTSTATUS internal_resolve_name(const char *name,
                         * This will result in a SRV record lookup */
                        status = resolve_ads(name, KDC_NAME_TYPE, sitename,
                                             return_iplist, return_count);
-                       if (NT_STATUS_IS_OK(status)) {
-                               /* Ensure we don't namecache
-                                * this with the KDC port. */
-                               name_type = KDC_NAME_TYPE;
-                               goto done;
+                       if (!NT_STATUS_IS_OK(status)) {
+                               continue;
                        }
+                       /* Ensure we don't namecache
+                        * this with the KDC port. */
+                       name_type = KDC_NAME_TYPE;
+                       goto done;
                } else if(strequal( tok, "ads")) {
                        /* deal with 0x1c and 0x1b names here.
                         * This will result in a SRV record lookup */