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

Cleanup ftruncate negative test.

Browse source 
Addresses post-submit comments from
I54692ab8105dd09db6af7a2c0894a17bdd118aa0.

Bug: http://b/21309901
Change-Id: Ie19ee8cdcc4055a65fe7dfc103156e54eafe8977
This commit is contained in:
Dan Albert 2015-06-01 11:28:31 -07:00
parent d70891687d
commit 9d476a02e9
2 changed files with 7 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
libc/bionic/ftruncate.cpp
View file

@ -18,11 +18,14 @@
#include <sys/cdefs.h>
#include <unistd.h>
#if !defined(__USE_FILE_OFFSET64) && !defined(__LP64__)
#if !defined(__LP64__)
static_assert(sizeof(off_t) == 4,
"libc can't be built with _FILE_OFFSET_BITS=64.");
// The kernel's implementation of ftruncate uses an unsigned long for the length
// parameter, so it will not catch negative values. On the other hand
// ftruncate64 does check for this, so just forward the call.
int ftruncate(int filedes, off_t length) {
return ftruncate64(filedes, length);
}
#endif
#endif // !defined(__LP64__)

6
tests/unistd_test.cpp
View file

@ -179,10 +179,8 @@ TEST(unistd, ftruncate64) {
TEST(unistd, ftruncate_negative) {
TemporaryFile tf;
errno = 0;
int rc = ftruncate(tf.fd, -123);
int err = errno;
ASSERT_EQ(-1, rc);
ASSERT_EQ(EINVAL, err);
ASSERT_EQ(-1, ftruncate(tf.fd, -123));
ASSERT_EQ(EINVAL, errno);
}
static bool g_pause_test_flag = false;