From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from sog-mx-3.v43.ch3.sourceforge.com ([172.29.43.193] helo=mx.sourceforge.net) by sfs-ml-4.v29.ch3.sourceforge.com with esmtp (Exim 4.76) (envelope-from ) id 1UdDQl-000201-CS for ltp-list@lists.sourceforge.net; Fri, 17 May 2013 05:42:15 +0000 Received: from [222.73.24.84] (helo=song.cn.fujitsu.com) by sog-mx-3.v43.ch3.sourceforge.com with esmtp (Exim 4.76) id 1UdDQj-0000g0-U5 for ltp-list@lists.sourceforge.net; Fri, 17 May 2013 05:42:15 +0000 Message-ID: <5195C294.2040809@cn.fujitsu.com> Date: Fri, 17 May 2013 13:39:32 +0800 From: DAN LI MIME-Version: 1.0 References: <51944C16.5080607@cn.fujitsu.com> In-Reply-To: <51944C16.5080607@cn.fujitsu.com> Subject: [LTP] [PATCH V2 2/2] shmat/shmat01.c: Test for specifying NULL to shmaddr List-Id: Linux Test Project General Discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ltp-list-bounces@lists.sourceforge.net To: LTP list Test for statement: "If shmaddr is NULL, the system chooses a suitable (unused) address at which to attach the segment." Signed-off-by: DAN LI --- testcases/kernel/syscalls/ipc/shmat/shmat01.c | 60 +++++++++++++++++---------- 1 file changed, 38 insertions(+), 22 deletions(-) diff --git a/testcases/kernel/syscalls/ipc/shmat/shmat01.c b/testcases/kernel/syscalls/ipc/shmat/shmat01.c index 3461782..3c09817 100644 --- a/testcases/kernel/syscalls/ipc/shmat/shmat01.c +++ b/testcases/kernel/syscalls/ipc/shmat/shmat01.c @@ -46,7 +46,7 @@ #define CASE1 20 char *TCID = "shmat01"; -int TST_TOTAL = 3; +int TST_TOTAL = 4; int shm_id_1 = -1; @@ -62,6 +62,7 @@ static struct test_case_t { int *shmid; int offset; int flags; + int getbase; } *TC; static void check_functionality(int); @@ -70,6 +71,7 @@ int main(int argc, char *argv[]) { int lc, i; char *msg; + void *attchaddr; msg = parse_opts(argc, argv, NULL, NULL); if (msg != NULL) @@ -83,12 +85,16 @@ int main(int argc, char *argv[]) for (i = 0; i < TST_TOTAL; i++) { - base_addr = probe_free_addr(); + if (TC[i].getbase) { + base_addr = probe_free_addr(); + attchaddr = base_addr + TC[i].offset; + } else { + attchaddr = NULL; + } - addr = shmat(*(TC[i].shmid), base_addr + TC[i].offset, - TC[i].flags); - TEST_ERRNO = errno; + addr = shmat(*(TC[i].shmid), attchaddr, TC[i].flags); + TEST_ERRNO = errno; if (addr == (void *)-1) { tst_brkm(TFAIL | TTERRNO, cleanup, "shmat call failed"); @@ -142,35 +148,36 @@ static void check_functionality(int i) /* check for specific conditions depending on the type of attach */ switch (i) { case 0: + case 1: /* - * Check the functionality of the first call by simply - * "writing" a value to the shared memory space. + * Check the functionality of shmat by simply "writing" + * a value to the shared memory space. * If this fails the program will get a SIGSEGV, dump * core and exit. */ *shared = CASE0; break; - case 1: + case 2: /* - * Check the functionality of the second call by writing - * a value to the shared memory space and then checking - * that the original address given was rounded down as + * Check the functionality of shmat by writing a value + * to the shared memory space and then checking that + * the original address given was rounded down as * specified in the man page. */ *shared = CASE1; - orig_add = addr + ((unsigned long)TC[1].offset % SHMLBA); - if (orig_add != base_addr + TC[1].offset) { + orig_add = addr + ((unsigned long)TC[2].offset % SHMLBA); + if (orig_add != base_addr + TC[2].offset) { tst_resm(TFAIL, "shared memory address is not " "correct"); fail = 1; } break; - case 2: + case 3: /* * This time the shared memory is read only. Read the value - * and check that it is equal to the value set in case #2, + * and check that it is equal to the value set in last case, * because shared memory is persistent. */ @@ -195,20 +202,29 @@ void setup(void) if (TC == NULL) tst_brkm(TFAIL | TERRNO, cleanup, "failed to allocate memory"); - /* a straight forward read/write attach */ + /* set NULL as attaching address*/ TC[0].shmid = &shm_id_1; TC[0].offset = 0; TC[0].flags = 0; + TC[0].getbase = 0; - /* an attach using unaligned memory */ + /* a straight forward read/write attach */ TC[1].shmid = &shm_id_1; - TC[1].offset = SHMLBA - 1; - TC[1].flags = SHM_RND; + TC[1].offset = 0; + TC[1].flags = 0; + TC[1].getbase = 1; - /* a read only attach */ + /* an attach using unaligned memory */ TC[2].shmid = &shm_id_1; - TC[2].offset = 0; - TC[2].flags = SHM_RDONLY; + TC[2].offset = SHMLBA - 1; + TC[2].flags = SHM_RND; + TC[2].getbase = 1; + + /* a read only attach */ + TC[3].shmid = &shm_id_1; + TC[3].offset = 0; + TC[3].flags = SHM_RDONLY; + TC[3].getbase = 1; tst_tmpdir(); -- 1.8.1 ------------------------------------------------------------------------------ AlienVault Unified Security Management (USM) platform delivers complete security visibility with the essential security capabilities. Easily and efficiently configure, manage, and operate all of your security controls from a single console and one unified framework. Download a free trial. http://p.sf.net/sfu/alienvault_d2d _______________________________________________ Ltp-list mailing list Ltp-list@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/ltp-list