X-Git-Url: http://git.marmaro.de/?a=blobdiff_plain;f=test%2Ffakesmtp.c;h=05be6deadeccab81be8737832c052ff0aa9ff916;hb=ab3e934551e83f49a04975b05c11c1b3d5768642;hp=700bc45d5ac0946e54733f97ced40db715ba95d5;hpb=fc6817f4dcebc81838636ab6dd01f945b80bac2a;p=mmh diff --git a/test/fakesmtp.c b/test/fakesmtp.c index 700bc45..05be6de 100644 --- a/test/fakesmtp.c +++ b/test/fakesmtp.c @@ -49,6 +49,33 @@ main(int argc, char *argv[]) exit(1); } + /* + * If there is a pid file already around, kill the previously running + * fakesmtp process. Hopefully this will reduce the race conditions + * that crop up when running the test suite. + */ + + if (stat(PIDFILE, &st) == 0) { + long oldpid; + + if (!(pid = fopen(PIDFILE, "r"))) { + fprintf(stderr, "Cannot open " PIDFILE + " (%s), continuing ...\n", strerror(errno)); + } else { + rc = fscanf(pid, "%ld", &oldpid); + fclose(pid); + + if (rc != 1) { + fprintf(stderr, "Unable to parse pid in " + PIDFILE ", continuing ...\n"); + } else { + kill((pid_t) oldpid, SIGTERM); + } + } + + unlink(PIDFILE); + } + memset(&hints, 0, sizeof(hints)); hints.ai_family = PF_INET; @@ -92,34 +119,10 @@ main(int argc, char *argv[]) } /* - * Now that our socket & files are set up, do the following things: - * - * - Check for a PID file. If there is one, kill the old version. - * - Wait 30 seconds for a connection. If there isn't one, then - * exit. + * Now that our socket & files are set up, wait 30 seconds for + * a connection. If there isn't one, then exit. */ - if (stat(PIDFILE, &st) == 0) { - long oldpid; - - if (!(pid = fopen(PIDFILE, "r"))) { - fprintf(stderr, "Cannot open " PIDFILE - " (%s), continuing ...\n", strerror(errno)); - } else { - rc = fscanf(pid, "%ld", &oldpid); - fclose(pid); - - if (rc != 1) { - fprintf(stderr, "Unable to parse pid in " - PIDFILE ", continuing ...\n"); - } else { - kill((pid_t) oldpid, SIGTERM); - } - } - - unlink(PIDFILE); - } - if (!(pid = fopen(PIDFILE, "w"))) { fprintf(stderr, "Cannot open " PIDFILE ": %s\n", strerror(errno));