X-Git-Url: http://git.marmaro.de/?p=mmh;a=blobdiff_plain;f=uip%2Fslocal.c;h=e12b940e26e1c7c677625e4abbbe2103664623c2;hp=c1ee80fbb367cb2954d668436424128c406e3365;hb=b1450ec537bd31be606633aa363201cc2ed95b9a;hpb=1691e80890e5d8ba258c51c214a3e91880e1db2b

diff --git a/uip/slocal.c b/uip/slocal.c
index c1ee80f..e12b940 100644
--- a/uip/slocal.c
+++ b/uip/slocal.c
@@ -27,9 +27,24 @@
 #include <pwd.h>
 #include <signal.h>
 #include <sys/ioctl.h>
-#include <ndbm.h>
 #include <fcntl.h>
 
+#ifdef INITGROUPS_HEADER
+#include INITGROUPS_HEADER
+#else
+/* On AIX 4.1, initgroups() is defined and even documented (giving the parameter
+   types as char* and int), but doesn't have a prototype in any of the system
+   header files.  AIX 4.3 and SunOS 4.1.3 have the same problem. */
+extern int  initgroups(char*, int);
+#endif
+
+
+#ifdef HAVE_DB1_NDBM_H
+#include <db1/ndbm.h>
+#else
+#include <ndbm.h>
+#endif
+
 #include <utmp.h>
 
 #ifndef UTMP_FILE
@@ -70,7 +85,7 @@ static struct swit switches[] = {
 #define VERSIONSW     13
     { "version", 0 },
 #define	HELPSW        14
-    { "help", 4 },
+    { "help", 0 },
     { NULL, 0 }
 };
 
@@ -334,18 +349,29 @@ main (int argc, char **argv)
 	if ((fd = copy_message (fileno (stdin), tmpfil, 1)) == -1)
 	    adios (NULL, "unable to create temporary file");
     }
+
     if (debug)
 	debug_printf ("temporary file=\"%s\"\n", tmpfil);
-    else
-	unlink (tmpfil);
+
+    /* Delete the temp file now or a copy of every single message passed through
+       slocal will be left in the /tmp directory until deleted manually!  This
+       unlink() used to be under an 'else' of the 'if (debug)' above, but since
+       some people like to always run slocal with -debug and log the results,
+       the /tmp directory would get choked over time.  Of course, now that we
+       always delete the temp file, the "temporary file=" message above is
+       somewhat pointless -- someone watching debug output wouldn't have a
+       chance to 'tail -f' or 'ln' the temp file before it's unlinked.  The best
+       thing would be to delay this unlink() until later if debug == 1, but I'll
+       leave that for someone who cares about the temp-file-accessing
+       functionality (they'll have to watch out for cases where we adios()). */
+    unlink (tmpfil);
 
     if (!(fp = fdopen (fd, "r+")))
 	adios (NULL, "unable to access temporary file");
 
     /*
      * If no sender given, extract it
-     * from envelope information.
-     */
+     * from envelope information.  */
     if (sender == NULL)
 	get_sender (envelope, &sender);
 
@@ -373,7 +399,7 @@ main (int argc, char **argv)
     /* deliver the message */
     status = localmail (fd, mdlvr);
 
-    done (status != -1 ? RCV_MOK : RCV_MBX);
+    return done (status != -1 ? RCV_MOK : RCV_MBX);
 }
 
 
@@ -740,7 +766,7 @@ parse (int fd)
 		for (p = hdrs; p->p_name; p++) {
 		    if (!strcasecmp (p->p_name, name)) {
 			if (!(p->p_flags & P_HID)) {
-			    if ((cp = p->p_value))
+			    if ((cp = p->p_value)) {
 				if (p->p_flags & P_ADR) {
 				    dp = cp + strlen (cp) - 1;
 				    if (*dp == '\n')
@@ -749,6 +775,7 @@ parse (int fd)
 				} else {
 				    cp = add ("\t", cp);
 				}
+			    }
 			    p->p_value = add (lp, cp);
 			}
 			free (lp);