Attachment 8709 Details for Bug 9758 – v4-0-test patch

[patch] v4-0-test patch

look (text/plain), 1.90 KB, created by Andreas Schneider on 2013-04-02 16:16:17 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Andreas Schneider

Created: 2013-04-02 16:16:17 UTC

Size: 1.90 KB

patch

obsolete

>From 06f649ac60074cdf55272b3f7c401f16bc48a12e Mon Sep 17 00:00:00 2001
>From: Andreas Schneider <asn@samba.org>
>Date: Tue, 2 Apr 2013 13:09:13 +0200
>Subject: [PATCH 1/2] epm: Increase debug level for already registered
> endpoints.
>
>Reviewed-by: Alexander Bokovoy <ab@samba.org>
>---
> source3/rpc_server/epmapper/srv_epmapper.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/source3/rpc_server/epmapper/srv_epmapper.c b/source3/rpc_server/epmapper/srv_epmapper.c
>index ac215cb..d05d552 100644
>--- a/source3/rpc_server/epmapper/srv_epmapper.c
>+++ b/source3/rpc_server/epmapper/srv_epmapper.c
>@@ -373,7 +373,7 @@ error_status_t _epm_Insert(struct pipes_struct *p,
> 		 * endpoint.
> 		 */
> 		if (find_interface(ep, iface) != NULL) {
>-			DEBUG(0, ("dcesrv_interface_register: interface '%s' "
>+			DEBUG(8, ("dcesrv_interface_register: interface '%s' "
> 				  "already registered on endpoint\n",
> 				  iface->name));
> 			/* FIXME wrong error code? */
>-- 
>1.8.2
>
>
>From 4ea081db12e92b1b47611a02e5bd3a99bb6a8d58 Mon Sep 17 00:00:00 2001
>From: Andreas Schneider <asn@samba.org>
>Date: Tue, 2 Apr 2013 13:08:19 +0200
>Subject: [PATCH 2/2] BUG 9758: Don't leak the epm_Map policy handle.
>
>Reviewed-by: Alexander Bokovoy <ab@samba.org>
>
>Autobuild-User(master): Alexander Bokovoy <ab@samba.org>
>Autobuild-Date(master): Tue Apr  2 17:16:56 CEST 2013 on sn-devel-104
>---
> source3/rpc_server/rpc_ep_register.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
>diff --git a/source3/rpc_server/rpc_ep_register.c b/source3/rpc_server/rpc_ep_register.c
>index e0fe962..96a3705 100644
>--- a/source3/rpc_server/rpc_ep_register.c
>+++ b/source3/rpc_server/rpc_ep_register.c
>@@ -249,6 +249,10 @@ static void rpc_ep_monitor_loop(struct tevent_req *subreq)
> 		ok = false;
> 	}
> 
>+	dcerpc_epm_LookupHandleFree(state->h,
>+				    tmp_ctx,
>+				    &entry_handle,
>+				    &result);
> 	talloc_free(tmp_ctx);
> 
> 	subreq = tevent_wakeup_send(state->mem_ctx,
>-- 
>1.8.2
>

Flags:

ab: review+

Actions: View

Attachments on bug 9758: 8709