diff options
author | Andreas Schneider <asn@samba.org> | 2011-07-25 21:39:01 +0200 |
---|---|---|
committer | Andreas Schneider <asn@samba.org> | 2011-07-27 08:50:00 +0200 |
commit | 80c1a8e04e4d287bcd58091d65549e499c2fa11a (patch) | |
tree | 475c2a4071b97995aadac8d5d60f732bb0a3c7fc | |
parent | baccb06658fde3d49b58c6aeb17f0e9c320dbd01 (diff) | |
download | samba-80c1a8e04e4d287bcd58091d65549e499c2fa11a.tar.gz samba-80c1a8e04e4d287bcd58091d65549e499c2fa11a.tar.bz2 samba-80c1a8e04e4d287bcd58091d65549e499c2fa11a.zip |
s3-spoolss: Use tmp_ctx in winreg_printer_setform1_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 df419ec0e5..17c6c6ea70 100644 --- a/source3/rpc_server/spoolss/srv_spoolss_util.c +++ b/source3/rpc_server/spoolss/srv_spoolss_util.c @@ -671,13 +671,26 @@ WERROR winreg_printer_setform1_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; + } - return winreg_printer_setform1(mem_ctx, b, - form_name, - form); + result = winreg_printer_binding_handle(tmp_ctx, session_info, msg_ctx, &b); + if (!W_ERROR_IS_OK(result)) { + talloc_free(tmp_ctx); + return result; + } + + result = winreg_printer_setform1(mem_ctx, + b, + form_name, + form); + + talloc_free(tmp_ctx); + return result; } WERROR winreg_printer_deleteform1_internal(TALLOC_CTX *mem_ctx, |