Attachment 12301 Details for Bug 12016 – Patch for 4.3 backported from master

[patch] Patch for 4.3 backported from master

v43-bug12016.patch (text/plain), 2.62 KB, created by Ralph Böhme on 2016-08-02 09:31:38 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Ralph Böhme

Created: 2016-08-02 09:31:38 UTC

Size: 2.62 KB

patch

obsolete

>From 8ddbed34e4c077c65ab1388f1bf38e5b91e107bc Mon Sep 17 00:00:00 2001
>From: Ralph Boehme <slow@samba.org>
>Date: Tue, 5 Jul 2016 15:38:31 +0200
>Subject: [PATCH 1/2] s3-rpc_server/mdssd: use smbd_reinit_after_fork()
>
>Using smbd_reinit_after_fork() rather then reinit_after_fork() ensures
>am_parent is reset to NULL. Otherwise, when exiting for some reason, the
>inherited atexit handler killkids() calls kill(0,SIGTERM) terminating
>our whole process group including the main smbd.
>
>Bug: https://bugzilla.samba.org/show_bug.cgi?id=12016
>
>Signed-off-by: Ralph Boehme <slow@samba.org>
>Reviewed-by: Volker Lendecke <vl@samba.org>
>
>Autobuild-User(master): Volker Lendecke <vl@samba.org>
>Autobuild-Date(master): Mon Jul 11 02:02:33 CEST 2016 on sn-devel-144
>
>(backported from commit 6efd0af5e8d8ee81b5f53afb126d87950088b7cd)
>---
> source3/rpc_server/mdssd.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
>diff --git a/source3/rpc_server/mdssd.c b/source3/rpc_server/mdssd.c
>index fac386b..d826447 100644
>--- a/source3/rpc_server/mdssd.c
>+++ b/source3/rpc_server/mdssd.c
>@@ -671,9 +671,7 @@ void start_mdssd(struct tevent_context *ev_ctx,
> 		return;
> 	}
> 
>-	status = reinit_after_fork(msg_ctx,
>-				   ev_ctx,
>-				   true);
>+	status = smbd_reinit_after_fork(msg_ctx, ev_ctx, true);
> 	if (!NT_STATUS_IS_OK(status)) {
> 		DEBUG(0,("reinit_after_fork() failed\n"));
> 		smb_panic("reinit_after_fork() failed");
>-- 
>2.7.4
>
>
>From bffb58e3b74452def8e606cb9c07725f6de5578b Mon Sep 17 00:00:00 2001
>From: Ralph Boehme <slow@samba.org>
>Date: Tue, 5 Jul 2016 15:37:53 +0200
>Subject: [PATCH 2/2] smbd/notifyd: use smbd_reinit_after_fork()
>
>Using smbd_reinit_after_fork() rather then reinit_after_fork() ensures
>am_parent is reset to NULL. Otherwise, when exiting for some reason, the
>inherited atexit handler killkids() calls kill(0,SIGTERM) terminating
>our whole process group including the main smbd.
>
>Bug: https://bugzilla.samba.org/show_bug.cgi?id=12016
>
>Signed-off-by: Ralph Boehme <slow@samba.org>
>Reviewed-by: Volker Lendecke <vl@samba.org>
>(backported from commit bb90a8484a734b76ddd4d270f31fcdb4ae6cc48d)
>---
> source3/smbd/server.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/source3/smbd/server.c b/source3/smbd/server.c
>index 8c77500..bc4b37f 100644
>--- a/source3/smbd/server.c
>+++ b/source3/smbd/server.c
>@@ -389,7 +389,7 @@ static bool smbd_notifyd_init(struct messaging_context *msg, bool interactive)
> 		return true;
> 	}
> 
>-	status = reinit_after_fork(msg, ev, true);
>+	status = smbd_reinit_after_fork(msg, ev, true);
> 	if (!NT_STATUS_IS_OK(status)) {
> 		DEBUG(1, ("%s: reinit_after_fork failed: %s\n",
> 			  __func__, nt_errstr(status)));
>-- 
>2.7.4
>

Flags:

metze: review+
slow: review? (jra)

Actions: View

Attachments on bug 12016: 12300 | 12301