chiark / gitweb /
bsmtp-pull: Note race condition pointed out by Peter Benie.
authorColin Watson <cjwatson@chiark.greenend.org.uk>
Sun, 11 Jul 2004 22:34:59 +0000 (22:34 +0000)
committerColin Watson <cjwatson@chiark.greenend.org.uk>
Sun, 11 Jul 2004 22:34:59 +0000 (22:34 +0000)
bsmtp-pull

index 5c95f92..600763d 100755 (executable)
@@ -19,6 +19,8 @@ HOST="$1"
 # By default, lockfile-create gives up after three minutes, so don't cron
 # this any more frequently than that without supplying a --retry argument.
 lockfile-create "$HOST"
+# Race condition pointed out by pjb: this doesn't guarantee that the lock is
+# held before the critical section starts.
 lockfile-touch "$HOST" &
 TOUCH="$!"
 trap 'kill "$TOUCH"; lockfile-remove "$HOST"' EXIT ERR HUP INT QUIT TERM