aboutsummaryrefslogtreecommitdiffstats
path: root/tests/fortify_test.cpp
diff options
context:
space:
mode:
authorNick Kralevich <nnk@google.com>2013-10-09 20:16:34 -0700
committerNick Kralevich <nnk@google.com>2013-10-09 20:17:03 -0700
commitb036b5ca36c1e12b075909b3eca6eab73ee611cf (patch)
treec3c2056a4e3868618d473047b2519aa91c3233c2 /tests/fortify_test.cpp
parent92f3cc50c8d884d1eb3496ebfba7d3916fc711e8 (diff)
downloadandroid_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.cpp10
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);