diff options
| author | Günther Deschner <gd@samba.org> | 2009-03-20 11:21:36 +0100 |
|---|---|---|
| committer | Günther Deschner <gd@samba.org> | 2009-03-20 11:21:36 +0100 |
| commit | 3a6133d70202ef80ead4203740c000be5e93b288 (patch) | |
| tree | 03831cd712f2c0b467b25678b3710375f2255bd2 /source3 | |
| parent | f2243e8197aa064adea3da2a4d1c08250668943f (diff) | |
| download | samba-3a6133d70202ef80ead4203740c000be5e93b288.tar.gz samba-3a6133d70202ef80ead4203740c000be5e93b288.tar.bz2 samba-3a6133d70202ef80ead4203740c000be5e93b288.zip | |
s3-netapi: Fix Coverity #774 (REVERSE_INULL).
Guenther
Diffstat (limited to 'source3')
| -rw-r--r-- | source3/lib/netapi/user.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/source3/lib/netapi/user.c b/source3/lib/netapi/user.c index b1bd27af2a..e760a8b1de 100644 --- a/source3/lib/netapi/user.c +++ b/source3/lib/netapi/user.c @@ -2806,6 +2806,7 @@ WERROR NetUserGetGroups_r(struct libnetapi_ctx *ctx, *r->out.buffer = NULL; *r->out.entries_read = 0; + *r->out.total_entries = 0; switch (r->in.level) { case 0: @@ -2899,12 +2900,8 @@ WERROR NetUserGetGroups_r(struct libnetapi_ctx *ctx, } } - if (r->out.entries_read) { - *r->out.entries_read = entries_read; - } - if (r->out.total_entries) { - *r->out.total_entries = entries_read; - } + *r->out.entries_read = entries_read; + *r->out.total_entries = entries_read; done: if (ctx->disable_policy_handle_cache) { |