1998-09-28 Ben Elliston <bje@cygnus.com>

	* acgeneral.m4 (AC_CHECK_PROG): Fix a bug if the supplied path
	contains colons. This was observed with some versions of NetBSD
	`sh' and some versions of `bash'.
	(AC_PATH_PROG): Likewise. Contributed by Tom Yu <tlyu@mit.edu>.

diff --git a/ChangeLog b/ChangeLog
index f5e012a02c3e6677caf682ffdaab42a5e7767b83..6641a7b57e2ee5944086255e675b6e4d53b3726a 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+1998-09-28  Ben Elliston  <bje@cygnus.com>
+
+       * acgeneral.m4 (AC_CHECK_PROG): Fix a bug if the supplied path
+       contains colons. This was observed with some versions of NetBSD
+       `sh' and some versions of `bash'.
+       (AC_PATH_PROG): Likewise. Contributed by Tom Yu <tlyu@mit.edu>.
+
 1998-09-27  Ben Elliston  <bje@cygnus.com>
 
        * Makefile.in (all): Generate frozen .m4 files at build time.

diff --git a/acgeneral.m4 b/acgeneral.m4
index 6af89acfb36424d78857907e25d04e7ce95fb6fc..d722ff2a17c56cb9efd0f8bc4fad0777687befc6 100644 (file)
--- a/acgeneral.m4
+++ b/acgeneral.m4
@@ -1288,9 +1288,10 @@ else
 ifelse([$6], , , [  ac_prog_rejected=no
 ])dnl
 dnl $ac_dummy forces splitting on constant user-supplied paths.
-dnl bash word splitting is done only on the output of word expansions,
+dnl POSIX.2 word splitting is done only on the output of word expansions,
 dnl not every word.  This closes a longstanding sh security hole.
-  for ac_dir in ifelse([$5], , $PATH, [$5$ac_dummy]); do
+  ac_dummy="ifelse([$5], , $PATH, [$5])"
+  for ac_dir in $ac_dummy; do
     test -z "$ac_dir" && ac_dir=.
     if test -f $ac_dir/$ac_word; then
 ifelse([$6], , , dnl
@@ -1355,9 +1356,10 @@ AC_CACHE_VAL(ac_cv_path_$1,
   *)
   IFS="${IFS=  }"; ac_save_ifs="$IFS"; IFS=":"
 dnl $ac_dummy forces splitting on constant user-supplied paths.
-dnl bash word splitting is done only on the output of word expansions,
+dnl POSIX.2 word splitting is done only on the output of word expansions,
 dnl not every word.  This closes a longstanding sh security hole.
-  for ac_dir in ifelse([$4], , $PATH, [$4$ac_dummy]); do
+  ac_dummy="ifelse([$4], , $PATH, [$4])"
+  for ac_dir in $ac_dummy; do 
     test -z "$ac_dir" && ac_dir=.
     if test -f $ac_dir/$ac_word; then
       ac_cv_path_$1="$ac_dir/$ac_word"

diff --git a/lib/autoconf/general.m4 b/lib/autoconf/general.m4
index 6af89acfb36424d78857907e25d04e7ce95fb6fc..d722ff2a17c56cb9efd0f8bc4fad0777687befc6 100644 (file)
--- a/lib/autoconf/general.m4
+++ b/lib/autoconf/general.m4
@@ -1288,9 +1288,10 @@ else
 ifelse([$6], , , [  ac_prog_rejected=no
 ])dnl
 dnl $ac_dummy forces splitting on constant user-supplied paths.
-dnl bash word splitting is done only on the output of word expansions,
+dnl POSIX.2 word splitting is done only on the output of word expansions,
 dnl not every word.  This closes a longstanding sh security hole.
-  for ac_dir in ifelse([$5], , $PATH, [$5$ac_dummy]); do
+  ac_dummy="ifelse([$5], , $PATH, [$5])"
+  for ac_dir in $ac_dummy; do
     test -z "$ac_dir" && ac_dir=.
     if test -f $ac_dir/$ac_word; then
 ifelse([$6], , , dnl
@@ -1355,9 +1356,10 @@ AC_CACHE_VAL(ac_cv_path_$1,
   *)
   IFS="${IFS=  }"; ac_save_ifs="$IFS"; IFS=":"
 dnl $ac_dummy forces splitting on constant user-supplied paths.
-dnl bash word splitting is done only on the output of word expansions,
+dnl POSIX.2 word splitting is done only on the output of word expansions,
 dnl not every word.  This closes a longstanding sh security hole.
-  for ac_dir in ifelse([$4], , $PATH, [$4$ac_dummy]); do
+  ac_dummy="ifelse([$4], , $PATH, [$4])"
+  for ac_dir in $ac_dummy; do 
     test -z "$ac_dir" && ac_dir=.
     if test -f $ac_dir/$ac_word; then
       ac_cv_path_$1="$ac_dir/$ac_word"