[lkml]   [2017]   [Aug]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH 4/4] selftests/seccomp: Test thread vs process killing
On Mon, Aug 7, 2017 at 6:29 PM, Tyler Hicks <> wrote:
>> + /* Only the thread died. Let parent know this thread didn't die. */
> This read a little odd to me. How about, "Only the created thread died.
> Let parent know the this creating thread didn't die."?

Sounds good. I've updated this to be more descriptive.

>> + ASSERT_EQ(1, WIFEXITED(status));
> This is probably nitpicky but, after reading the wait(2) man page, I
> feel like this should be ASSERT_TRUE(WIFEXITED(status)) instead of
> comparing to 1. There's no documented guarantee that 1 will be returned.

That's a fair point. I've updated this and WIFSIGNALED now, thanks!


Kees Cook
Pixel Security

 \ /
  Last update: 2017-08-08 03:37    [W:0.046 / U:2.552 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site