bsmtp-pull: Note race condition pointed out by Peter Benie.

[bin.git] / bsmtp-pull

diff --git a/bsmtp-pull b/bsmtp-pull
index 2c7e7b9813135bca6a6d73d47e24abd8e891189f..600763d58e4540b7cd1c6f2ee912aea9fbd3cbd1 100755 (executable)
--- a/bsmtp-pull
+++ b/bsmtp-pull
@@ -7,7 +7,7 @@ if [ -z "$1" ]; then
     exit 1
 fi
 
-DIR="$HOME/.bsmtp"
+DIR="$HOME/tmp/.bsmtp"
 mkdir -p "$DIR"
 cd "$DIR"
 
@@ -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