diff options
| author | Marcos Pividori <mpividori@google.com> | 2017-01-30 18:23:37 +0000 |
|---|---|---|
| committer | Marcos Pividori <mpividori@google.com> | 2017-01-30 18:23:37 +0000 |
| commit | 7ac943c46344ef4c40096fc161ec913c733cdffc (patch) | |
| tree | f307a9dd92837b0b3f3ce81f8badb7db33bc987c /compiler-rt/lib/interception/tests | |
| parent | 0a5ab5c4dbd9435fe0c435ca2d1116435535f2be (diff) | |
| download | bcm5719-llvm-7ac943c46344ef4c40096fc161ec913c733cdffc.tar.gz bcm5719-llvm-7ac943c46344ef4c40096fc161ec913c733cdffc.zip | |
[interception] Check for export table's size before referring to its elements.
This fix a bug, when calling InternalGetProcAddress() for an executable that
doesn't export any symbol. So the table is empty.
If we don't check for this condition, the program fails with Error 0xc0000142.
Also, I add a regression test for Windows.
Differential Revision: https://reviews.llvm.org/D28502
llvm-svn: 293521
Diffstat (limited to 'compiler-rt/lib/interception/tests')
| -rw-r--r-- | compiler-rt/lib/interception/tests/interception_win_test.cc | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/compiler-rt/lib/interception/tests/interception_win_test.cc b/compiler-rt/lib/interception/tests/interception_win_test.cc index 684ee030355..a705768d6d5 100644 --- a/compiler-rt/lib/interception/tests/interception_win_test.cc +++ b/compiler-rt/lib/interception/tests/interception_win_test.cc @@ -613,6 +613,13 @@ TEST(Interception, PatchableFunctionPadding) { EXPECT_FALSE(TestFunctionPatching(kUnpatchableCode6, override, prefix)); } +TEST(Interception, EmptyExportTable) { + // We try to get a pointer to a function from an executable that doesn't + // export any symbol (empty export table). + uptr FunPtr = InternalGetProcAddress((void *)GetModuleHandleA(0), "example"); + EXPECT_EQ(0U, FunPtr); +} + } // namespace __interception #endif // SANITIZER_WINDOWS |

