diff options
author | Andreas Schneider <asn@samba.org> | 2011-07-25 21:35:24 +0200 |
---|---|---|
committer | Andreas Schneider <asn@samba.org> | 2011-07-27 08:50:00 +0200 |
commit | 81eae6ebd4746e4c14fe86e7a77ce78a172d9234 (patch) | |
tree | a6e766f69b8a27cf3d6f020dd9f25f706f51e631 | |
parent | 41c2e711bd04892c68a94d62c98e51bf95af33e8 (diff) | |
download | samba-81eae6ebd4746e4c14fe86e7a77ce78a172d9234.tar.gz samba-81eae6ebd4746e4c14fe86e7a77ce78a172d9234.tar.bz2 samba-81eae6ebd4746e4c14fe86e7a77ce78a172d9234.zip |
s3-spoolss: Use tmp_ctx in winreg_set_printer_secdesc_internal.
-rw-r--r-- | source3/rpc_server/spoolss/srv_spoolss_util.c | 23 |
1 files changed, 18 insertions, 5 deletions
diff --git a/source3/rpc_server/spoolss/srv_spoolss_util.c b/source3/rpc_server/spoolss/srv_spoolss_util.c index 7b14a1ca45..445e2cfcaa 100644 --- a/source3/rpc_server/spoolss/srv_spoolss_util.c +++ b/source3/rpc_server/spoolss/srv_spoolss_util.c @@ -553,13 +553,26 @@ WERROR winreg_set_printer_secdesc_internal(TALLOC_CTX *mem_ctx, { WERROR result; struct dcerpc_binding_handle *b; + TALLOC_CTX *tmp_ctx; - result = winreg_printer_binding_handle(mem_ctx, session_info, msg_ctx, &b); - W_ERROR_NOT_OK_RETURN(result); + tmp_ctx = talloc_stackframe(); + if (tmp_ctx == NULL) { + return WERR_NOMEM; + } + + result = winreg_printer_binding_handle(tmp_ctx, session_info, msg_ctx, &b); + if (!W_ERROR_IS_OK(result)) { + talloc_free(tmp_ctx); + return result; + } - return winreg_set_printer_secdesc(mem_ctx, b, - sharename, - secdesc); + result = winreg_set_printer_secdesc(mem_ctx, + b, + sharename, + secdesc); + + talloc_free(tmp_ctx); + return result; } WERROR winreg_printer_enumforms1_internal(TALLOC_CTX *mem_ctx, |