[openssl] master update
Richard Levitte
levitte at openssl.org
Mon Jan 13 08:03:27 UTC 2020
The branch master has been updated
via bbe486cf6154df3d3aaedbae6c5b82d4ed31a5f8 (commit)
from 42b3f10b5e461496aab1f74d24103d6902ebfcd5 (commit)
- Log -----------------------------------------------------------------
commit bbe486cf6154df3d3aaedbae6c5b82d4ed31a5f8
Author: Richard Levitte <levitte at openssl.org>
Date: Thu Jan 9 14:49:53 2020 +0100
Configure: use $list_separator_re only for defines and includes
This regexp was used a bit too uncontrolled, which had it split flag
values where it should not have.
Fixes #10792
Reviewed-by: Paul Dale <paul.dale at oracle.com>
(Merged from https://github.com/openssl/openssl/pull/10793)
-----------------------------------------------------------------------
Summary of changes:
Configure | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/Configure b/Configure
index 8ee0fbdb25..1e86bfa198 100755
--- a/Configure
+++ b/Configure
@@ -590,7 +590,7 @@ while ((my $first, my $second) = (shift @list, shift @list)) {
&usage if ($#ARGV < 0);
-# For the "make variables" CINCLUDES and CDEFINES, we support lists with
+# For the "make variables" CPPINCLUDES and CPPDEFINES, we support lists with
# platform specific list separators. Users from those platforms should
# recognise those separators from how you set up the PATH to find executables.
# The default is the Unix like separator, :, but as an exception, we also
@@ -1030,7 +1030,11 @@ foreach (keys %user) {
if (defined $value) {
if (ref $user{$_} eq 'ARRAY') {
- $user{$_} = [ split /$list_separator_re/, $value ];
+ if ($_ eq 'CPPDEFINES' || $_ eq 'CPPINCLUDES') {
+ $user{$_} = [ split /$list_separator_re/, $value ];
+ } else {
+ $user{$_} = [ $value ];
+ }
} elsif (!defined $user{$_}) {
$user{$_} = $value;
}
More information about the openssl-commits
mailing list