In the unlikely but possible case where LOCKDIR contained files _other_ than
the PIDFILE being tested for, rmdir wouldn't remove it, causing the lock to
persist and then rapidly be tested over and over as the script respawns.

Now uses rm -rf to zap the LOCKDIR (as is done at other points in the code).
---
 yum-cron/yum-cron.sh |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/yum-cron/yum-cron.sh b/yum-cron/yum-cron.sh
index de1608e..d549471 100755
--- a/yum-cron/yum-cron.sh
+++ b/yum-cron/yum-cron.sh
@@ -71,7 +71,7 @@ else
   # happened.  We can't know the process name, so, clean up the old lockdir
   # and restart.
   if [[ ! -f $PIDFILE ]]; then
-    rmdir $LOCKDIR 2>/dev/null
+    rm -rf "${LOCKDIR}"
     echo "yum-cron: no lock PID, clearing and restarting myself" >&2
     exec $0 "$@"
   fi
-- 
1.7.6

_______________________________________________
Yum-devel mailing list
Yum-devel@lists.baseurl.org
http://lists.baseurl.org/mailman/listinfo/yum-devel

Reply via email to