[LTP] [PATCH 1/3] lib: Add SAFE_CHOWN()

[LTP] [PATCH 1/3] lib: Add SAFE_CHOWN()

[Xing Gu]

Signed-off-by: Xing Gu <gux.fnst@cn.fujitsu.com>
---
 include/safe_macros.h |  5 +++++
 lib/safe_macros.c     | 15 +++++++++++++++
 2 files changed, 20 insertions(+)

diff --git a/include/safe_macros.h b/include/safe_macros.h
index ea1e976..f521347 100644
--- a/include/safe_macros.h
+++ b/include/safe_macros.h
@@ -231,6 +231,11 @@ int safe_fchmod(const char *file, const int lineno, void (cleanup_fn)(void),
 #define SAFE_FCHMOD(cleanup_fn, fd, mode) \
 	safe_fchmod(__FILE__, __LINE__, (cleanup_fn), (fd), (mode))
 
+int safe_chown(const char *file, const int lineno, void (cleanup_fn)(void),
+			const char *path, uid_t owner, gid_t group);
+#define SAFE_CHOWN(cleanup_fn, path, owner, group) \
+	safe_chown(__FILE__, __LINE__, (cleanup_fn), (path), (owner), (group))
+
 int safe_fchown(const char *file, const int lineno, void (cleanup_fn)(void),
                 int fd, uid_t owner, gid_t group);
 #define SAFE_FCHOWN(cleanup_fn, fd, owner, group) \
diff --git a/lib/safe_macros.c b/lib/safe_macros.c
index 13cc08e..a07cffa 100644
--- a/lib/safe_macros.c
+++ b/lib/safe_macros.c
@@ -662,6 +662,21 @@ int safe_fchmod(const char *file, const int lineno,
 	return rval;
 }
 
+int safe_chown(const char *file, const int lineno, void (cleanup_fn)(void),
+			const char *path, uid_t owner, gid_t group)
+{
+	int rval;
+
+	rval = chown(path, owner, group);
+
+	if (rval == -1) {
+		tst_brkm(TBROK | TERRNO, cleanup_fn,
+			"%s:%d: chown(%s,%d,%d) failed",
+			file, lineno, path, owner, group);
+	}
+
+	return rval;
+}
 
 int safe_fchown(const char *file, const int lineno, void (cleanup_fn)(void),
                 int fd, uid_t owner, gid_t group)
-- 
1.9.3


------------------------------------------------------------------------------
HPCC Systems Open Source Big Data Platform from LexisNexis Risk Solutions
Find What Matters Most in Your Big Data with HPCC Systems
Open Source. Fast. Scalable. Simple. Ideal for Dirty Data.
Leverages Graph Analysis for Fast Processing & Easy Data Exploration
http://p.sf.net/sfu/hpccsystems
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

[LTP] [PATCH 2/3] utimes/utimes01.c: cleanup

[Xing Gu]

Delete some useless comments.
Use some SAFE_* macros.
Move the test body from main() to utimes_verify().

Signed-off-by: Xing Gu <gux.fnst@cn.fujitsu.com>
---
 testcases/kernel/syscalls/utimes/utimes01.c | 440 ++++++++--------------------
 1 file changed, 124 insertions(+), 316 deletions(-)

diff --git a/testcases/kernel/syscalls/utimes/utimes01.c b/testcases/kernel/syscalls/utimes/utimes01.c
index fec55b8..25d6bfa 100644
--- a/testcases/kernel/syscalls/utimes/utimes01.c
+++ b/testcases/kernel/syscalls/utimes/utimes01.c
@@ -1,48 +1,34 @@
-/******************************************************************************/
-/* Copyright (c) Crackerjack Project., 2007 ,Hitachi, Ltd		     */
-/*	  Author(s): Takahiro Yasui <takahiro.yasui.mp@hitachi.com>,	      */
-/*								  	      */
-/* This program is free software;  you can redistribute it and/or modify      */
-/* it under the terms of the GNU General Public License as published by       */
-/* the Free Software Foundation; either version 2 of the License, or	  */
-/* (at your option) any later version.					*/
-/*									    */
-/* This program is distributed in the hope that it will be useful,	    */
-/* but WITHOUT ANY WARRANTY;  without even the implied warranty of	    */
-/* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See		  */
-/* the GNU General Public License for more details.			   */
-/*									    */
-/* You should have received a copy of the GNU General Public License	  */
-/* along with this program;  if not, write to the Free Software	       */
-/* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA    */
-/*									    */
-/******************************************************************************/
-/******************************************************************************/
-/*									    */
-/* File:	utimes01.c						    */
-/*									    */
-/* Description: This tests the utimes() syscall			       */
-/*									      */
-/* 									      */
-/*									      */
-/*									      */
-/*									      */
-/*									    */
-/* Usage:  <for command-line>						 */
-/* utimes01 [-c n] [-e][-i n] [-I x] [-p x] [-t]			      */
-/*      where,  -c n : Run n copies concurrently.			     */
-/*	      -e   : Turn on errno logging.				 */
-/*	      -i n : Execute test n times.				  */
-/*	      -I x : Execute test for x seconds.			    */
-/*	      -P x : Pause for x seconds between iterations.		*/
-/*	      -t   : Turn on syscall timing.				*/
-/*									    */
-/* Total Tests: 1							     */
-/*									    */
-/* Test Name:   utimes01						      */
-/* History:     Porting from Crackerjack to LTP is done by		    */
-/*	      Manas Kumar Nayak maknayak@in.ibm.com>			*/
-/******************************************************************************/
+/*
+ * Copyright (c) Crackerjack Project., 2007 ,Hitachi, Ltd
+ * Author(s): Takahiro Yasui <takahiro.yasui.mp@hitachi.com>
+ *
+ * This program is free software;  you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY;  without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See
+ * the GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program;  if not, write to the Free Software Foundation,
+ * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+ */
+/*
+ * Description:
+ *   Verify that,
+ *   1) utimes() returns -1 and sets errno to EACCES if times
+ *      is NULL, the caller's effective user ID does not match
+ *      the owner of the file, the caller does not have write
+ *      access to the file, and the caller is not privileged.
+ *   2) utimes() returns -1 and sets errno to ENOENT if filename
+ *      does not exist.
+ *   3) utimes() returns -1 and sets errno to EFAULT if filename
+ *      is NULL.
+ */
+
 #include <sys/types.h>
 #include <sys/stat.h>
 #include <sys/time.h>
@@ -53,309 +39,131 @@
 #include <stdlib.h>
 #include <errno.h>
 #include <stdio.h>
-
-#include "../utils/include_j_h.h"
-#include "../utils/common_j_h.c"
+#include <pwd.h>
 
 #include "test.h"
 #include "usctest.h"
+#include "safe_macros.h"
 #include "linux_syscall_numbers.h"
 
+#define TESTFILE1 "testfile1"
+#define TESTFILE2 "testfile2"
+#define FILE_MODE (S_IRWXU | S_IRGRP | S_IXGRP | \
+					S_IROTH | S_IXOTH)
+
+#define LTPUSER1 "nobody"
+#define LTPUSER2 "bin"
+
+static struct timeval a_tv[2] = { {0, 0}, {1000, 0} };
+static struct timeval m_tv[2] = { {1000, 0}, {0, 0} };
+static struct timeval tv[2] = { {1000, 0}, {2000, 0} };
+
+static struct test_case_t {
+	char *pathname;
+	struct timeval *times;
+	int exp_errno;
+} test_cases[] = {
+	{ TESTFILE1, a_tv, 0 },
+	{ TESTFILE1, m_tv, 0 },
+	{ TESTFILE2, NULL, EACCES },
+	{ "notexistfile", tv, ENOENT },
+	{ NULL, tv, EFAULT },
+};
+
+static void setup(void);
+static void cleanup(void);
+static void utimes_verify(const struct test_case_t *);
+
 char *TCID = "utimes01";
-int testno;
-int TST_TOTAL = 1;
-
-/* Extern Global Functions */
-/******************************************************************************/
-/*									    */
-/* Function:    cleanup						       */
-/*									    */
-/* Description: Performs all one time clean up for this test on successful    */
-/*	      completion,  premature exit or  failure. Closes all temporary */
-/*	      files, removes all temporary directories exits the test with  */
-/*	      appropriate return code by calling tst_exit() function.       */
-/*									    */
-/* Input:       None.							 */
-/*									    */
-/* Output:      None.							 */
-/*									    */
-/* Return:      On failure - Exits calling tst_exit(). Non '0' return code.   */
-/*	      On success - Exits calling tst_exit(). With '0' return code.  */
-/*									    */
-/******************************************************************************/
-void cleanup(void)
+int TST_TOTAL = ARRAY_SIZE(test_cases);
+static int exp_enos[] = { EACCES, ENOENT, EFAULT, 0 };
+
+int main(int ac, char **av)
 {
+	int i, lc;
+	const char *msg;
 
-	TEST_CLEANUP;
-	tst_rmdir();
+	msg = parse_opts(ac, av, NULL, NULL);
+	if (msg != NULL)
+		tst_brkm(TBROK, NULL, "OPTION PARSING ERROR - %s", msg);
 
+	setup();
+
+	TEST_EXP_ENOS(exp_enos);
+
+	for (lc = 0; TEST_LOOPING(lc); ++lc) {
+		tst_count = 0;
+
+		for (i = 0; i < TST_TOTAL; i++)
+			utimes_verify(&test_cases[i]);
+	}
+
+	cleanup();
+	tst_exit();
 }
 
-/* Local  Functions */
-/******************************************************************************/
-/*									    */
-/* Function:    setup							 */
-/*									    */
-/* Description: Performs all one time setup for this test. This function is   */
-/*	      typically used to capture signals, create temporary dirs      */
-/*	      and temporary files that may be used in the course of this    */
-/*	      test.							 */
-/*									    */
-/* Input:       None.							 */
-/*									    */
-/* Output:      None.							 */
-/*									    */
-/* Return:      On failure - Exits by calling cleanup().		      */
-/*	      On success - returns 0.				       */
-/*									    */
-/******************************************************************************/
 void setup(void)
 {
+	struct passwd *ltpuser;
+
 	tst_require_root(NULL);
 
-	/* Capture signals if any */
-	/* Create temporary directories */
+	tst_sig(NOFORK, DEF_HANDLER, cleanup);
+
 	TEST_PAUSE;
+
 	tst_tmpdir();
-}
 
-/*
- * Macros
- */
-#define SYSCALL_NAME    "utimes"
+	SAFE_TOUCH(cleanup, TESTFILE1, FILE_MODE, NULL);
+	ltpuser = SAFE_GETPWNAM(cleanup, LTPUSER1);
+	SAFE_CHOWN(cleanup, TESTFILE1, ltpuser->pw_uid,
+		ltpuser->pw_gid);
 
-enum test_type {
-	NORMAL,
-	FILE_NOT_EXIST,
-	NO_FNAME,
-};
+	SAFE_TOUCH(cleanup, TESTFILE2, FILE_MODE, NULL);
+	ltpuser = SAFE_GETPWNAM(cleanup, LTPUSER2);
+	SAFE_CHOWN(cleanup, TESTFILE2, ltpuser->pw_uid,
+		ltpuser->pw_gid);
 
-/*
- * Data Structure
- */
-struct test_case {
-	int ttype;
-	long a_sec;
-	long m_sec;
-	char *user;
-	int ret;
-	int err;
+	ltpuser = SAFE_GETPWNAM(cleanup, LTPUSER1);
+	SAFE_SETEUID(cleanup, ltpuser->pw_uid);
+}
 
-};
+static void utimes_verify(const struct test_case_t *tc)
+{
+	struct stat st;
+	struct timeval tmp_tv[2];
 
-/* Test cases
- *
- *   test status of errors on man page
- *
- *   EACCES	     v (permission denied)
- *   ENOENT	     v (file does not exist)
- *
- *   test status of errors on man page
- *
- *   EFAULT	     v (points to not process address space)
- */
+	if (tc->exp_errno == 0) {
+		SAFE_STAT(cleanup, tc->pathname, &st);
 
-static struct test_case tcase[] = {
-	{			// case00
-	 .ttype = NORMAL,
-	 .a_sec = 0,
-	 .m_sec = 1000,
-	 .ret = 0,
-	 .err = 0,
-	 },
-	{			// case01
-	 .ttype = NORMAL,
-	 .a_sec = 1000,
-	 .m_sec = 0,
-	 .ret = 0,
-	 .err = 0,
-	 },
-	{			// case02
-	 .ttype = NORMAL,
-	 .user = "nobody",
-	 .ret = -1,
-	 .err = EACCES,		// RHEL4U1 + 2.6.18 returns EPERM
-	 },
-	{			// case03
-	 .ttype = FILE_NOT_EXIST,
-	 .a_sec = 1000,
-	 .m_sec = 2000,
-	 .ret = -1,
-	 .err = ENOENT,
-	 },
-
-	{			// case04
-	 .ttype = NO_FNAME,
-	 .a_sec = 1000,
-	 .m_sec = 2000,
-	 .ret = -1,
-	 .err = EFAULT,
-	 },
-};
+		tmp_tv[0].tv_sec = st.st_atime;
+		tmp_tv[0].tv_usec = 0;
+		tmp_tv[1].tv_sec = st.st_mtime;
+		tmp_tv[1].tv_usec = 0;
+	}
 
-/*
- * do_test()
- *
- *   Input  : TestCase Data
- *   Return : RESULT_OK(0), RESULT_NG(1)
- *
- */
+	TEST(utimes(tc->pathname, tc->times));
 
-static int do_test(struct test_case *tc)
-{
-	int sys_ret;
-	int sys_errno;
-	int result = RESULT_OK;
-	struct timeval tv[2];
-	char fpath[PATH_MAX], c = '\0';
-	int rc, len, cmp_ok = 1;
-	struct stat st;
-	uid_t old_uid;
-
-	/* XXX (garrcoop): memory leak with tst_get_tmpdir. */
-	TEST(rc = setup_file(tst_get_tmpdir(), "test.file", fpath));
-	if (rc < 0)
-		return 1;
-	/* The test just needs the file, so no need to keep it open. */
-	close(rc);
-
-	/*
-	 * Change effective user id
-	 */
-	if (tc->user != NULL) {
-		TEST(rc = setup_euid(tc->user, &old_uid));
-		if (rc < 0)
-			goto EXIT2;
-	}
+	TEST_ERROR_LOG(TEST_ERRNO);
 
-	/*
-	 * Execute system call
-	 */
-	memset(tv, 0, 2 * sizeof(struct timeval));
-	tv[0].tv_sec = tc->a_sec;
-	tv[1].tv_sec = tc->m_sec;
-	TEST(len = strlen(fpath));
-	if (tc->ttype == FILE_NOT_EXIST) {
-		c = fpath[len - 1];
-		fpath[len - 1] = '\0';
-	}
-	errno = 0;
-	if (tc->ttype == NO_FNAME) {
-		/**
-		 * Note (garrcoop):
-		 *
-		 * If you do NULL directly, then gcc [4.3] will complain when
-		 * one specifies -Wnonnull in CPPFLAGS. This is a negative
-		 * test, but let's not allow the compiler to complain about
-		 * something trivial like this.
-		 **/
-		const char *dummy = NULL;
-		TEST(sys_ret = utimes(dummy, tv));
+	if (TEST_ERRNO == tc->exp_errno) {
+		tst_resm(TPASS | TTERRNO, "utimes() failed as expected");
 	} else {
-		if (tc->user == NULL)
-			TEST(sys_ret = utimes(fpath, tv));
-		else
-			TEST(sys_ret = utimes(fpath, NULL));
+		tst_resm(TFAIL | TTERRNO,
+			"utimes() failed unexpectedly; expected: %d - %s",
+			tc->exp_errno, strerror(tc->exp_errno));
 	}
-	tv[0].tv_sec = tc->a_sec;
-	tv[1].tv_sec = tc->m_sec;
-	TEST(len = strlen(fpath));
-	if (tc->ttype == FILE_NOT_EXIST) {
-		c = fpath[len - 1];
-		fpath[len - 1] = '\0';
-	}
-	errno = 0;
-	if (tc->ttype == NO_FNAME) {
-		/**
-                 * Note (garrcoop):
-                 *
-                 * If you do NULL directly, then gcc [4.3] will complain when
-                 * one specifies -Wnonnull in CPPFLAGS. This is a negative
-                 * test, but let's not allow the compiler to complain about
-                 * something trivial like this.
-                 **/
-		const char *dummy = NULL;
-		TEST(sys_ret = utimes(dummy, tv));
-	} else {
-		if (tc->user == NULL)
-			TEST(sys_ret = utimes(fpath, tv));
-		else
-			TEST(sys_ret = utimes(fpath, NULL));
-	}
-	sys_errno = errno;
-	if (tc->ttype == FILE_NOT_EXIST)
-		fpath[len - 1] = c;
-	if (sys_ret < 0)
-		goto TEST_END;
-
-	/*
-	 * Check test file's time stamp
-	 */
-	rc = stat(fpath, &st);
-	if (rc < 0) {
-		EPRINTF("stat failed.\n");
-		result = 1;
-		goto EXIT1;
-	}
-	tst_resm(TINFO, "E:%ld,%ld <=> R:%ld,%ld", tv[0].tv_sec, tv[1].tv_sec,
-		 st.st_atime, st.st_mtime);
-	cmp_ok = st.st_atime == tv[0].tv_sec && st.st_mtime == tv[1].tv_sec;
-
-	/*
-	 * Check results
-	 */
-TEST_END:
-	result |= (sys_errno != tc->err) || !cmp_ok;
-	PRINT_RESULT_CMP(sys_ret >= 0, tc->ret, tc->err, sys_ret, sys_errno,
-			 cmp_ok);
-
-	/*
-	 * Restore effective user id
-	 */
-EXIT1:
-	if (tc->user != NULL) {
-		TEST(rc = cleanup_euid(old_uid));
-		if (rc < 0)
-			return 1;
-	}
-EXIT2:
-	TEST(cleanup_file(fpath));
 
-	return result;
+	if (TEST_ERRNO == 0 && utimes(tc->pathname, tmp_tv) == -1)
+			tst_brkm(TBROK | TERRNO, cleanup, "utimes() failed.");
 }
 
-/*
- * main()
- */
-
-int main(int ac, char **av)
+void cleanup(void)
 {
-	int result = RESULT_OK;
-	int i;
-	int lc;
-	const char *msg;
+	if (seteuid(0) == -1)
+		tst_resm(TWARN | TERRNO, "seteuid(0) failed");
 
-	if ((msg = parse_opts(ac, av, NULL, NULL)) != NULL)
-		tst_brkm(TBROK, NULL, "OPTION PARSING ERROR - %s", msg);
-
-	setup();
+	TEST_CLEANUP;
 
-	for (lc = 0; TEST_LOOPING(lc); ++lc) {
-		tst_count = 0;
-		for (testno = 0; testno < TST_TOTAL; ++testno) {
-
-			for (i = 0; i < (int)(sizeof(tcase) / sizeof(tcase[0]));
-			     i++) {
-				int ret;
-				tst_resm(TINFO, "(case%02d) START", i);
-				ret = do_test(&tcase[i]);
-				tst_resm(TINFO, "(case%02d) END => %s",
-					 i, (ret == 0) ? "OK" : "NG");
-				result |= ret;
-			}
-
-		}
-	}
-	cleanup();
-	tst_exit();
+	tst_rmdir();
 }
-- 
1.9.3


------------------------------------------------------------------------------
HPCC Systems Open Source Big Data Platform from LexisNexis Risk Solutions
Find What Matters Most in Your Big Data with HPCC Systems
Open Source. Fast. Scalable. Simple. Ideal for Dirty Data.
Leverages Graph Analysis for Fast Processing & Easy Data Exploration
http://p.sf.net/sfu/hpccsystems
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

[LTP] [PATCH 3/3] utimes/utimes01.c: add EPERM and EROFS error value tests

[Xing Gu]

Signed-off-by: Xing Gu <gux.fnst@cn.fujitsu.com>
---
 runtest/syscalls                            |  2 +-
 testcases/kernel/syscalls/utimes/utimes01.c | 59 ++++++++++++++++++++++++++++-
 2 files changed, 58 insertions(+), 3 deletions(-)

diff --git a/runtest/syscalls b/runtest/syscalls
index c132a69..8514942 100644
--- a/runtest/syscalls
+++ b/runtest/syscalls
@@ -1310,7 +1310,7 @@ utime04 utime04
 utime05 utime05
 utime06 utime06
 
-utimes01 utimes01
+utimes01 utimes01 -D $LTP_DEV -T $LTP_DEV_FS_TYPE
 
 # Introduced from Kernel 2.6.22 onwards
 utimensat01 utimensat_tests.sh
diff --git a/testcases/kernel/syscalls/utimes/utimes01.c b/testcases/kernel/syscalls/utimes/utimes01.c
index 25d6bfa..7f9dffb 100644
--- a/testcases/kernel/syscalls/utimes/utimes01.c
+++ b/testcases/kernel/syscalls/utimes/utimes01.c
@@ -27,6 +27,11 @@
  *      does not exist.
  *   3) utimes() returns -1 and sets errno to EFAULT if filename
  *      is NULL.
+ *   4) utimes() returns -1 and sets errno to EPERM if times is
+ *      not NULL, the caller's effective UID does not match the
+ *      owner of the file, and the caller is not privileged.
+ *   5) utimes() returns -1 and sets errno to EROFS if path resides
+ *      on a read-only file system.
  */
 
 #include <sys/types.h>
@@ -40,20 +45,34 @@
 #include <errno.h>
 #include <stdio.h>
 #include <pwd.h>
+#include <sys/mount.h>
 
 #include "test.h"
 #include "usctest.h"
 #include "safe_macros.h"
 #include "linux_syscall_numbers.h"
 
+#define MNTPOINT "mntpoint"
 #define TESTFILE1 "testfile1"
 #define TESTFILE2 "testfile2"
+#define TESTFILE3 "mntpoint/testfile"
 #define FILE_MODE (S_IRWXU | S_IRGRP | S_IXGRP | \
 					S_IROTH | S_IXOTH)
+#define DIR_MODE (S_IRWXU | S_IRWXG | S_IRWXO)
 
 #define LTPUSER1 "nobody"
 #define LTPUSER2 "bin"
 
+static char *fstype = "ext2";
+static char *device;
+static int mount_flag;
+
+static option_t options[] = {
+	{"T:", NULL, &fstype},
+	{"D:", NULL, &device},
+	{NULL, NULL, NULL}
+};
+
 static struct timeval a_tv[2] = { {0, 0}, {1000, 0} };
 static struct timeval m_tv[2] = { {1000, 0}, {0, 0} };
 static struct timeval tv[2] = { {1000, 0}, {2000, 0} };
@@ -68,25 +87,34 @@ static struct test_case_t {
 	{ TESTFILE2, NULL, EACCES },
 	{ "notexistfile", tv, ENOENT },
 	{ NULL, tv, EFAULT },
+	{ TESTFILE2, tv, EPERM },
+	{ TESTFILE3, tv, EROFS },
 };
 
 static void setup(void);
 static void cleanup(void);
 static void utimes_verify(const struct test_case_t *);
+static void help(void);
 
 char *TCID = "utimes01";
 int TST_TOTAL = ARRAY_SIZE(test_cases);
-static int exp_enos[] = { EACCES, ENOENT, EFAULT, 0 };
+static int exp_enos[] = { EACCES, ENOENT, EFAULT,
+							EPERM, EROFS, 0 };
 
 int main(int ac, char **av)
 {
 	int i, lc;
 	const char *msg;
 
-	msg = parse_opts(ac, av, NULL, NULL);
+	msg = parse_opts(ac, av, options, help);
 	if (msg != NULL)
 		tst_brkm(TBROK, NULL, "OPTION PARSING ERROR - %s", msg);
 
+	if (!device) {
+		tst_brkm(TCONF, NULL, "you must specify the device "
+			"used for mounting with -D option");
+	}
+
 	setup();
 
 	TEST_EXP_ENOS(exp_enos);
@@ -124,6 +152,23 @@ void setup(void)
 	SAFE_CHOWN(cleanup, TESTFILE2, ltpuser->pw_uid,
 		ltpuser->pw_gid);
 
+	tst_mkfs(NULL, device, fstype, NULL);
+	SAFE_MKDIR(cleanup, MNTPOINT, DIR_MODE);
+	if (mount(device, MNTPOINT, fstype, 0, NULL) == -1) {
+		tst_brkm(TBROK | TERRNO, cleanup,
+			"mount device:%s failed", device);
+	}
+	SAFE_TOUCH(cleanup, TESTFILE3, FILE_MODE, NULL);
+	ltpuser = SAFE_GETPWNAM(cleanup, LTPUSER1);
+	SAFE_CHOWN(cleanup, TESTFILE3, ltpuser->pw_uid,
+		ltpuser->pw_gid);
+	if (mount(device, MNTPOINT, fstype,
+			MS_REMOUNT | MS_RDONLY, NULL) == -1) {
+		tst_brkm(TBROK | TERRNO, cleanup,
+			"mount device:%s failed", device);
+	}
+	mount_flag = 1;
+
 	ltpuser = SAFE_GETPWNAM(cleanup, LTPUSER1);
 	SAFE_SETEUID(cleanup, ltpuser->pw_uid);
 }
@@ -163,7 +208,17 @@ void cleanup(void)
 	if (seteuid(0) == -1)
 		tst_resm(TWARN | TERRNO, "seteuid(0) failed");
 
+	if (mount_flag && umount(MNTPOINT) == -1)
+		tst_resm(TWARN | TERRNO, "umount %s failed", MNTPOINT);
+
 	TEST_CLEANUP;
 
 	tst_rmdir();
 }
+
+static void help(void)
+{
+	printf("-T type   : specifies the type of filesystem to be mounted. "
+		"Default ext2.\n");
+	printf("-D device : device used for mounting.\n");
+}
-- 
1.9.3


------------------------------------------------------------------------------
HPCC Systems Open Source Big Data Platform from LexisNexis Risk Solutions
Find What Matters Most in Your Big Data with HPCC Systems
Open Source. Fast. Scalable. Simple. Ideal for Dirty Data.
Leverages Graph Analysis for Fast Processing & Easy Data Exploration
http://p.sf.net/sfu/hpccsystems
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

Re: [LTP] [PATCH 2/3] utimes/utimes01.c: cleanup

[chrubis]

Hi!
> -/* History:     Porting from Crackerjack to LTP is done by		    */
> -/*	      Manas Kumar Nayak maknayak@in.ibm.com>			*/

I've added this to the header under the Authors.

> -		/**
> -		 * Note (garrcoop):
> -		 *
> -		 * If you do NULL directly, then gcc [4.3] will complain when
> -		 * one specifies -Wnonnull in CPPFLAGS. This is a negative
> -		 * test, but let's not allow the compiler to complain about
> -		 * something trivial like this.
> -		 **/
> -		const char *dummy = NULL;
> -		TEST(sys_ret = utimes(dummy, tv));
> +	if (TEST_ERRNO == tc->exp_errno) {
> +		tst_resm(TPASS | TTERRNO, "utimes() failed as expected");
                                                      ^
				      I've changed this to "worked as expected"
				      because the first two testcases
				      do not fait at all.

Also I may be better to split the test into two files one for positive
tests and one for negative tests because as it is the code does not
check that utimes() actually changed the times and we may add test that
calls utimes() with NULL times, which should set the times to current
time. Adding these changes to the currect code would overcomplicate the
verify function...

Patchset applied, Thanks.

-- 
Cyril Hrubis
chrubis@suse.cz

------------------------------------------------------------------------------
HPCC Systems Open Source Big Data Platform from LexisNexis Risk Solutions
Find What Matters Most in Your Big Data with HPCC Systems
Open Source. Fast. Scalable. Simple. Ideal for Dirty Data.
Leverages Graph Analysis for Fast Processing & Easy Data Exploration
http://p.sf.net/sfu/hpccsystems
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list