diff options
author | Fabricio Voznika <fvoznika@google.com> | 2019-02-25 12:16:44 -0800 |
---|---|---|
committer | Shentubot <shentubot@google.com> | 2019-02-25 12:17:46 -0800 |
commit | 10426e0f31e427e90e69fee83f199ea521b8fe3d (patch) | |
tree | 377e4d2f38e196bcf26ee5276e3471e9c0c4da40 /test | |
parent | c14a1a161869804af5ae2f1f73fd8eeb135ff043 (diff) |
Handle invalid offset in sendfile(2)
PiperOrigin-RevId: 235578698
Change-Id: I608ff5e25eac97f6e1bda058511c1f82b0e3b736
Diffstat (limited to 'test')
-rw-r--r-- | test/syscalls/linux/sendfile.cc | 19 |
1 files changed, 19 insertions, 0 deletions
diff --git a/test/syscalls/linux/sendfile.cc b/test/syscalls/linux/sendfile.cc index 92b7b9478..15fd01ff0 100644 --- a/test/syscalls/linux/sendfile.cc +++ b/test/syscalls/linux/sendfile.cc @@ -46,6 +46,25 @@ TEST(SendFileTest, SendZeroBytes) { SyscallSucceedsWithValue(0)); } +TEST(SendFileTest, InvalidOffset) { + // Create temp files. + const TempPath in_file = ASSERT_NO_ERRNO_AND_VALUE(TempPath::CreateFile()); + const TempPath out_file = ASSERT_NO_ERRNO_AND_VALUE(TempPath::CreateFile()); + + // Open the input file as read only. + const FileDescriptor inf = + ASSERT_NO_ERRNO_AND_VALUE(Open(in_file.path(), O_RDONLY)); + + // Open the output file as write only. + const FileDescriptor outf = + ASSERT_NO_ERRNO_AND_VALUE(Open(out_file.path(), O_WRONLY)); + + // Send data and verify that sendfile returns the correct value. + off_t offset = -1; + EXPECT_THAT(sendfile(outf.get(), inf.get(), &offset, 0), + SyscallFailsWithErrno(EINVAL)); +} + TEST(SendFileTest, SendTrivially) { // Create temp files. constexpr char kData[] = "To be, or not to be, that is the question:"; |