X-Git-Url: https://diplodocus.org/git/nmh/blobdiff_plain/5dd6771b28c257af405d7248639ed0e3bcdce38b..3e4f493e89d831b319c585b3222004ffca5669f7:/sbr/context_save.c?ds=sidebyside

diff --git a/sbr/context_save.c b/sbr/context_save.c
index 4ee40ca8..70809595 100644
--- a/sbr/context_save.c
+++ b/sbr/context_save.c
@@ -23,6 +23,7 @@ context_save (void)
     register struct node *np;
     FILE *out;
     sigset_t set, oset;
+    int failed_to_lock = 0;
     
     /* No context in use -- silently ignore any changes! */
     if (!ctxpath)
@@ -41,18 +42,25 @@ context_save (void)
     sigaddset (&set, SIGINT);
     sigaddset (&set, SIGQUIT);
     sigaddset (&set, SIGTERM);
-    SIGPROCMASK (SIG_BLOCK, &set, &oset);
-
-    if (!(out = lkfopen (ctxpath, "w")))
-	adios (ctxpath, "unable to write");
+    sigprocmask (SIG_BLOCK, &set, &oset);
+
+    if (!(out = lkfopendata (ctxpath, "w", &failed_to_lock))) {
+	if (failed_to_lock) {
+	    adios (ctxpath, "failed to lock");
+	} else {
+	    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);
+    lkfclosedata (out, ctxpath);
 
-    SIGPROCMASK (SIG_SETMASK, &oset, &set); /* reset the signal mask */
+    sigprocmask (SIG_SETMASK, &oset, &set); /* reset the signal mask */
 
     if (action == 0)
+	/* This must be _exit(), not exit(), because the child didn't
+	   call unregister_for_removal() in m_chkids(). */
 	_exit (0);		/* we are child, time to die */
 }
 
@@ -83,8 +91,8 @@ m_chkids (void)
 	    break;
 
 	case 0:
-	    setgid (getgid ());
-	    setuid (getuid ());
+	    /* It's not necessary to call unregister_for_removal(0)
+               because the child calls _exit() in context_save(). */
 	    break;
 
 	default: