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());
 }
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to