summaryrefslogtreecommitdiff
path: root/tests/stdio_test.cpp
diff options
context:
space:
mode:
authorElliott Hughes <enh@google.com>2016-02-05 21:57:37 -0800
committerElliott Hughes <enh@google.com>2016-02-05 21:57:37 -0800
commit8200e55d941305f6f24f1745492b90271aa417e9 (patch)
tree49a1fffff9cb70861415949cb8c83419fab2ae9d /tests/stdio_test.cpp
parent831c8a52498d3c7be204a00847275f6e9163626f (diff)
Add a test for a possible overflow in *printf.
It turns out we don't have any bugs here, but glibc does. Found while chasing down a toybox failure I saw on the host, but we may as well add the test in case we ever screw up here in future. Change-Id: Ib8dd227ed3b742dc4dab8c09dc08e6ea9a35c807
Diffstat (limited to 'tests/stdio_test.cpp')
-rw-r--r--tests/stdio_test.cpp15
1 files changed, 15 insertions, 0 deletions
diff --git a/tests/stdio_test.cpp b/tests/stdio_test.cpp
index 79a86ad2f..7e826124f 100644
--- a/tests/stdio_test.cpp
+++ b/tests/stdio_test.cpp
@@ -553,6 +553,21 @@ TEST(STDIO_TEST, snprintf_small_stack) {
ASSERT_EQ(0, pthread_join(t, nullptr));
}
+TEST(STDIO_TEST, snprintf_asterisk_overflow) {
+ char buf[128];
+ ASSERT_EQ(5, snprintf(buf, sizeof(buf), "%.*s%c", 4, "hello world", '!'));
+ ASSERT_EQ(12, snprintf(buf, sizeof(buf), "%.*s%c", INT_MAX/2, "hello world", '!'));
+ ASSERT_EQ(12, snprintf(buf, sizeof(buf), "%.*s%c", INT_MAX-1, "hello world", '!'));
+ ASSERT_EQ(12, snprintf(buf, sizeof(buf), "%.*s%c", INT_MAX, "hello world", '!'));
+ ASSERT_EQ(12, snprintf(buf, sizeof(buf), "%.*s%c", -1, "hello world", '!'));
+
+ // INT_MAX-1, INT_MAX, INT_MAX+1.
+ ASSERT_EQ(12, snprintf(buf, sizeof(buf), "%.2147483646s%c", "hello world", '!'));
+ ASSERT_EQ(12, snprintf(buf, sizeof(buf), "%.2147483647s%c", "hello world", '!'));
+ ASSERT_EQ(-1, snprintf(buf, sizeof(buf), "%.2147483648s%c", "hello world", '!'));
+ ASSERT_EQ(ENOMEM, errno);
+}
+
TEST(STDIO_TEST, fprintf_failures_7229520) {
// http://b/7229520
FILE* fp;