We are making POSIX signal support a requirement.

[mmh] / sbr / context_save.c

diff --git a/sbr/context_save.c b/sbr/context_save.c
index d4a56ed..d1371d0 100644 (file)
--- a/sbr/context_save.c
+++ b/sbr/context_save.c
@@ -1,22 +1,22 @@
 /*
- * context_save.c -- write out the updated context file
- *
- * This code is Copyright (c) 2002, by the authors of nmh.  See the
- * COPYRIGHT file in the root directory of the nmh distribution for
- * complete copyright information.
- */
+** context_save.c -- write out the updated context file
+**
+** This code is Copyright (c) 2002, by the authors of nmh.  See the
+** COPYRIGHT file in the root directory of the nmh distribution for
+** complete copyright information.
+*/
 
 #include <h/mh.h>
 #include <h/signals.h>
 
 /*
- * static prototypes
- */
+** static prototypes
+*/
 static int m_chkids(void);
 
 
 void
-context_save (void)
+context_save(void)
 {
        int action;
        register struct node *np;
@@ -31,64 +31,60 @@ context_save (void)
                return;
        ctxflags &= ~CTXMOD;
 
-       if ((action = m_chkids ()) > 0)
+       if ((action = m_chkids()) > 0)
                return;  /* child did it for us */
 
        /* block a few signals */
-       sigemptyset (&set);
-       sigaddset (&set, SIGHUP);
-       sigaddset (&set, SIGINT);
-       sigaddset (&set, SIGQUIT);
-       sigaddset (&set, SIGTERM);
-       SIGPROCMASK (SIG_BLOCK, &set, &oset);
-
-       if (!(out = lkfopen (ctxpath, "w")))
-               adios (ctxpath, "unable to write");
+       sigemptyset(&set);
+       sigaddset(&set, SIGHUP);
+       sigaddset(&set, SIGINT);
+       sigaddset(&set, SIGQUIT);
+       sigaddset(&set, SIGTERM);
+       sigprocmask(SIG_BLOCK, &set, &oset);
+
+       if (!(out = lkfopen(ctxpath, "w")))
+               adios(ctxpath, "unable to write");
        for (np = m_defs; np; np = np->n_next)
                if (np->n_context)
-                       fprintf (out, "%s: %s\n", np->n_name, np->n_field);
-       lkfclose (out, ctxpath);
+                       fprintf(out, "%s: %s\n", np->n_name, np->n_field);
+       lkfclose(out, ctxpath);
 
-       SIGPROCMASK (SIG_SETMASK, &oset, &set); /* reset the signal mask */
+       sigprocmask(SIG_SETMASK, &oset, &set); /* reset the signal mask */
 
        if (action == 0)
-               _exit (0);    /* we are child, time to die */
+               _exit(0);    /* we are child, time to die */
 }
 
 /*
- * This hack brought to you so we can handle set[ug]id MH programs.
- * If we return -1, then no fork is made, we update .mh_profile
- * normally, and return to the caller normally.  If we return 0,
- * then the child is executing, .mh_profile is modified after
- * we set our [ug]ids to the norm.  If we return > 0, then the
- * parent is executed and .mh_profile has already be modified.
- * We can just return to the caller immediately.
- */
+** This hack brought to you so we can handle set[ug]id MH programs.
+** If we return -1, then no fork is made, we update .mh_profile
+** normally, and return to the caller normally.  If we return 0,
+** then the child is executing, .mh_profile is modified after
+** we set our [ug]ids to the norm.  If we return > 0, then the
+** parent is executed and .mh_profile has already be modified.
+** We can just return to the caller immediately.
+*/
 
 static int
-m_chkids (void)
+m_chkids(void)
 {
-       int i;
        pid_t pid;
 
-       if (getuid () == geteuid ())
+       if (getuid() == geteuid())
                return (-1);
 
-       for (i = 0; (pid = fork ()) == -1 && i < 5; i++)
-               sleep (5);
+       switch (pid = fork()) {
+       case -1:
+               break;
 
-       switch (pid) {
-               case -1:
-                       break;
+       case 0:
+               setgid(getgid());
+               setuid(getuid());
+               break;
 
-               case 0:
-                       setgid (getgid ());
-                       setuid (getuid ());
-                       break;
-
-               default:
-                       pidwait (pid, -1);
-                       break;
+       default:
+               pidwait(pid, -1);
+               break;
        }
 
        return pid;