diff options
| author | Volker Lendecke <vl@samba.org> | 2010-08-02 19:22:22 +0200 |
|---|---|---|
| committer | Volker Lendecke <vl@samba.org> | 2010-08-04 20:32:50 +0200 |
| commit | 867626abcad88b84684e9d328abf51d4f410a1cb (patch) | |
| tree | a15ee1aefc33c3c1278e8ab955dcb8db38ab5463 /source3/libgpo | |
| parent | 2ff73f0df3257c27cb3cdae779e679de3170be17 (diff) | |
| download | samba-867626abcad88b84684e9d328abf51d4f410a1cb.tar.gz samba-867626abcad88b84684e9d328abf51d4f410a1cb.tar.bz2 samba-867626abcad88b84684e9d328abf51d4f410a1cb.zip | |
s3: Convert cli_list() to return NTSTATUS
If needed, the callback functions can count themselves
Diffstat (limited to 'source3/libgpo')
| -rw-r--r-- | source3/libgpo/gpo_filesync.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/source3/libgpo/gpo_filesync.c b/source3/libgpo/gpo_filesync.c index 461ebb742f..d85df1f7e1 100644 --- a/source3/libgpo/gpo_filesync.c +++ b/source3/libgpo/gpo_filesync.c @@ -110,15 +110,15 @@ static NTSTATUS gpo_copy_dir(const char *unix_path) static bool gpo_sync_files(struct sync_context *ctx) { + NTSTATUS status; + DEBUG(3,("calling cli_list with mask: %s\n", ctx->mask)); - if (cli_list(ctx->cli, - ctx->mask, - ctx->attribute, - gpo_sync_func, - ctx) == -1) { - DEBUG(1,("listing [%s] failed with error: %s\n", - ctx->mask, cli_errstr(ctx->cli))); + status = cli_list(ctx->cli, ctx->mask, ctx->attribute, gpo_sync_func, + ctx); + if (!NT_STATUS_IS_OK(status)) { + DEBUG(1, ("listing [%s] failed with error: %s\n", + ctx->mask, nt_errstr(status))); return false; } |