depcomp: safer quoting in variable expansion
authorStefano Lattarini <stefano.lattarini@gmail.com>
Thu, 18 Oct 2012 11:28:54 +0000 (13:28 +0200)
committerStefano Lattarini <stefano.lattarini@gmail.com>
Sat, 24 Nov 2012 14:52:41 +0000 (15:52 +0100)
* lib/depcomp (pgcc): Here.  Probably not needed, but since the
rest of the script seems to employ proper quoting, better to be
consistent.

Signed-off-by: Stefano Lattarini <stefano.lattarini@gmail.com>
lib/depcomp

index fb818b0..af4b18c 100755 (executable)
@@ -365,24 +365,24 @@ pgcc)
   lockdir=$base.d-lock
   trap "
     echo '$0: caught signal, cleaning up...' >&2
-    rmdir $lockdir
+    rmdir '$lockdir'
     exit 1
   " 1 2 13 15
   numtries=100
   i=$numtries
   while test $i -gt 0 ; do
     # mkdir is a portable test-and-set.
-    if mkdir $lockdir 2>/dev/null; then
+    if mkdir "$lockdir" 2>/dev/null; then
       # This process acquired the lock.
       "$@" -MD
       stat=$?
       # Release the lock.
-      rmdir $lockdir
+      rmdir "$lockdir"
       break
     else
       ## the lock is being held by a different process,
       ## wait until the winning process is done or we timeout
-      while test -d $lockdir && test $i -gt 0; do
+      while test -d "$lockdir" && test $i -gt 0; do
         sleep 1
         i=`expr $i - 1`
       done