[PATCH] pam_ecryptfs: waitpid()

I'm experiencing some strange hangs in various Xwindows desktop managers
(Gnome, KDE, XFCE), related to the pam_ecryptfs module.  It seems to be
related to this wait() call.  All issues I'm experiencing are fixed by
this one-line patch, using waitpid() instead.

waitpid() seems to be used elsewhere in pam_ecryptfs, so I think this
should be preferred, for consistency.


-- 
:-Dustin

Dustin Kirkland
Ubuntu Server Developer
Canonical, LTD
[EMAIL PROTECTED]
GPG: 1024D/83A61194


diff --git a/src/pam_ecryptfs/pam_ecryptfs.c b/src/pam_ecryptfs/pam_ecryptfs.c
index a284989..40c4022 100644
--- a/src/pam_ecryptfs/pam_ecryptfs.c
+++ b/src/pam_ecryptfs/pam_ecryptfs.c
@@ -256,7 +256,7 @@ static int private_dir(pam_handle_t *pamh, int mount)
                }
                return 1;
        } else {
-               wait(&rc);
+               waitpid(pid, &rc, NULL);
                syslog(LOG_INFO, 
                       "Mount of private directory return code [%d]", rc);
                goto out;

diff --git a/src/pam_ecryptfs/pam_ecryptfs.c b/src/pam_ecryptfs/pam_ecryptfs.c
index a284989..40c4022 100644
--- a/src/pam_ecryptfs/pam_ecryptfs.c
+++ b/src/pam_ecryptfs/pam_ecryptfs.c
@@ -256,7 +256,7 @@ static int private_dir(pam_handle_t *pamh, int mount)
 		}
 		return 1;
 	} else {
-		wait(&rc);
+		waitpid(pid, &rc, NULL);
 		syslog(LOG_INFO, 
 		       "Mount of private directory return code [%d]", rc);
 		goto out;

Attachment: signature.asc
Description: This is a digitally signed message part

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
eCryptfs-devel mailing list
eCryptfs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ecryptfs-devel

Reply via email to