From 32a189220c1e7422dff591c8c8d5e8db04fcca83 Mon Sep 17 00:00:00 2001
From: Evan Prodromou <evan@prodromou.name>
Date: Thu, 4 Sep 2008 15:10:31 -0400
Subject: [PATCH] more robust code for setting daemon uid/gid

darcs-hash:20080904191031-84dde-bb457c429c76eedb9bd4ea838b7ccad28844effd.gz
---
 lib/daemon.php | 30 +++++++++++++++++++++---------
 1 file changed, 21 insertions(+), 9 deletions(-)

diff --git a/lib/daemon.php b/lib/daemon.php
index 0bda0272f4..a7a49ce94b 100644
--- a/lib/daemon.php
+++ b/lib/daemon.php
@@ -85,16 +85,28 @@ class Daemon {
 
 	function changeUser() {
 
-		if (common_config('daemon', 'user')) {
-			$user_info = posix_getpwnam(common_config('daemon', 'user'));
-			common_log(LOG_INFO, "Setting user to " . common_config('daemon', 'user'));
-			posix_setuid($user_info['uid']);
-		}
+		$username = common_config('daemon', 'user');
 		
-		if (common_config('daemon', 'group')) {
-			$group_info = posix_getgrnam(common_config('daemon', 'group'));
-			common_log(LOG_INFO, "Setting group to " . common_config('daemon', 'group'));
-			posix_setgid($group_info['gid']);
+		if ($username) {
+			$user_info = posix_getpwnam($username);
+			if (!$user_info) {
+				common_log(LOG_WARNING, 'Ignoring unknown user for daemon: ' . $username);
+			} else {
+				common_log(LOG_INFO, "Setting user to " . $username);
+				posix_setuid($user_info['uid']);
+			}
+		}
+
+		$groupname = common_config('daemon', 'group');
+		
+		if ($groupname) {
+			$group_info = posix_getgrnam($groupname);
+			if (!$group_info) {
+				common_log(LOG_WARNING, 'Ignoring unknown group for daemon: ' . $groupname);
+			} else {
+				common_log(LOG_INFO, "Setting group to " . $groupname);
+				posix_setgid($group_info['gid']);
+			}
 		}
 	}