svn commit: r351469 - head/tests/sys/posixshm

Kyle Evans kevans at FreeBSD.org
Sun Aug 25 02:38:46 UTC 2019


Author: kevans
Date: Sun Aug 25 02:38:45 2019
New Revision: 351469
URL: https://svnweb.freebsd.org/changeset/base/351469

Log:
  tests: shm_open(2): Verify FD_CLOEXEC
  
  Motivated by the fact that I'm messing around near the implementation and
  wanting to ensure this doesn't get messed up in the process.
  
  MFC after:	1 week

Modified:
  head/tests/sys/posixshm/posixshm_test.c

Modified: head/tests/sys/posixshm/posixshm_test.c
==============================================================================
--- head/tests/sys/posixshm/posixshm_test.c	Sun Aug 25 01:09:31 2019	(r351468)
+++ head/tests/sys/posixshm/posixshm_test.c	Sun Aug 25 02:38:45 2019	(r351469)
@@ -609,6 +609,27 @@ ATF_TC_BODY(shm_functionality_across_fork, tc)
 	shm_unlink(test_path);
 }
 
+ATF_TC_WITHOUT_HEAD(cloexec);
+ATF_TC_BODY(cloexec, tc)
+{
+	int fd;
+
+	gen_test_path();
+
+	/* shm_open(2) is required to set FD_CLOEXEC */
+	fd = shm_open(SHM_ANON, O_RDWR, 0777);
+	ATF_REQUIRE_MSG(fd >= 0, "shm_open failed; errno=%d", errno);
+	ATF_REQUIRE((fcntl(fd, F_GETFD) & FD_CLOEXEC) != 0);
+	close(fd);
+
+	/* Also make sure that named shm is correct */
+	fd = shm_open(test_path, O_CREAT | O_RDWR, 0600);
+	ATF_REQUIRE_MSG(fd >= 0, "shm_open failed; errno=%d", errno);
+	ATF_REQUIRE((fcntl(fd, F_GETFD) & FD_CLOEXEC) != 0);
+	close(fd);
+}
+
+
 ATF_TP_ADD_TCS(tp)
 {
 
@@ -630,6 +651,7 @@ ATF_TP_ADD_TCS(tp)
 	ATF_TP_ADD_TC(tp, unlink_bad_path_pointer);
 	ATF_TP_ADD_TC(tp, unlink_path_too_long);
 	ATF_TP_ADD_TC(tp, object_resize);
+	ATF_TP_ADD_TC(tp, cloexec);
 
 	return (atf_no_error());
 }


More information about the svn-src-all mailing list