tequilaOS/platform_bionic
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Merge "libc fortify: error on realpath(NULL, foo)" am: 6beab08440

Browse source 
am: 87fcfefe26

Change-Id: I289b496e80533d3fe45781f9b86d000e25c5dfbd
This commit is contained in:
George Burgess IV 2017-08-15 17:50:31 +00:00 committed by android-build-merger
commit ff69b4fefc
3 changed files with 8 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
libc/include/bits/fortify/stdlib.h
View file

@ -40,7 +40,8 @@
#if defined(__clang__)
char* realpath(const char* path, char* resolved)
__clang_error_if(__bos(resolved) != __BIONIC_FORTIFY_UNKNOWN_SIZE &&
__bos(resolved) < __PATH_MAX, __realpath_buf_too_small_str);
__bos(resolved) < __PATH_MAX, __realpath_buf_too_small_str)
__clang_error_if(!path, "'realpath': NULL path is never correct; flipped arguments?");
/* No need for a definition; the only issues we can catch are at compile-time. */
#else /* defined(__clang__) */

5
tests/fortify_compilation_test.cpp
View file

@ -376,6 +376,7 @@ void test_realpath() {
// This is fine.
realpath(".", NULL);
// FIXME: But we should warn on this.
realpath(NULL, buf);
char bigbuf[PATH_MAX];
// CLANG: error: 'realpath': NULL path is never correct; flipped arguments?
realpath(NULL, bigbuf);
}

4
tests/stdlib_test.cpp
View file

@ -184,7 +184,9 @@ TEST(stdlib, posix_memalign_overflow) {
TEST(stdlib, realpath__NULL_filename) {
errno = 0;
char* p = realpath(NULL, NULL);
// Work around the compile-time error generated by FORTIFY here.
const char* path = NULL;
char* p = realpath(path, NULL);
ASSERT_TRUE(p == NULL);
ASSERT_EQ(EINVAL, errno);
}