generic-library/bionic
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

am 14d02ac7: am 2b18b107: Merge "Fix use-after-free errors in stdio_test."

Browse Source 
* commit '14d02ac74cd3cd1cb22e7d1b12a555e38df2402a':
  Fix use-after-free errors in stdio_test.
This commit is contained in:
Elliott Hughes 2014-05-12 22:39:25 +00:00 committed by Android Git Automerger
commit 04211b2c47
1 changed files with 6 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
tests/stdio_test.cpp
View File

@ -102,14 +102,12 @@ TEST(stdio, getdelim_invalid) {
ASSERT_EQ(getdelim(&buffer, NULL, ' ', fp), -1);
ASSERT_EQ(EINVAL, errno);
// The stream can't be closed.
fclose(fp);
// The underlying fd can't be closed.
ASSERT_EQ(0, close(fileno(fp)));
errno = 0;
ASSERT_EQ(getdelim(&buffer, &buffer_length, ' ', fp), -1);
// glibc sometimes doesn't set errno in this particular case.
#if defined(__BIONIC__)
ASSERT_EQ(EBADF, errno);
#endif // __BIONIC__
fclose(fp);
}
TEST(stdio, getline) {
@ -171,14 +169,12 @@ TEST(stdio, getline_invalid) {
ASSERT_EQ(getline(&buffer, NULL, fp), -1);
ASSERT_EQ(EINVAL, errno);
// The stream can't be closed.
fclose(fp);
// The underlying fd can't be closed.
ASSERT_EQ(0, close(fileno(fp)));
errno = 0;
ASSERT_EQ(getline(&buffer, &buffer_length, fp), -1);
// glibc sometimes doesn't set errno in this particular case.
#if defined(__BIONIC__)
ASSERT_EQ(EBADF, errno);
#endif // __BIONIC__
fclose(fp);
}
TEST(stdio, printf_ssize_t) {