From ec45a47f935d60c709ac0959a94da1e8318a4d24 Mon Sep 17 00:00:00 2001 From: Vinothkumar Siddharth Date: Thu, 23 Mar 2017 10:17:42 -0700 Subject: [PATCH 1/2] Improve pause-container warning message Signed-off-by: Vinothkumar Siddharth --- build/pause/pause.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build/pause/pause.c b/build/pause/pause.c index f9853915f82..dd6cbea0850 100644 --- a/build/pause/pause.c +++ b/build/pause/pause.c @@ -34,7 +34,7 @@ static void sigreap(int signo) { int main() { if (getpid() != 1) /* Not an error because pause sees use outside of infra containers. */ - fprintf(stderr, "Warning: pause should be the first process in a pod\n"); + fprintf(stderr, "Warning: pause should be the first process\n"); if (sigaction(SIGINT, &(struct sigaction){.sa_handler = sigdown}, NULL) < 0) return 1; From 9fa5e857e574bfdbf40f2d996d619093a8dcdeaf Mon Sep 17 00:00:00 2001 From: Vinothkumar Siddharth Date: Wed, 17 May 2017 10:34:20 -0700 Subject: [PATCH 2/2] Fix file semantics Signed-off-by: Vinothkumar Siddharth --- build/pause/pause.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/build/pause/pause.c b/build/pause/pause.c index dd6cbea0850..f2e21b41d55 100644 --- a/build/pause/pause.c +++ b/build/pause/pause.c @@ -27,8 +27,7 @@ static void sigdown(int signo) { } static void sigreap(int signo) { - while (waitpid(-1, NULL, WNOHANG) > 0) - ; + while (waitpid(-1, NULL, WNOHANG) > 0); } int main() {