Re-add --with-lockdir=DIR configure option
[mmh] / sbr / lock_file.c
index d3d447e..7309c86 100644 (file)
 ** Ruud de Rooij <ruud@debian.org>  Sun, 28 Mar 1999 15:34:03 +0200
 */
 
+#include <unistd.h>
 #include <h/mh.h>
 #include <h/signals.h>
 #include <h/utils.h>
+#include <sys/stat.h>
 
-#ifdef TIME_WITH_SYS_TIME
+#ifdef HAVE_SYS_TIME_H
 # include <sys/time.h>
-# include <time.h>
-#else
-# ifdef TM_IN_SYS_TIME
-#  include <sys/time.h>
-# else
-#  include <time.h>
-# endif
 #endif
+#include <time.h>
 
-#ifdef HAVE_ERRNO_H
-# include <errno.h>
-#endif
+#include <errno.h>
 
 #ifdef HAVE_FCNTL_H
 # include <fcntl.h>
 #include <lockfile.h>
 #endif
 
-#ifdef LOCKDIR
+#ifdef DOT_LOCKING
+# ifdef LOCKDIR
 char *lockdir = LOCKDIR;
+# endif
 #endif
 
 /* Are we using any kernel locking? */
@@ -98,19 +94,19 @@ static struct lock *l_top = NULL;
 ** static prototypes
 */
 #ifdef KERNEL_LOCKING
-static int lkopen_kernel (char *, int, mode_t);
+static int lkopen_kernel(char *, int, mode_t);
 #endif
 
 #ifdef DOT_LOCKING
-static int lkopen_dot (char *, int, mode_t);
-static void lockname (char *, struct lockinfo *, int);
-static void timerON (char *, int);
-static void timerOFF (int);
-static RETSIGTYPE alrmser (int);
-#endif
+static int lkopen_dot(char *, int, mode_t);
+static void lockname(char *, struct lockinfo *, int);
+static void timerON(char *, int);
+static void timerOFF(int);
+static void alrmser(int);
 
 #if !defined(HAVE_LIBLOCKFILE)
-static int lockit (struct lockinfo *);
+static int lockit(struct lockinfo *);
+#endif
 #endif
 
 /*
@@ -119,7 +115,7 @@ static int lockit (struct lockinfo *);
 */
 
 int
-lkopen (char *file, int access, mode_t mode)
+lkopen(char *file, int access, mode_t mode)
 {
 #ifdef KERNEL_LOCKING
        return lkopen_kernel(file, access, mode);
@@ -137,7 +133,7 @@ lkopen (char *file, int access, mode_t mode)
 */
 
 int
-lkclose (int fd, char *file)
+lkclose(int fd, char *file)
 {
 #ifdef FCNTL_LOCKING
        struct flock buf;
@@ -159,26 +155,26 @@ lkclose (int fd, char *file)
 #endif
 
 #ifdef FLOCK_LOCKING
-       flock (fd, LOCK_UN);
+       flock(fd, LOCK_UN);
 #endif
 
 #ifdef LOCKF_LOCKING
        /* make sure we unlock the whole thing */
-       lseek (fd, (off_t) 0, SEEK_SET);
-       lockf (fd, F_ULOCK, 0L);
+       lseek(fd, (off_t) 0, SEEK_SET);
+       lockf(fd, F_ULOCK, 0L);
 #endif
 
 #ifdef DOT_LOCKING
-       lockname (file, &lkinfo, 0);  /* get name of lock file */
+       lockname(file, &lkinfo, 0);  /* get name of lock file */
 #if !defined(HAVE_LIBLOCKFILE)
-       unlink (lkinfo.curlock);  /* remove lock file */
+       unlink(lkinfo.curlock);  /* remove lock file */
 #else
        lockfile_remove(lkinfo.curlock);
 #endif /* HAVE_LIBLOCKFILE */
-       timerOFF (fd);  /* turn off lock timer   */
+       timerOFF(fd);  /* turn off lock timer   */
 #endif /* DOT_LOCKING */
 
-       return (close (fd));
+       return (close(fd));
 }
 
 
@@ -188,33 +184,33 @@ lkclose (int fd, char *file)
 */
 
 FILE *
-lkfopen (char *file, char *mode)
+lkfopen(char *file, char *mode)
 {
        int fd, access;
        FILE *fp;
 
-       if (strcmp (mode, "r") == 0)
+       if (strcmp(mode, "r") == 0)
                access = O_RDONLY;
-       else if (strcmp (mode, "r+") == 0)
+       else if (strcmp(mode, "r+") == 0)
                access = O_RDWR;
-       else if (strcmp (mode, "w") == 0)
+       else if (strcmp(mode, "w") == 0)
                access = O_WRONLY | O_CREAT | O_TRUNC;
-       else if (strcmp (mode, "w+") == 0)
+       else if (strcmp(mode, "w+") == 0)
                access = O_RDWR | O_CREAT | O_TRUNC;
-       else if (strcmp (mode, "a") == 0)
+       else if (strcmp(mode, "a") == 0)
                access = O_WRONLY | O_CREAT | O_APPEND;
-       else if (strcmp (mode, "a+") == 0)
+       else if (strcmp(mode, "a+") == 0)
                access = O_RDWR | O_CREAT | O_APPEND;
        else {
                errno = EINVAL;
                return NULL;
        }
 
-       if ((fd = lkopen (file, access, 0666)) == -1)
+       if ((fd = lkopen(file, access, 0666)) == -1)
                return NULL;
 
-       if ((fp = fdopen (fd, mode)) == NULL) {
-               close (fd);
+       if ((fp = fdopen(fd, mode)) == NULL) {
+               close(fd);
                return NULL;
        }
 
@@ -228,7 +224,7 @@ lkfopen (char *file, char *mode)
 */
 
 int
-lkfclose (FILE *fp, char *file)
+lkfclose(FILE *fp, char *file)
 {
 #ifdef FCNTL_LOCKING
        struct flock buf;
@@ -250,26 +246,26 @@ lkfclose (FILE *fp, char *file)
 #endif
 
 #ifdef FLOCK_LOCKING
-       flock (fileno(fp), LOCK_UN);
+       flock(fileno(fp), LOCK_UN);
 #endif
 
 #ifdef LOCKF_LOCKING
        /* make sure we unlock the whole thing */
-       fseek (fp, 0L, SEEK_SET);
-       lockf (fileno(fp), F_ULOCK, 0L);
+       fseek(fp, 0L, SEEK_SET);
+       lockf(fileno(fp), F_ULOCK, 0L);
 #endif
 
 #ifdef DOT_LOCKING
-       lockname (file, &lkinfo, 0);  /* get name of lock file */
+       lockname(file, &lkinfo, 0);  /* get name of lock file */
 #if !defined(HAVE_LIBLOCKFILE)
-       unlink (lkinfo.curlock);  /* remove lock file */
+       unlink(lkinfo.curlock);  /* remove lock file */
 #else
        lockfile_remove(lkinfo.curlock);
 #endif /* HAVE_LIBLOCKFILE */
-       timerOFF (fileno(fp));  /* turn off lock timer   */
+       timerOFF(fileno(fp));  /* turn off lock timer   */
 #endif /* DOT_LOCKING */
 
-       return (fclose (fp));
+       return (fclose(fp));
 }
 
 
@@ -280,7 +276,7 @@ lkfclose (FILE *fp, char *file)
 */
 
 static int
-lkopen_kernel (char *file, int access, mode_t mode)
+lkopen_kernel(char *file, int access, mode_t mode)
 {
        int fd, i, j;
 
@@ -307,7 +303,7 @@ lkopen_kernel (char *file, int access, mode_t mode)
                }
 # endif /* LOCKF_LOCKING || FCNTL_LOCKING */
 
-               if ((fd = open (file, access | O_NDELAY, mode)) == -1)
+               if ((fd = open(file, access | O_NDELAY, mode)) == -1)
                        return -1;
 
 # ifdef FCNTL_LOCKING
@@ -315,31 +311,31 @@ lkopen_kernel (char *file, int access, mode_t mode)
                buf.l_whence = SEEK_SET;
                buf.l_start  = 0;
                buf.l_len = 0;
-               if (fcntl (fd, F_SETLK, &buf) != -1)
+               if (fcntl(fd, F_SETLK, &buf) != -1)
                        return fd;
 # endif
 
 # ifdef FLOCK_LOCKING
-               if (flock (fd, (((access & 03) == O_RDONLY) ? LOCK_SH :
+               if (flock(fd, (((access & 03) == O_RDONLY) ? LOCK_SH :
                        LOCK_EX) | LOCK_NB) != -1)
                        return fd;
 # endif
 
 # ifdef LOCKF_LOCKING
-               if (lockf (fd, F_TLOCK, 0L) != -1) {
+               if (lockf(fd, F_TLOCK, 0L) != -1) {
                        /* see if we should be at the end */
                        if (j & O_APPEND)
-                               lseek (fd, (off_t) 0, SEEK_END);
+                               lseek(fd, (off_t) 0, SEEK_END);
                        return fd;
                }
 # endif
 
                j = errno;
-               close (fd);
-               sleep (5);
+               close(fd);
+               sleep(1);
        }
 
-       close (fd);
+       close(fd);
        errno = j;
        return -1;
 }
@@ -354,29 +350,29 @@ lkopen_kernel (char *file, int access, mode_t mode)
 */
 
 static int
-lkopen_dot (char *file, int access, mode_t mode)
+lkopen_dot(char *file, int access, mode_t mode)
 {
        int fd;
        struct lockinfo lkinfo;
 
        /* open the file */
-       if ((fd = open (file, access, mode)) == -1)
+       if ((fd = open(file, access, mode)) == -1)
                return -1;
 
        /*
        ** Get the name of the eventual lock file, as well
        ** as a name for a temporary lock file.
        */
-       lockname (file, &lkinfo, 1);
+       lockname(file, &lkinfo, 1);
 
 #if !defined(HAVE_LIBLOCKFILE)
        {
                int i;
                for (i = 0;;) {
                        /* attempt to create lock file */
-                       if (lockit (&lkinfo) == 0) {
+                       if (lockit(&lkinfo) == 0) {
                                /* if successful, turn on timer and return */
-                               timerON (lkinfo.curlock, fd);
+                               timerON(lkinfo.curlock, fd);
                                return fd;
                        } else {
                                /*
@@ -385,25 +381,25 @@ lkopen_dot (char *file, int access, mode_t mode)
                                ** lock file.
                                */
                                struct stat st;
-                               if (stat (lkinfo.curlock, &st) == -1) {
+                               if (stat(lkinfo.curlock, &st) == -1) {
                                        if (i++ > 5)
                                                return -1;
-                                       sleep (5);
+                                       sleep(1);
                                } else {
                                        time_t curtime;
                                        i = 0;
-                                       time (&curtime);
+                                       time(&curtime);
 
                                        /*
                                        ** check for stale lockfile,
                                        ** else sleep
                                        */
                                        if (curtime > st.st_ctime + RSECS)
-                                               unlink (lkinfo.curlock);
+                                               unlink(lkinfo.curlock);
                                        else
-                                               sleep (5);
+                                               sleep(1);
                                }
-                               lockname (file, &lkinfo, 1);
+                               lockname(file, &lkinfo, 1);
                        }
                }
        }
@@ -425,7 +421,7 @@ lkopen_dot (char *file, int access, mode_t mode)
 */
 
 static int
-lockit (struct lockinfo *li)
+lockit(struct lockinfo *li)
 {
        int fd;
        char *curlock, *tmplock;
@@ -437,26 +433,18 @@ lockit (struct lockinfo *li)
        curlock = li->curlock;
        tmplock = li->tmplock;
 
-#ifdef HAVE_MKSTEMP
-       if ((fd = mkstemp(tmplock)) == -1)
-               return -1;
-#else
-       if (mktemp(tmplock) == NULL)
+       if ((fd = mkstemp(tmplock)) == -1) {
+               advise(NULL, "unable to create temporary file in %s", tmplock);
                return -1;
-       if (unlink(tmplock) == -1 && errno != ENOENT)
-               return -1;
-       /* create the temporary lock file */
-       if ((fd = creat(tmplock, 0600)) == -1)
-               return -1;
-#endif
+       }
 
 #if 0
        /* write our process id into lock file */
-       snprintf (buffer, sizeof(buffer), "nmh lock: pid %d\n", (int) getpid());
+       snprintf(buffer, sizeof(buffer), "nmh lock: pid %d\n", (int) getpid());
        write(fd, buffer, strlen(buffer) + 1);
 #endif
 
-       close (fd);
+       close(fd);
 
        /*
        ** Now try to create the real lock file
@@ -474,48 +462,31 @@ lockit (struct lockinfo *li)
 */
 
 static void
-lockname (char *file, struct lockinfo *li, int isnewlock)
+lockname(char *file, struct lockinfo *li, int isnewlock)
 {
        int bplen, tmplen;
        char *bp, *cp;
 
-#if 0
-       struct stat st;
-#endif
-
-       if ((cp = strrchr (file, '/')) == NULL || *++cp == 0)
+       if ((cp = strrchr(file, '/')) == NULL || *++cp == 0)
                cp = file;
 
        bp = li->curlock;
        bplen = 0;
 #ifdef LOCKDIR
-       snprintf (bp, sizeof(li->curlock), "%s/", lockdir);
-       tmplen = strlen (bp);
+       snprintf(bp, sizeof(li->curlock), "%s/", lockdir);
+       tmplen = strlen(bp);
        bp += tmplen;
        bplen += tmplen;
 #else
        if (cp != file) {
-               snprintf (bp, sizeof(li->curlock), "%.*s", (int)(cp - file), file);
-               tmplen = strlen (bp);
+               snprintf(bp, sizeof(li->curlock), "%.*s", (int)(cp - file), file);
+               tmplen = strlen(bp);
                bp += tmplen;
                bplen += tmplen;
        }
 #endif
 
-#if 0
-       /*
-       ** mmdf style dot locking.  Currently not supported.
-       ** If we start supporting mmdf style dot locking,
-       ** we will need to change the return value of lockname
-       */
-       if (stat (file, &st) == -1)
-               return -1;
-
-       snprintf (bp, sizeof(li->curlock) - bplen, "LCK%05d.%05d",
-               st.st_dev, st.st_ino);
-#endif
-
-       snprintf (bp, sizeof(li->curlock) - bplen, "%s.lock", cp);
+       snprintf(bp, sizeof(li->curlock) - bplen, "%s.lock", cp);
 
 #if !defined(HAVE_LIBLOCKFILE)
        /*
@@ -523,11 +494,12 @@ lockname (char *file, struct lockinfo *li, int isnewlock)
        ** the temporary lock file for lockit()
        */
        if (isnewlock) {
-               if ((cp = strrchr (li->curlock, '/')) == NULL || *++cp == 0)
-                       strncpy (li->tmplock, ",LCK.XXXXXX", sizeof(li->tmplock));
+               if ((cp = strrchr(li->curlock, '/')) == NULL || *++cp == 0)
+                       strncpy(li->tmplock, ",LCK.XXXXXX", sizeof(li->tmplock));
                else
-                       snprintf (li->tmplock, sizeof(li->tmplock), "%.*s,LCK.XXXXXX",
-                                (int)(cp - li->curlock), li->curlock);
+                       snprintf(li->tmplock, sizeof(li->tmplock),
+                                       "%.*s,LCK.XXXXXX",
+                                       (int)(cp - li->curlock), li->curlock);
        }
 #endif
 }
@@ -539,23 +511,23 @@ lockname (char *file, struct lockinfo *li, int isnewlock)
 */
 
 static void
-timerON (char *curlock, int fd)
+timerON(char *curlock, int fd)
 {
        struct lock *lp;
        size_t len;
 
-       lp = (struct lock *) mh_xmalloc (sizeof(*lp));
+       lp = (struct lock *) mh_xmalloc(sizeof(*lp));
 
        len = strlen(curlock) + 1;
        lp->l_fd = fd;
-       lp->l_lock = mh_xmalloc (len);
-       memcpy (lp->l_lock, curlock, len);
+       lp->l_lock = mh_xmalloc(len);
+       memcpy(lp->l_lock, curlock, len);
        lp->l_next = l_top;
 
        if (!l_top) {
                /* perhaps SIGT{STP,TIN,TOU} ? */
-               SIGNAL (SIGALRM, alrmser);
-               alarm (NSECS);
+               SIGNAL(SIGALRM, alrmser);
+               alarm(NSECS);
        }
 
        l_top = lp;
@@ -568,7 +540,7 @@ timerON (char *curlock, int fd)
 */
 
 static void
-timerOFF (int fd)
+timerOFF(int fd)
 {
        struct lock *pp, *lp;
 
@@ -585,14 +557,14 @@ timerOFF (int fd)
                        else
                                pp->l_next = lp->l_next;
 
-                       free (lp->l_lock);
-                       free (lp);
+                       free(lp->l_lock);
+                       free(lp);
                }
        }
 
        /* if there are locks left, restart timer */
        if (l_top)
-               alarm (NSECS);
+               alarm(NSECS);
 }
 
 
@@ -601,24 +573,20 @@ timerOFF (int fd)
 ** lockfiles, so another command doesn't remove them.
 */
 
-static RETSIGTYPE
-alrmser (int sig)
+static void
+alrmser(int sig)
 {
        char *lockfile;
        struct lock *lp;
 
-#ifndef RELIABLE_SIGNALS
-       SIGNAL (SIGALRM, alrmser);
-#endif
-
        /* update the ctime of all the lock files */
        for (lp = l_top; lp; lp = lp->l_next) {
                lockfile = lp->l_lock;
 #if !defined(HAVE_LIBLOCKFILE)
                {
                        int j;
-                       if (*lockfile && (j = creat (lockfile, 0600)) != -1)
-                               close (j);
+                       if (*lockfile && (j = creat(lockfile, 0600)) != -1)
+                               close(j);
                }
 #else
        lockfile_touch(lockfile);
@@ -626,7 +594,7 @@ alrmser (int sig)
        }
 
        /* restart the alarm */
-       alarm (NSECS);
+       alarm(NSECS);
 }
 
 #endif /* DOT_LOCKING */