From: Dmitry V. Levin <ldv@altlinux.org>
Date: Tue, 26 Jul 2016 10:20:24 +0000 (+0000)
Subject: tests: fix vhangup.test
X-Git-Tag: v4.13~5
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=cb44289ee3fd1482d3b8551d2e3b4568eb57e114;p=strace

tests: fix vhangup.test

* tests/vhangup.c (main): Fix expected output for the case when
the calling process has CAP_SYS_TTY_CONFIG capability.

Reported-by: Yun-Chih Chen <b03902074@ntu.edu.tw>
---

diff --git a/tests/vhangup.c b/tests/vhangup.c
index ba903190..468f3344 100644
--- a/tests/vhangup.c
+++ b/tests/vhangup.c
@@ -13,7 +13,18 @@ main(void)
 		perror_msg_and_skip("setsid");
 
 	long rc = syscall(__NR_vhangup);
-	printf("vhangup() = %ld %s (%m)\n", rc, errno2name());
+
+	/*
+	 * On setsid() success, the new session has no controlling terminal,
+	 * therefore a subsequent vhangup() has nothing to hangup.
+	 *
+	 * The system call, however, returns 0 iff the calling process
+	 * has CAP_SYS_TTY_CONFIG capability.
+	 */
+	if (rc)
+		printf("vhangup() = %ld %s (%m)\n", rc, errno2name());
+	else
+		puts("vhangup() = 0");
 
 	puts("+++ exited with 0 +++");
 	return 0;