diff options
author | Volker Lendecke <vl@samba.org> | 2010-10-02 11:50:26 +0200 |
---|---|---|
committer | Volker Lendecke <vl@samba.org> | 2010-10-02 12:02:02 +0200 |
commit | 13756d0e803c228f7ff75ba996acd6afd1df0dcb (patch) | |
tree | 451805eabad52144e81a34bb6ea803ed23973044 /source3/lib/netapi/netlogon.c | |
parent | fc6e910e69f49cd482966eb9af3f27c0b65bcf4b (diff) | |
download | samba-13756d0e803c228f7ff75ba996acd6afd1df0dcb.tar.gz samba-13756d0e803c228f7ff75ba996acd6afd1df0dcb.tar.bz2 samba-13756d0e803c228f7ff75ba996acd6afd1df0dcb.zip |
s3: Attempt to fix bug 7665
Quite a few of our internal routines put stuff on talloc_tos() these days.
In top-level netapi routines, properly allocate a stackframe and clean it
again. Also, don't leak memory in the rpccli_ callers onto the libnetapi
context.
Diffstat (limited to 'source3/lib/netapi/netlogon.c')
-rw-r--r-- | source3/lib/netapi/netlogon.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/source3/lib/netapi/netlogon.c b/source3/lib/netapi/netlogon.c index 932ab8bb1d..8c30d6caab 100644 --- a/source3/lib/netapi/netlogon.c +++ b/source3/lib/netapi/netlogon.c @@ -139,7 +139,7 @@ WERROR I_NetLogonControl_r(struct libnetapi_ctx *ctx, goto done; } - status = rpccli_netr_LogonControl(pipe_cli, ctx, + status = rpccli_netr_LogonControl(pipe_cli, talloc_tos(), r->in.server_name, r->in.function_code, r->in.query_level, @@ -197,7 +197,7 @@ WERROR I_NetLogonControl2_r(struct libnetapi_ctx *ctx, case NETLOGON_CONTROL_TC_VERIFY: case NETLOGON_CONTROL_SET_DBFLAG: case NETLOGON_CONTROL_FORCE_DNS_REG: - status = rpccli_netr_LogonControl2Ex(pipe_cli, ctx, + status = rpccli_netr_LogonControl2Ex(pipe_cli, talloc_tos(), r->in.server_name, r->in.function_code, r->in.query_level, @@ -206,7 +206,7 @@ WERROR I_NetLogonControl2_r(struct libnetapi_ctx *ctx, &werr); break; default: - status = rpccli_netr_LogonControl2(pipe_cli, ctx, + status = rpccli_netr_LogonControl2(pipe_cli, talloc_tos(), r->in.server_name, r->in.function_code, r->in.query_level, |