Page MenuHomeFreeBSD
Files F143618152

D6012.diff
No OneTemporary
Actions

D6012.diff
View Options

Index: tests/sys/posixshm/posixshm_test.c
===================================================================
--- tests/sys/posixshm/posixshm_test.c
+++ tests/sys/posixshm/posixshm_test.c
@@ -50,12 +50,9 @@
static void
gen_test_path(void)
{
- char *tmpdir = getenv("TMPDIR");
- if (tmpdir == NULL)
- tmpdir = "/tmp";
-
- snprintf(test_path, sizeof(test_path), "%s/tmp.XXXXXX", tmpdir);
+ snprintf(test_path, sizeof(test_path), "%s/tmp.XXXXXX",
+ getenv("TMPDIR") == NULL ? "/tmp" : getenv("TMPDIR"));
test_path[sizeof(test_path) - 1] = '\0';
ATF_REQUIRE_MSG(mkstemp(test_path) != -1,
"mkstemp failed; errno=%d", errno);
@@ -99,10 +96,12 @@
scribble_object(void)
{
char *page;
- int fd;
+ int fd, pagesize;
gen_test_path();
+ ATF_REQUIRE(0 < (pagesize = getpagesize()));
+
fd = shm_open(test_path, O_CREAT|O_EXCL|O_RDWR, 0777);
if (fd < 0 && errno == EEXIST) {
if (shm_unlink(test_path) < 0)
@@ -111,16 +110,15 @@
}
if (fd < 0)
atf_tc_fail("shm_open failed; errno=%d", errno);
- if (ftruncate(fd, getpagesize()) < 0)
+ if (ftruncate(fd, pagesize) < 0)
atf_tc_fail("ftruncate failed; errno=%d", errno);
- page = mmap(0, getpagesize(), PROT_READ | PROT_WRITE, MAP_SHARED, fd,
- 0);
+ page = mmap(0, pagesize, PROT_READ | PROT_WRITE, MAP_SHARED, fd, 0);
if (page == MAP_FAILED)
atf_tc_fail("mmap failed; errno=%d", errno);
page[0] = '1';
- if (munmap(page, getpagesize()) < 0)
+ if (munmap(page, pagesize) < 0)
atf_tc_fail("munmap failed; errno=%d", errno);
return (fd);
@@ -130,12 +128,13 @@
ATF_TC_BODY(remap_object, tc)
{
char *page;
- int fd;
+ int fd, pagesize;
+
+ ATF_REQUIRE(0 < (pagesize = getpagesize()));
fd = scribble_object();
- page = mmap(0, getpagesize(), PROT_READ | PROT_WRITE, MAP_SHARED, fd,
- 0);
+ page = mmap(0, pagesize, PROT_READ | PROT_WRITE, MAP_SHARED, fd, 0);
if (page == MAP_FAILED)
atf_tc_fail("mmap(2) failed; errno=%d", errno);
@@ -143,7 +142,7 @@
atf_tc_fail("missing data ('%c' != '1')", page[0]);
close(fd);
- if (munmap(page, getpagesize()) < 0)
+ if (munmap(page, pagesize) < 0)
atf_tc_fail("munmap failed; errno=%d", errno);
ATF_REQUIRE_MSG(shm_unlink(test_path) != -1,
@@ -154,7 +153,9 @@
ATF_TC_BODY(reopen_object, tc)
{
char *page;
- int fd;
+ int fd, pagesize;
+
+ ATF_REQUIRE(0 < (pagesize = getpagesize()));
fd = scribble_object();
close(fd);
@@ -163,14 +164,14 @@
if (fd < 0)
atf_tc_fail("shm_open(2) failed; errno=%d", errno);
- page = mmap(0, getpagesize(), PROT_READ, MAP_SHARED, fd, 0);
+ page = mmap(0, pagesize, PROT_READ, MAP_SHARED, fd, 0);
if (page == MAP_FAILED)
atf_tc_fail("mmap(2) failed; errno=%d", errno);
if (page[0] != '1')
atf_tc_fail("missing data ('%c' != '1')", page[0]);
- munmap(page, getpagesize());
+ munmap(page, pagesize);
close(fd);
ATF_REQUIRE_MSG(shm_unlink(test_path) != -1,
"shm_unlink failed; errno=%d", errno);
@@ -180,7 +181,9 @@
ATF_TC_BODY(readonly_mmap_write, tc)
{
char *page;
- int fd;
+ int fd, pagesize;
+
+ ATF_REQUIRE(0 < (pagesize = getpagesize()));
gen_test_path();
@@ -188,8 +191,7 @@
ATF_REQUIRE_MSG(fd >= 0, "shm_open failed; errno=%d", errno);
/* PROT_WRITE should fail with EACCES. */
- page = mmap(0, getpagesize(), PROT_READ | PROT_WRITE, MAP_SHARED, fd,
- 0);
+ page = mmap(0, pagesize, PROT_READ | PROT_WRITE, MAP_SHARED, fd, 0);
if (page != MAP_FAILED)
atf_tc_fail("mmap(PROT_WRITE) succeeded unexpectedly");
@@ -359,49 +361,49 @@
{
pid_t pid;
struct stat sb;
- char err_buf[1024], *page;
- int fd, status;
+ char *page;
+ int fd, pagesize, status;
+
+ ATF_REQUIRE(0 < (pagesize = getpagesize()));
/* Start off with a size of a single page. */
fd = shm_open(SHM_ANON, O_CREAT|O_RDWR, 0777);
if (fd < 0)
atf_tc_fail("shm_open failed; errno=%d", errno);
- if (ftruncate(fd, getpagesize()) < 0)
+ if (ftruncate(fd, pagesize) < 0)
atf_tc_fail("ftruncate(1) failed; errno=%d", errno);
if (fstat(fd, &sb) < 0)
atf_tc_fail("fstat(1) failed; errno=%d", errno);
- if (sb.st_size != getpagesize())
+ if (sb.st_size != pagesize)
atf_tc_fail("first resize failed (%d != %d)",
- (int)sb.st_size, getpagesize());
+ (int)sb.st_size, pagesize);
/* Write a '1' to the first byte. */
- page = mmap(0, getpagesize(), PROT_READ|PROT_WRITE, MAP_SHARED, fd,
- 0);
+ page = mmap(0, pagesize, PROT_READ|PROT_WRITE, MAP_SHARED, fd, 0);
if (page == MAP_FAILED)
atf_tc_fail("mmap(1)");
page[0] = '1';
- if (munmap(page, getpagesize()) < 0)
+ if (munmap(page, pagesize) < 0)
atf_tc_fail("munmap(1) failed; errno=%d", errno);
/* Grow the object to 2 pages. */
- if (ftruncate(fd, getpagesize() * 2) < 0)
+ if (ftruncate(fd, pagesize * 2) < 0)
atf_tc_fail("ftruncate(2) failed; errno=%d", errno);
if (fstat(fd, &sb) < 0)
atf_tc_fail("fstat(2) failed; errno=%d", errno);
- if (sb.st_size != getpagesize() * 2)
+ if (sb.st_size != pagesize * 2)
atf_tc_fail("second resize failed (%d != %d)",
- (int)sb.st_size, getpagesize() * 2);
+ (int)sb.st_size, pagesize * 2);
/* Check for '1' at the first byte. */
- page = mmap(0, getpagesize() * 2, PROT_READ|PROT_WRITE, MAP_SHARED,
- fd, 0);
+ page = mmap(0, pagesize * 2, PROT_READ|PROT_WRITE, MAP_SHARED, fd, 0);
if (page == MAP_FAILED)
atf_tc_fail("mmap(2) failed; errno=%d", errno);
@@ -409,18 +411,18 @@
atf_tc_fail("'%c' != '1'", page[0]);
/* Write a '2' at the start of the second page. */
- page[getpagesize()] = '2';
+ page[pagesize] = '2';
/* Shrink the object back to 1 page. */
- if (ftruncate(fd, getpagesize()) < 0)
+ if (ftruncate(fd, pagesize) < 0)
atf_tc_fail("ftruncate(3) failed; errno=%d", errno);
if (fstat(fd, &sb) < 0)
atf_tc_fail("fstat(3) failed; errno=%d", errno);
- if (sb.st_size != getpagesize())
+ if (sb.st_size != pagesize)
atf_tc_fail("third resize failed (%d != %d)",
- (int)sb.st_size, getpagesize());
+ (int)sb.st_size, pagesize);
/*
* Fork a child process to make sure the second page is no
@@ -435,16 +437,16 @@
char c;
/* Don't generate a core dump. */
- getrlimit(RLIMIT_CORE, &lim);
+ ATF_REQUIRE(getrlimit(RLIMIT_CORE, &lim) == 0);
lim.rlim_cur = 0;
- setrlimit(RLIMIT_CORE, &lim);
+ ATF_REQUIRE(setrlimit(RLIMIT_CORE, &lim) == 0);
/*
* The previous ftruncate(2) shrunk the backing object
* so that this address is no longer valid, so reading
* from it should trigger a SIGSEGV.
*/
- c = page[getpagesize()];
+ c = page[pagesize];
fprintf(stderr, "child: page 1: '%c'\n", c);
exit(0);
}
@@ -456,15 +458,15 @@
atf_tc_fail("child terminated with status %x", status);
/* Grow the object back to 2 pages. */
- if (ftruncate(fd, getpagesize() * 2) < 0)
+ if (ftruncate(fd, pagesize * 2) < 0)
atf_tc_fail("ftruncate(2) failed; errno=%d", errno);
if (fstat(fd, &sb) < 0)
atf_tc_fail("fstat(2) failed; errno=%d", errno);
- if (sb.st_size != getpagesize() * 2)
+ if (sb.st_size != pagesize * 2)
atf_tc_fail("fourth resize failed (%d != %d)",
- (int)sb.st_size, getpagesize());
+ (int)sb.st_size, pagesize);
/*
* Note that the mapping at 'page' for the second page is
@@ -475,9 +477,9 @@
* object was shrunk and the new pages when an object are
* grown are zero-filled.
*/
- if (page[getpagesize()] != 0)
+ if (page[pagesize] != 0)
atf_tc_fail("invalid data at %d: %x != 0",
- getpagesize(), (int)page[getpagesize()]);
+ pagesize, (int)page[pagesize]);
close(fd);
}
@@ -524,7 +526,7 @@
scval = sysconf(_SC_PAGESIZE);
if (scval == -1 && errno != 0) {
atf_tc_fail("sysconf(_SC_PAGESIZE) failed; errno=%d", errno);
- } else if (scval <= 0 || (size_t)psize != psize) {
+ } else if (scval <= 0) {
fprintf(stderr, "bogus return from sysconf(_SC_PAGESIZE): %ld",
scval);
psize = 4096;
@@ -542,8 +544,7 @@
ATF_REQUIRE_MSG(ftruncate(desc, (off_t)psize) != -1,
"ftruncate failed; errno=%d", errno);
- region = mmap((void *)0, psize, PROT_READ | PROT_WRITE, MAP_SHARED,
- desc, (off_t)0);
+ region = mmap(NULL, psize, PROT_READ | PROT_WRITE, MAP_SHARED, desc, 0);
ATF_REQUIRE_MSG(region != MAP_FAILED, "mmap failed; errno=%d", errno);
memset(region, '\377', psize);
@@ -601,6 +602,9 @@
strsignal(WTERMSIG(status)));
}
}
+
+ munmap(region, psize);
+ shm_unlink(test_path);
}
ATF_TP_ADD_TCS(tp)

File Metadata

Mime Type
text/plain
Expires
Sun, Feb 1, 2:12 AM (20 h, 38 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
28247552
Default Alt Text
D6012.diff (8 KB)

Event Timeline