diff options
author | Günther Deschner <gd@samba.org> | 2011-07-05 00:55:35 +0200 |
---|---|---|
committer | Günther Deschner <gd@samba.org> | 2011-07-07 18:06:02 +0200 |
commit | 72b1f8be5619ed778c4aa0b967f6a4f34d7e9de8 (patch) | |
tree | 9b82a9252c69476d80b1bfe30a6a2747aa7bb473 /source3/printing/nt_printing_ads.c | |
parent | 0a1ec73b965f66977a90fb7febb3b56b52ebab20 (diff) | |
download | samba-72b1f8be5619ed778c4aa0b967f6a4f34d7e9de8.tar.gz samba-72b1f8be5619ed778c4aa0b967f6a4f34d7e9de8.tar.bz2 samba-72b1f8be5619ed778c4aa0b967f6a4f34d7e9de8.zip |
s3-printing: safe a ton of roundtrips by reusing existing winreg binding_handles.
Guenther
Pair-Programmed-With: David Disseldorp <ddiss@suse.de>
Diffstat (limited to 'source3/printing/nt_printing_ads.c')
-rw-r--r-- | source3/printing/nt_printing_ads.c | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/source3/printing/nt_printing_ads.c b/source3/printing/nt_printing_ads.c index c5e3f99fb6..beb5240423 100644 --- a/source3/printing/nt_printing_ads.c +++ b/source3/printing/nt_printing_ads.c @@ -399,8 +399,17 @@ bool is_printer_published(TALLOC_CTX *mem_ctx, uint32_t data_size; WERROR result; NTSTATUS status; + struct dcerpc_binding_handle *b; - result = winreg_get_printer_internal(mem_ctx, session_info, msg_ctx, + result = winreg_printer_binding_handle(mem_ctx, + session_info, + msg_ctx, + &b); + if (!W_ERROR_IS_OK(result)) { + return false; + } + + result = winreg_get_printer(mem_ctx, b, printer, &pinfo2); if (!W_ERROR_IS_OK(result)) { return false; @@ -417,7 +426,7 @@ bool is_printer_published(TALLOC_CTX *mem_ctx, /* fetching printer guids really ought to be a separate function. */ - result = winreg_get_printer_dataex_internal(mem_ctx, session_info, msg_ctx, + result = winreg_get_printer_dataex(mem_ctx, b, printer, SPOOL_DSSPOOLER_KEY, "objectGUID", &type, &data, &data_size); |