s3: libsmb: Cleanup - change to early continue in internal_resolve_name() for resolve...
authorJeremy Allison <jra@samba.org>
Tue, 21 Jul 2020 00:00:35 +0000 (17:00 -0700)
committerIsaac Boukris <iboukris@sn-devel-184>
Tue, 4 Aug 2020 08:51:41 +0000 (08:51 +0000)
No logic change.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Isaac Boukris <iboukris@samba.org>
source3/libsmb/namequery.c

index 37f9f0fa3ddcaa5c23c9d4012ff5553cac92cc26..e72504cfcafc9ab8d39f16ddfbf979c2df9f7b7b 100644 (file)
@@ -2733,15 +2733,16 @@ NTSTATUS internal_resolve_name(const char *name,
                                              talloc_tos(),
                                              &ss_list,
                                              return_count);
-                       if (NT_STATUS_IS_OK(status)) {
-                               if (!convert_ss2service(return_iplist,
-                                                       ss_list,
-                                                       return_count)) {
-                                       status = NT_STATUS_NO_MEMORY;
-                                       goto fail;
-                               }
-                               goto done;
+                       if (!NT_STATUS_IS_OK(status)) {
+                               continue;
+                       }
+                       if (!convert_ss2service(return_iplist,
+                                               ss_list,
+                                               return_count)) {
+                               status = NT_STATUS_NO_MEMORY;
+                               goto fail;
                        }
+                       goto done;
                } else if (strequal(tok, "bcast")) {
                        struct sockaddr_storage *ss_list = NULL;
                        status = name_resolve_bcast(