Doubled size of buffers used by m_getfld(), scan, and fmttest to

[nmh] / uip / dropsbr.c

diff --git a/uip/dropsbr.c b/uip/dropsbr.c
index 7dc0a87be17ffdf140518e875cabd5b91ad7e99b..915e71e6c73e09328cb8740d5196720639cb2b9a 100644 (file)
--- a/uip/dropsbr.c
+++ b/uip/dropsbr.c
@@ -47,26 +47,16 @@ mbx_open (char *file, int mbx_style, uid_t uid, gid_t gid, mode_t mode)
 
     /* attempt to open and lock file */
     for (count = 4; count > 0; count--) {
 
     /* attempt to open and lock file */
     for (count = 4; count > 0; count--) {

+        int failed_to_lock = 0;

        if ((fd = lkopenspool (file, O_RDWR | O_CREAT |
        if ((fd = lkopenspool (file, O_RDWR | O_CREAT |

-                              O_NONBLOCK, mode)) == NOTOK) {
-           switch (errno) {
-#if defined(FCNTL_LOCKING) || defined(LOCKF_LOCKING)
-               case EACCES:
-               case EAGAIN:
-#endif
-
-#ifdef FLOCK_LOCKING
-               case EWOULDBLOCK:
-#endif
-               case ETXTBSY: 
-                   j = errno;
-                   sleep (5);
-                   continue;
-
-               default: 
-                   /* just return error */
-                   return NOTOK;
-           }
+                              O_NONBLOCK, mode, &failed_to_lock)) == NOTOK) {
+            if (failed_to_lock) {
+                j = errno;
+                sleep (5);
+                continue;
+            } else {
+                return NOTOK;
+            }

        }
 
        /* good file descriptor */
        }
 
        /* good file descriptor */


@@ -306,6 +296,10 @@ mbx_copy (char *mailbox, int mbx_style, int md, int fd,
            }
                    
            while ((i = read (fd, buffer, sizeof(buffer))) > 0) {
            }
                    
            while ((i = read (fd, buffer, sizeof(buffer))) > 0) {

+                /* valgrind noticed that stringdex depends on null
+                   termination. */
+                buffer[i] = '\0';
+

                for (j = 0;
                        (j = stringdex (mmdlm1, buffer)) >= 0;
                        buffer[j]++)
                for (j = 0;
                        (j = stringdex (mmdlm1, buffer)) >= 0;
                        buffer[j]++)


@@ -563,7 +557,7 @@ map_write (char *mailbox, int md, int id, long last, off_t start,
        clear = 1;
 
     if (!clear && map_chk (file, fd, &d1, pos, noisy)) {
        clear = 1;
 
     if (!clear && map_chk (file, fd, &d1, pos, noisy)) {

-       unlink (file);
+       (void) m_unlink (file);

        mbx_close (file, fd);
        if ((fd = map_open (file, md)) == NOTOK)
            return NOTOK;
        mbx_close (file, fd);
        if ((fd = map_open (file, md)) == NOTOK)
            return NOTOK;