diff options
author | Elliott Hughes <enh@google.com> | 2017-11-28 12:55:19 -0800 |
---|---|---|
committer | Elliott Hughes <enh@google.com> | 2017-11-28 12:55:19 -0800 |
commit | cabc77f9172d74ff0bfc56d0797a6a8255b14f2a (patch) | |
tree | 48d55bb7530c8fdf5ae0b72d65236d5c480def44 /libc/malloc_debug/tests/malloc_debug_unit_tests.cpp | |
parent | 7d5777ef15136e4e12a27c85d7b06d6806b98e70 (diff) |
Always wrap waitpid in TEMP_FAILURE_RETRY.
Strictly not needed in the WNOHANG case, but it's probably best to have
every waitpid wrapped for future copy & pasters.
Bug: https://issuetracker.google.com/69525592
Test: ran tests
Change-Id: I013b0a52d2753e3d32638e9b84c79af7327fb405
Diffstat (limited to 'libc/malloc_debug/tests/malloc_debug_unit_tests.cpp')
-rw-r--r-- | libc/malloc_debug/tests/malloc_debug_unit_tests.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libc/malloc_debug/tests/malloc_debug_unit_tests.cpp b/libc/malloc_debug/tests/malloc_debug_unit_tests.cpp index d7ba379c9..4e9066812 100644 --- a/libc/malloc_debug/tests/malloc_debug_unit_tests.cpp +++ b/libc/malloc_debug/tests/malloc_debug_unit_tests.cpp @@ -1385,7 +1385,7 @@ TEST_F(MallocDebugTest, backtrace_dump_on_exit) { exit(0); } ASSERT_NE(-1, pid); - ASSERT_EQ(pid, waitpid(pid, nullptr, 0)); + ASSERT_EQ(pid, TEMP_FAILURE_RETRY(waitpid(pid, nullptr, 0))); // Read all of the contents. std::string actual; |