r8938: fix querydispinfo search semantics.
authorGünther Deschner <gd@samba.org>
Tue, 2 Aug 2005 19:48:42 +0000 (19:48 +0000)
committerGerald (Jerry) Carter <jerry@samba.org>
Wed, 10 Oct 2007 16:00:21 +0000 (11:00 -0500)
Just in case some uses rpcclient for code-testing...

Guenther
(This used to be commit 624b1c220801834820a605274b1abce883597403)

source3/rpcclient/cmd_samr.c

index bd150f2a353a7a8beb4000c83796be21f3c001f7..acb392710507b5c26293e8f03a132a9a8d3b2b41 100644 (file)
@@ -1095,7 +1095,7 @@ static NTSTATUS cmd_samr_query_dispinfo(struct cli_state *cli,
        }
 
 
-       while(1) {
+       do {
 
                if (!got_params)
                        get_query_dispinfo_params(
@@ -1108,7 +1108,7 @@ static NTSTATUS cmd_samr_query_dispinfo(struct cli_state *cli,
 
                loop_count++;
 
-               if (!NT_STATUS_IS_OK(result) && !NT_STATUS_EQUAL(result, STATUS_MORE_ENTRIES)) 
+               if (NT_STATUS_IS_ERR(result))
                        break;
 
                if (num_entries == 0) 
@@ -1133,7 +1133,7 @@ static NTSTATUS cmd_samr_query_dispinfo(struct cli_state *cli,
                                break;
                        }
                }
-       }
+       } while ( NT_STATUS_EQUAL(result, STATUS_MORE_ENTRIES));
 
  done:
        return result;