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 c25205424e38206c73e62c2d09037c3e1ce681df..8bfb91d0099911a7d1cdb54d2cc2ce76c07ce64c 100644 (file)
--- a/uip/viamail.c
+++ b/uip/viamail.c
@@ -5,15 +5,25 @@
  * complete copyright information.
  */
 
-#include <h/mh.h>
+#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"
+#include "sbr/print_help.h"
+#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>
-#include <h/mhparse.h>
+#include "h/mts.h"
+#include "h/tws.h"
+#include "h/mime.h"
+#include "h/mhparse.h"
 #include "h/done.h"
-#include <h/utils.h>
+#include "h/utils.h"
 #include "sbr/m_mktemp.h"
 #include "sbr/base64.h"
 
@@ -24,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) \
@@ -40,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;
 
@@ -100,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;
 
@@ -133,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 */
 }
 
@@ -144,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;
@@ -159,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);
@@ -198,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);