Attachment 158592 Details for Bug 245674 – patch -- have cifsd disallow all signals and use force_sig

[patch] patch -- have cifsd disallow all signals and use force_sig

0003-BZ-245674-have-cifsd-block-userspace-signals-and-us.patch (text/plain), 1.72 KB, created by Jeff Layton on 2007-07-05 14:24:32 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jeff Layton

Created: 2007-07-05 14:24:32 UTC

Size: 1.72 KB

patch

obsolete

>From e825eebfd3049ece92ec7ca6454f32a818a21189 Mon Sep 17 00:00:00 2001
>From: Jeff Layton <jlayton@redhat.com>
>Date: Thu, 28 Jun 2007 10:27:02 -0400
>Subject: [PATCH] BZ#245674: have cifsd block userspace signals and use force_sig internally
>
>---
> fs/cifs/connect.c |    7 +++----
> 1 files changed, 3 insertions(+), 4 deletions(-)
>
>diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c
>index 9009d59..427f81a 100644
>--- a/fs/cifs/connect.c
>+++ b/fs/cifs/connect.c
>@@ -369,7 +369,6 @@ cifs_demultiplex_thread(struct TCP_Server_Info *server)
> 	sprintf(current->comm,"cifsd");
> #else
> 	daemonize("cifsd");
>-	allow_signal(SIGKILL);
> #endif
> 	current->flags |= PF_MEMALLOC;
> 	server->tsk = current;	/* save process info to wake at shutdown */
>@@ -2054,7 +2053,7 @@ cifs_mount(struct super_block *sb, struct cifs_sb_info *cifs_sb,
> 			srvTcp->tcpStatus = CifsExiting;
> 			spin_unlock(&GlobalMid_Lock);
> 			if(srvTcp->tsk) {
>-				send_sig(SIGKILL,srvTcp->tsk,1);
>+				force_sig(SIGKILL,srvTcp->tsk);
> 				wait_for_completion(&cifsd_complete);
> 			}
> 		}
>@@ -2070,7 +2069,7 @@ cifs_mount(struct super_block *sb, struct cifs_sb_info *cifs_sb,
> 					/* if the socketUseCount is now zero */
> 					if((temp_rc == -ESHUTDOWN) &&
> 					   (pSesInfo->server->tsk)) {
>-						send_sig(SIGKILL,pSesInfo->server->tsk,1);
>+						force_sig(SIGKILL,pSesInfo->server->tsk);
> 						wait_for_completion(&cifsd_complete);
> 					}
> 				} else
>@@ -3391,7 +3390,7 @@ cifs_umount(struct super_block *sb, struct cifs_sb_info *cifs_sb)
> 			} else if (rc == -ESHUTDOWN) {
> 				cFYI(1,("Waking up socket by sending it signal"));
> 				if(cifsd_task) {
>-					send_sig(SIGKILL,cifsd_task,1);
>+					force_sig(SIGKILL,cifsd_task);
> 					wait_for_completion(&cifsd_complete);
> 				}
> 				rc = 0;
>-- 
>1.5.2.1
>

Actions: View | Diff

Attachments on bug 245674: 158592