Changeset 23310

Timestamp:

07/16/07 08:47:31 (17 months ago)

Author:

zarzycki@…

Message:

clarity++

Files:

• trunk/launchd/src/launchd_core_logic.c (modified) (2 diffs)

trunk/launchd/src/launchd_core_logic.c

@@ -2608 +2608 @@
         }
 
+        /*
+         * We must copy the results of getpw*().
+         *
+         * Why? Because subsequent API calls may call getpw*() as a part of
+         * their implementation. Since getpw*() returns a [now thread scoped]
+         * global, we must therefore cache the results before continuing.
+         */
+
+        desired_uid = pwe->pw_uid;
+        desired_gid = pwe->pw_gid;
+
         strlcpy(shellpath, pwe->pw_shell, sizeof(shellpath));
         strlcpy(loginname, pwe->pw_name, sizeof(loginname));
@@ -2617 +2628 @@
         }
 
-        desired_uid = pwe->pw_uid;
-        desired_gid = pwe->pw_gid;
 
         if (j->username && strcmp(j->username, loginname) != 0) {