From 21b03f4e6223f9823cc667b352ba325d721e30d7 Mon Sep 17 00:00:00 2001 From: David Disseldorp Date: Mon, 26 May 2014 15:20:13 +0200 Subject: s3/rpc_server/spoolss: remove SETUP_SPOOLSS_NOTIFY_DATA_SECDESC The macro is only used once, and is broken in two ways: - it relies on an externally defined mem_ctx - _data->data.sd.sd_size is set zero twice for a NULL sd Signed-off-by: David Disseldorp Reviewed-by: Jeremy Allison Reviewed-by: Volker Lendecke --- source3/rpc_server/spoolss/srv_spoolss_nt.c | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) (limited to 'source3') diff --git a/source3/rpc_server/spoolss/srv_spoolss_nt.c b/source3/rpc_server/spoolss/srv_spoolss_nt.c index 1305b1006e6..588aae4eb90 100644 --- a/source3/rpc_server/spoolss/srv_spoolss_nt.c +++ b/source3/rpc_server/spoolss/srv_spoolss_nt.c @@ -846,14 +846,6 @@ static bool is_monitoring_event(struct printer_handle *p, uint16_t notify_type, #define SETUP_SPOOLSS_NOTIFY_DATA_DEVMODE(_data, _devmode) \ _data->data.devmode.devmode = _devmode; -#define SETUP_SPOOLSS_NOTIFY_DATA_SECDESC(_data, _sd) \ - _data->data.sd.sd = dup_sec_desc(mem_ctx, _sd); \ - if (!_data->data.sd.sd) { \ - _data->data.sd.sd_size = 0; \ - } \ - _data->data.sd.sd_size = \ - ndr_size_security_descriptor(_data->data.sd.sd, 0); - static void init_systemtime_buffer(TALLOC_CTX *mem_ctx, struct tm *t, const char **pp, @@ -2943,7 +2935,9 @@ static void spoolss_notify_security_desc(struct messaging_context *msg_ctx, struct spoolss_PrinterInfo2 *pinfo2, TALLOC_CTX *mem_ctx) { - SETUP_SPOOLSS_NOTIFY_DATA_SECDESC(data, pinfo2->secdesc); + data->data.sd.sd = dup_sec_desc(mem_ctx, pinfo2->secdesc); + data->data.sd.sd_size = ndr_size_security_descriptor(data->data.sd.sd, + 0); } /******************************************************************* -- cgit