Elliott Hughes | ad88a08 | 2012-10-24 18:37:21 -0700 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (C) 2012 The Android Open Source Project |
| 3 | * |
| 4 | * Licensed under the Apache License, Version 2.0 (the "License"); |
| 5 | * you may not use this file except in compliance with the License. |
| 6 | * You may obtain a copy of the License at |
| 7 | * |
| 8 | * http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | * |
| 10 | * Unless required by applicable law or agreed to in writing, software |
| 11 | * distributed under the License is distributed on an "AS IS" BASIS, |
| 12 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 13 | * See the License for the specific language governing permissions and |
| 14 | * limitations under the License. |
| 15 | */ |
| 16 | |
| 17 | /* |
| 18 | * Contributed by: Intel Corporation |
| 19 | */ |
| 20 | |
| 21 | #include <gtest/gtest.h> |
| 22 | |
| 23 | #include <pthread.h> |
| 24 | #include <stdint.h> |
| 25 | #include <stdio.h> |
Elliott Hughes | ad88a08 | 2012-10-24 18:37:21 -0700 | [diff] [blame] | 26 | #include <unistd.h> |
| 27 | #include <set> |
| 28 | |
Elliott Hughes | 141b917 | 2021-04-09 17:13:09 -0700 | [diff] [blame^] | 29 | #include <android-base/silent_death_test.h> |
| 30 | |
Elliott Hughes | fc69a8a | 2016-03-04 11:53:09 -0800 | [diff] [blame] | 31 | #include "private/bionic_tls.h" |
Elliott Hughes | ad88a08 | 2012-10-24 18:37:21 -0700 | [diff] [blame] | 32 | |
Elliott Hughes | fc69a8a | 2016-03-04 11:53:09 -0800 | [diff] [blame] | 33 | extern "C" pid_t gettid(); // glibc defines this but doesn't declare it anywhere. |
| 34 | |
| 35 | #if defined(__BIONIC__) |
| 36 | extern uintptr_t __stack_chk_guard; |
| 37 | #endif |
Elliott Hughes | ad88a08 | 2012-10-24 18:37:21 -0700 | [diff] [blame] | 38 | |
| 39 | struct stack_protector_checker { |
| 40 | std::set<pid_t> tids; |
Elliott Hughes | fc69a8a | 2016-03-04 11:53:09 -0800 | [diff] [blame] | 41 | std::set<void*> guards; |
Elliott Hughes | ad88a08 | 2012-10-24 18:37:21 -0700 | [diff] [blame] | 42 | |
| 43 | void Check() { |
| 44 | pid_t tid = gettid(); |
Elliott Hughes | fc69a8a | 2016-03-04 11:53:09 -0800 | [diff] [blame] | 45 | void* guard = __get_tls()[TLS_SLOT_STACK_GUARD]; |
Elliott Hughes | ad88a08 | 2012-10-24 18:37:21 -0700 | [diff] [blame] | 46 | |
Elliott Hughes | fc69a8a | 2016-03-04 11:53:09 -0800 | [diff] [blame] | 47 | printf("[thread %d] TLS stack guard = %p\n", tid, guard); |
Elliott Hughes | ad88a08 | 2012-10-24 18:37:21 -0700 | [diff] [blame] | 48 | |
| 49 | // Duplicate tid. gettid(2) bug? Seeing this would be very upsetting. |
| 50 | ASSERT_TRUE(tids.find(tid) == tids.end()); |
Elliott Hughes | d3920b3 | 2013-02-07 18:39:34 -0800 | [diff] [blame] | 51 | |
Elliott Hughes | fc69a8a | 2016-03-04 11:53:09 -0800 | [diff] [blame] | 52 | // Uninitialized guard. Our bug. Note this is potentially flaky; we _could_ |
| 53 | // get four random zero bytes, but it should be vanishingly unlikely. |
| 54 | ASSERT_NE(guard, nullptr); |
| 55 | |
| 56 | #if defined(__BIONIC__) |
| 57 | // bionic always has the global too. |
| 58 | ASSERT_EQ(__stack_chk_guard, reinterpret_cast<uintptr_t>(guard)); |
| 59 | #endif |
Elliott Hughes | ad88a08 | 2012-10-24 18:37:21 -0700 | [diff] [blame] | 60 | |
| 61 | tids.insert(tid); |
| 62 | guards.insert(guard); |
| 63 | } |
| 64 | }; |
| 65 | |
Elliott Hughes | d3920b3 | 2013-02-07 18:39:34 -0800 | [diff] [blame] | 66 | TEST(stack_protector, same_guard_per_thread) { |
Elliott Hughes | fc69a8a | 2016-03-04 11:53:09 -0800 | [diff] [blame] | 67 | // Everyone has the TLS slot set, even if their stack protector |
| 68 | // implementation doesn't yet use it. |
Elliott Hughes | ad88a08 | 2012-10-24 18:37:21 -0700 | [diff] [blame] | 69 | stack_protector_checker checker; |
Elliott Hughes | fc69a8a | 2016-03-04 11:53:09 -0800 | [diff] [blame] | 70 | |
| 71 | // Check the main thread. |
| 72 | ASSERT_EQ(getpid(), gettid()); // We are the main thread, right? |
| 73 | checker.Check(); |
| 74 | |
| 75 | size_t thread_count = 9; |
| 76 | for (size_t i = 1; i < thread_count; ++i) { |
Elliott Hughes | ad88a08 | 2012-10-24 18:37:21 -0700 | [diff] [blame] | 77 | pthread_t t; |
Yi Kong | 32bc0fc | 2018-08-02 17:31:13 -0700 | [diff] [blame] | 78 | ASSERT_EQ(0, pthread_create(&t, nullptr, [](void* arg) -> void* { |
Elliott Hughes | fc69a8a | 2016-03-04 11:53:09 -0800 | [diff] [blame] | 79 | stack_protector_checker* checker = reinterpret_cast<stack_protector_checker*>(arg); |
| 80 | checker->Check(); |
| 81 | return nullptr; |
| 82 | }, &checker)); |
Elliott Hughes | ad88a08 | 2012-10-24 18:37:21 -0700 | [diff] [blame] | 83 | void* result; |
| 84 | ASSERT_EQ(0, pthread_join(t, &result)); |
Yi Kong | 32bc0fc | 2018-08-02 17:31:13 -0700 | [diff] [blame] | 85 | ASSERT_EQ(nullptr, result); |
Elliott Hughes | ad88a08 | 2012-10-24 18:37:21 -0700 | [diff] [blame] | 86 | } |
| 87 | ASSERT_EQ(thread_count, checker.tids.size()); |
| 88 | |
Elliott Hughes | fc69a8a | 2016-03-04 11:53:09 -0800 | [diff] [blame] | 89 | // Both bionic and glibc use the same guard for every thread. |
Elliott Hughes | ad88a08 | 2012-10-24 18:37:21 -0700 | [diff] [blame] | 90 | ASSERT_EQ(1U, checker.guards.size()); |
Elliott Hughes | ad88a08 | 2012-10-24 18:37:21 -0700 | [diff] [blame] | 91 | } |
| 92 | |
Christopher Ferris | f04935c | 2013-12-20 18:43:21 -0800 | [diff] [blame] | 93 | TEST(stack_protector, global_guard) { |
Elliott Hughes | fc69a8a | 2016-03-04 11:53:09 -0800 | [diff] [blame] | 94 | #if defined(__BIONIC__) |
| 95 | // Bionic always has a global, even if it's using TLS. |
Christopher Ferris | f04935c | 2013-12-20 18:43:21 -0800 | [diff] [blame] | 96 | ASSERT_NE(0, gettid()); |
| 97 | ASSERT_NE(0U, __stack_chk_guard); |
Elliott Hughes | fc69a8a | 2016-03-04 11:53:09 -0800 | [diff] [blame] | 98 | #else |
Elliott Hughes | bcaa454 | 2019-03-08 15:20:23 -0800 | [diff] [blame] | 99 | GTEST_SKIP() << "glibc doesn't have a global __stack_chk_guard"; |
Elliott Hughes | fc69a8a | 2016-03-04 11:53:09 -0800 | [diff] [blame] | 100 | #endif |
Nick Kralevich | dcab1b2 | 2013-01-10 17:12:29 -0800 | [diff] [blame] | 101 | } |
| 102 | |
Elliott Hughes | 141b917 | 2021-04-09 17:13:09 -0700 | [diff] [blame^] | 103 | using stack_protector_DeathTest = SilentDeathTest; |
Yabin Cui | 9df7040 | 2014-11-05 18:01:01 -0800 | [diff] [blame] | 104 | |
| 105 | TEST_F(stack_protector_DeathTest, modify_stack_protector) { |
Elliott Hughes | fc69a8a | 2016-03-04 11:53:09 -0800 | [diff] [blame] | 106 | // In another file to prevent inlining, which removes stack protection. |
| 107 | extern void modify_stack_protector_test(); |
Evgenii Stepanov | 7cc6706 | 2019-02-05 18:43:34 -0800 | [diff] [blame] | 108 | #if __has_feature(hwaddress_sanitizer) |
| 109 | ASSERT_EXIT(modify_stack_protector_test(), |
| 110 | testing::KilledBySignal(SIGABRT), "tag-mismatch"); |
| 111 | #else |
Elliott Hughes | fc69a8a | 2016-03-04 11:53:09 -0800 | [diff] [blame] | 112 | ASSERT_EXIT(modify_stack_protector_test(), |
| 113 | testing::KilledBySignal(SIGABRT), "stack corruption detected"); |
Evgenii Stepanov | 7cc6706 | 2019-02-05 18:43:34 -0800 | [diff] [blame] | 114 | #endif |
Christopher Ferris | f04935c | 2013-12-20 18:43:21 -0800 | [diff] [blame] | 115 | } |