Refined number of bytes copied to buffer in several places.

[nmh] / uip / viamail.c

diff --git a/uip/viamail.c b/uip/viamail.c
index 6001183b698d42a9e34b76069ccffd50fee1d596..8bfb91d0099911a7d1cdb54d2cc2ce76c07ce64c 100644 (file)
--- a/uip/viamail.c
+++ b/uip/viamail.c
@@ -6,6 +6,11 @@
  */
 
 #include "h/mh.h"
+#include "sendsbr.h"
+#include "sbr/m_gmprot.h"
+#include "sbr/getarguments.h"
+#include "sbr/concat.h"
+#include "sbr/smatch.h"
 #include "sbr/context_find.h"
 #include "sbr/ambigsw.h"
 #include "sbr/print_version.h"
@@ -13,7 +18,6 @@
 #include "sbr/arglist.h"
 #include "sbr/error.h"
 #include <fcntl.h>
-#include "h/md5.h"
 #include "h/mts.h"
 #include "h/tws.h"
 #include "h/mime.h"
@@ -30,7 +34,6 @@
     X("parameters arguments", 0, PARAMSW) \
     X("description text", 0, DESCRIPTSW) \
     X("comment text", 0, COMMENTSW) \
-    X("delay seconds", 0, DELAYSW) \
     X("verbose", 0, VERBSW) \
     X("noverbose", 0, NVERBSW) \
     X("version", 0, VERSIONSW) \
@@ -46,21 +49,19 @@ DEFINE_SWITCH_ARRAY(VIAMAIL, switches);
 #undef X
 
 extern int debugsw;
-extern int splitsw;
 extern bool verbsw;
 
 /*
  * static prototypes
  */
-static int via_mail (char *, char *, char *, char *, char *, int, char *);
+static int via_mail (char *, char *, char *, char *, char *, char *);
 
 
 int
 main (int argc, char **argv)
 {
-    int delay = 0;
     char *f1 = NULL, *f2 = NULL, *f3 = NULL;
-    char *f4 = NULL, *f5 = NULL, *f7 = NULL;
+    char *f4 = NULL, *f5 = NULL, *f6 = NULL;
     char *cp, buf[BUFSIZ];
     char **argp, **arguments;
 
@@ -106,19 +107,8 @@ main (int argc, char **argv)
                if (!(f5 = *argp++))
                    die("missing argument to %s", argp[-2]);
                continue;
-           case DELAYSW:
-               if (!(cp = *argp++) || *cp == '-')
-                   die("missing argument to %s", argp[-2]);
-
-               /*
-                * If there is an error, just reset the delay parameter
-                * to -1.  We will set a default delay later.
-                */
-               if (sscanf (cp, "%d", &delay) != 1)
-                   delay = -1;
-               continue;
            case FROMSW:
-               if (!(f7 = *argp++))
+               if (!(f6 = *argp++))
                    die("missing argument to %s", argp[-2]);
                continue;
 
@@ -139,7 +129,7 @@ main (int argc, char **argv)
     if (!f1)
        die("missing -viamail \"mailpath\" switch");
 
-    via_mail (f1, f2, f3, f4, f5, delay, f7);
+    via_mail (f1, f2, f3, f4, f5, f6);
     return 0;  /* dead code to satisfy the compiler */
 }
 
@@ -150,7 +140,7 @@ main (int argc, char **argv)
 
 static int
 via_mail (char *mailsw, char *subjsw, char *parmsw, char *descsw,
-          char *cmntsw, int delay, char *fromsw)
+          char *cmntsw, char *fromsw)
 {
     int        status, vecp;
     char tmpfil[BUFSIZ], *program;
@@ -165,7 +155,7 @@ via_mail (char *mailsw, char *subjsw, char *parmsw, char *descsw,
     if ((tfile = m_mktemp2(NULL, invo_name, NULL, &fp)) == NULL) {
        die("unable to create temporary file in %s", get_temp_dir());
     }
-    strncpy (tmpfil, tfile, sizeof(tmpfil));
+    strncpy (tmpfil, tfile, sizeof(tmpfil) - 1);
 
     if (!strchr(mailsw, '@'))
        mailsw = concat (mailsw, "@", LocalName (0), NULL);
@@ -204,11 +194,6 @@ via_mail (char *mailsw, char *subjsw, char *parmsw, char *descsw,
     if (fstat (fileno (fp), &st) == NOTOK)
        adios ("failed", "fstat of %s", tmpfil);
 
-    if (delay < 0)
-       splitsw = 10;
-    else
-       splitsw = delay;
-
     status = 0;
 
     vec = argsplit(postproc, &program, &vecp);