]> granicus.if.org Git - shadow/commitdiff
pid_child and kill_child are only used when USE_PAM.
authornekral-guest <nekral-guest@5a98b0ae-9ef6-0310-add3-de5d479b70d7>
Tue, 8 Sep 2009 21:33:14 +0000 (21:33 +0000)
committernekral-guest <nekral-guest@5a98b0ae-9ef6-0310-add3-de5d479b70d7>
Tue, 8 Sep 2009 21:33:14 +0000 (21:33 +0000)
src/su.c

index f6d55c10990376549c4a0e8373c2824eaedd52b2..871d7fee9fdd786565c4d74a473f0794ded587bb 100644 (file)
--- a/src/su.c
+++ b/src/su.c
@@ -78,8 +78,6 @@
  * Global variables
  */
 char *Prog;
-/* PID of the child, in case it needs to be killed */
-static pid_t pid_child = 0;
 
 /* not needed by sulog.c anymore */
 static char name[BUFSIZ];
@@ -91,6 +89,8 @@ static bool change_environment;
 #ifdef USE_PAM
 static pam_handle_t *pamh = NULL;
 static bool caught = false;
+/* PID of the child, in case it needs to be killed */
+static pid_t pid_child = 0;
 #endif
 
 extern struct passwd pwent;
@@ -108,8 +108,9 @@ extern size_t newenvc;
 static void execve_shell (const char *shellstr,
                           char *args[],
                           char *const envp[]);
-static RETSIGTYPE kill_child (int s);
-#ifndef USE_PAM
+#ifdef USE_PAM
+static RETSIGTYPE kill_child (int unused(s));
+#else                          /* !USE_PAM */
 static RETSIGTYPE die (int);
 static int iswheel (const char *);
 #endif                         /* !USE_PAM */
@@ -148,8 +149,7 @@ static int iswheel (const char *username)
        }
        return is_on_list (grp->gr_mem, username);
 }
-#endif                         /* !USE_PAM */
-
+#else                          /* USE_PAM */
 static RETSIGTYPE kill_child (int unused(s))
 {
        if (0 != pid_child) {
@@ -161,6 +161,7 @@ static RETSIGTYPE kill_child (int unused(s))
        }
        exit (255);
 }
+#endif                         /* USE_PAM */
 
 /* borrowed from GNU sh-utils' "su.c" */
 static bool restricted_shell (const char *shellstr)