Attachment 106478 Details for Bug 138788 – instead of stat to solve blocking on NFS for AS3 SysVinit-2.85-4

[patch] instead of stat to solve blocking on NFS for AS3 SysVinit-2.85-4

nfs_block_AS3.patch (text/plain), 1.64 KB, created by Constantine Gavrilov on 2004-11-11 07:02:21 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Constantine Gavrilov

Created: 2004-11-11 07:02:21 UTC

Size: 1.64 KB

patch

obsolete

>--- src/killall5.c	Thu Nov 11 02:04:02 2004
>+++ src/killall5.c	Thu Nov 11 02:01:55 2004
>@@ -50,8 +50,6 @@
>   char *fullname;	/* Name as found out from argv[0] */
>   char *basename;	/* Only the part after the last / */
>   char *statname;	/* the statname without braces    */
>-  ino_t ino;		/* Inode number			  */
>-  dev_t dev;		/* Device it is on		  */
>   pid_t pid;		/* Process ID.			  */
>   int sid;		/* Session ID.			  */
>   struct _proc_ *next;	/* Pointer to next struct. 	  */
>@@ -155,7 +153,6 @@
> 	FILE *fp;
> 	int pid, f;
> 	PROC *p, *n;
>-	struct stat st;
> 	int c;
> 
> 	/* Open the /proc directory. */
>@@ -225,6 +222,7 @@
> 		if (p->sid < 0) {
> 			p->sid = 0;
> 			nsyslog(LOG_ERR, "can't read sid for pid %d\n", pid);
>+			free(p->statname);
> 			free(p);
> 			continue;
> 		}
>@@ -249,17 +247,20 @@
> 				p->basename = p->fullname;
> 		} else {
> 			/* Process disappeared.. */
>+			free(p->statname);
> 			free(p);
> 			continue;
> 		}
> 
> 		/* Try to stat the executable. */
> 		snprintf(path, sizeof(path), "/proc/%s/exe", d->d_name);
>-		if (stat(path, &st) == 0) {
>-			p->dev = st.st_dev;
>-			p->ino = st.st_ino;
>-			p->pathname = (char *)xmalloc(PATH_MAX);
>-			readlink(path, p->pathname, PATH_MAX);
>+
>+		p->pathname = (char *)xmalloc(PATH_MAX);
>+		if(readlink(path, p->pathname, PATH_MAX) < 0) {
>+			free(p->statname);
>+			free(p->fullname);
>+			free(p);
>+			continue;
> 		}
> 
> 		/* Link it into the list. */
>@@ -345,7 +346,7 @@
> 	/* First try to find a match based on dev/ino pair. */
> 	if (dostat) {
> 		for (p = plist; p; p = p->next) {
>-			if (p->dev == st.st_dev && p->ino == st.st_ino) {
>+			if (strcmp(prog, p->pathname) == 0) {
> 				add_pid_to_q(q, p);
> 				foundone++;
> 			}

Actions: View | Diff

Attachments on bug 138788: 106475 | 106478 | 108533 | 108534 | 108535 | 113650