From 86140c6223b5d14d773cf3050ffd0d14977c2c2d Mon Sep 17 00:00:00 2001 From: Joseph Myers Date: Wed, 13 Feb 2019 13:34:24 +0000 Subject: Avoid fall-through in test-container if execlp fails. One of the implicit-fallthrough warnings from compiling glibc with -Wextra appears to indicate an actual bug: the test-container code could fall through inappropriately if execlp returns (which only occurs on error). This patch adds appropriate error handling in this case to avoid that fall-through. Tested for x86_64. * support/test-container.c (recursive_remove): Use FAIL_EXIT1 if execlp returns. --- support/test-container.c | 1 + 1 file changed, 1 insertion(+) (limited to 'support') diff --git a/support/test-container.c b/support/test-container.c index a24b24c03b..ed911e33f9 100644 --- a/support/test-container.c +++ b/support/test-container.c @@ -361,6 +361,7 @@ recursive_remove (char *path) case 0: /* Child. */ execlp ("rm", "rm", "-rf", path, NULL); + FAIL_EXIT1 ("exec rm: %m"); default: /* Parent. */ waitpid (child, &status, 0); -- cgit v1.2.3