diff options
| author | Brad King <brad.king@kitware.com> | 2016-02-01 14:49:08 (GMT) |
|---|---|---|
| committer | Brad King <brad.king@kitware.com> | 2016-02-01 15:05:10 (GMT) |
| commit | 6ffc4323670f3671f262b3e9f035f1ea3f714986 (patch) | |
| tree | 4058de83fa61e55afe5b841534f4b268a97a758e /Tests | |
| parent | 656768cffe981e02f12fe51d6723d21176a46329 (diff) | |
| download | CMake-6ffc4323670f3671f262b3e9f035f1ea3f714986.zip CMake-6ffc4323670f3671f262b3e9f035f1ea3f714986.tar.gz CMake-6ffc4323670f3671f262b3e9f035f1ea3f714986.tar.bz2 | |
cmConditionEvaluator: Fix matching of `CMAKE_MATCH_*` values (#15944)
While evaluating `if(MATCHES)` we get a `const char*` pointer to the
string to be matched. On code like
if(CMAKE_MATCH_COUNT MATCHES "Y")
the string to be matched may be owned by our own result variables.
We must move the value to our own buffer before clearing them.
Otherwise we risk reading freed storage.
Diffstat (limited to 'Tests')
| -rw-r--r-- | Tests/RunCMake/if/MatchesSelf.cmake | 4 | ||||
| -rw-r--r-- | Tests/RunCMake/if/RunCMakeTest.cmake | 2 |
2 files changed, 6 insertions, 0 deletions
diff --git a/Tests/RunCMake/if/MatchesSelf.cmake b/Tests/RunCMake/if/MatchesSelf.cmake new file mode 100644 index 0000000..3131ac4 --- /dev/null +++ b/Tests/RunCMake/if/MatchesSelf.cmake @@ -0,0 +1,4 @@ +foreach(n 0 1 2 3 4 5 6 7 8 9 COUNT) + if(CMAKE_MATCH_${n} MATCHES "x") + endif() +endforeach() diff --git a/Tests/RunCMake/if/RunCMakeTest.cmake b/Tests/RunCMake/if/RunCMakeTest.cmake index 3f4d2a2..077d00a 100644 --- a/Tests/RunCMake/if/RunCMakeTest.cmake +++ b/Tests/RunCMake/if/RunCMakeTest.cmake @@ -5,5 +5,7 @@ run_cmake(IsDirectory) run_cmake(IsDirectoryLong) run_cmake(elseif-message) +run_cmake(MatchesSelf) + run_cmake(TestNameThatExists) run_cmake(TestNameThatDoesNotExist) |