diff options
author | Richard Levitte <levitte@openssl.org> | 2020-01-09 14:49:53 +0100 |
---|---|---|
committer | Richard Levitte <levitte@openssl.org> | 2020-01-13 09:02:16 +0100 |
commit | bbe486cf6154df3d3aaedbae6c5b82d4ed31a5f8 (patch) | |
tree | 299417f2fcfdbe9f8860f2cb6e4911bb2e1f31b3 /Configure | |
parent | When deprecated symbols are removed, ensure liblegacy has WHIRLPOOL (diff) | |
download | openssl-bbe486cf6154df3d3aaedbae6c5b82d4ed31a5f8.tar.xz openssl-bbe486cf6154df3d3aaedbae6c5b82d4ed31a5f8.zip |
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@oracle.com>
(Merged from https://github.com/openssl/openssl/pull/10793)
Diffstat (limited to 'Configure')
-rwxr-xr-x | Configure | 8 |
1 files changed, 6 insertions, 2 deletions
@@ -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; } |