diff options
author | Nick Kralevich <nnk@google.com> | 2013-10-09 20:16:34 -0700 |
---|---|---|
committer | Nick Kralevich <nnk@google.com> | 2013-10-09 20:17:03 -0700 |
commit | b036b5ca36c1e12b075909b3eca6eab73ee611cf (patch) | |
tree | c3c2056a4e3868618d473047b2519aa91c3233c2 /tests/fortify_test.cpp | |
parent | 92f3cc50c8d884d1eb3496ebfba7d3916fc711e8 (diff) | |
download | android_bionic-b036b5ca36c1e12b075909b3eca6eab73ee611cf.tar.gz android_bionic-b036b5ca36c1e12b075909b3eca6eab73ee611cf.tar.bz2 android_bionic-b036b5ca36c1e12b075909b3eca6eab73ee611cf.zip |
FORTIFY_SOURCE: fortify read()
Change-Id: Ic7de163fe121db13e00560adb257331bc709814d
Diffstat (limited to 'tests/fortify_test.cpp')
-rw-r--r-- | tests/fortify_test.cpp | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/tests/fortify_test.cpp b/tests/fortify_test.cpp index d514a3d10..b42c6b781 100644 --- a/tests/fortify_test.cpp +++ b/tests/fortify_test.cpp @@ -21,6 +21,7 @@ #include <sys/stat.h> #include <sys/socket.h> #include <malloc.h> +#include <fcntl.h> // We have to say "DeathTest" here so gtest knows to run this test (which exits) // in its own process. Unfortunately, the C preprocessor doesn't give us an @@ -568,6 +569,15 @@ TEST(DEATHTEST, FD_ZERO_fortified) { ASSERT_EXIT(FD_ZERO(set), testing::KilledBySignal(SIGABRT), ""); } +TEST(DEATHTEST, read_fortified) { + ::testing::FLAGS_gtest_death_test_style = "threadsafe"; + char buf[1]; + size_t ct = atoi("2"); // prevent optimizations + int fd = open("/dev/null", O_RDONLY); + ASSERT_EXIT(read(fd, buf, ct), testing::KilledBySignal(SIGABRT), ""); + close(fd); +} + extern "C" char* __strncat_chk(char*, const char*, size_t, size_t); extern "C" char* __strcat_chk(char*, const char*, size_t); |