Skip to contentSkip to conversation

syscalls/fchmod03.c: Convert to new API && merge fchmod04 into fchmod03

PATCH New Jinhui huang 3 participants 4 messages Jun 14, 2018 – Nov 9, 2018

lore
Share & tools
lore
#0 lore raw

PATCH 1/2 syscalls/fchmod03.c: Convert to new API && merge fchmod04 into fchmod03

---
 runtest/ltplite                             |   1 -
 runtest/stress.part3                        |   1 -
 runtest/syscalls                            |   1 -
 testcases/kernel/syscalls/fchmod/.gitignore |   1 -
 testcases/kernel/syscalls/fchmod/fchmod03.c | 197 ++++++++--------------------
 testcases/kernel/syscalls/fchmod/fchmod04.c | 190 ---------------------------
 6 files changed, 53 insertions(+), 338 deletions(-)
 delete mode 100644 testcases/kernel/syscalls/fchmod/fchmod04.c
runtest/ltplite +0 -1
index 2492626..29a122e 100644
@@ -187,7 +187,6 @@ fchdir03 fchdir03
 fchmod01 fchmod01
 fchmod02 fchmod02
 fchmod03 fchmod03
-fchmod04 fchmod04
 fchmod05 fchmod05
 fchmod06 fchmod06
 
runtest/stress.part3 +0 -1
index d928719..bcea8bd 100644
@@ -128,7 +128,6 @@ fchdir03 fchdir03
 fchmod01 fchmod01
 fchmod02 fchmod02
 fchmod03 fchmod03
-fchmod04 fchmod04
 fchmod05 fchmod05
 fchmod06 fchmod06
 
runtest/syscalls +0 -1
index 65c96ed..4be1328 100644
@@ -180,7 +180,6 @@ fchdir03 fchdir03
 fchmod01 fchmod01
 fchmod02 fchmod02
 fchmod03 fchmod03
-fchmod04 fchmod04
 fchmod05 fchmod05
 fchmod06 fchmod06
 
testcases/kernel/syscalls/fchmod/.gitignore +0 -1
index 3697537..1469bdc 100644
@@ -1,6 +1,5 @@
 /fchmod01
 /fchmod02
 /fchmod03
-/fchmod04
 /fchmod05
 /fchmod06
testcases/kernel/syscalls/fchmod/fchmod03.c +53 -144

223 lines not on this page · raw message

testcases/kernel/syscalls/fchmod/fchmod04.c deleted +0 -190

190 lines not on this page · raw message

Signature · 1 line
1.8.3.1
#1 lore raw

PATCH 2/2 syscalls/fchmod05.c: Cleanup && convert to new API

---
 testcases/kernel/syscalls/fchmod/fchmod05.c | 238 ++++++----------------------
 1 file changed, 52 insertions(+), 186 deletions(-)
testcases/kernel/syscalls/fchmod/fchmod05.c +52 -186
index 0ab2023..ffddd58 100644
@@ -1,24 +1,7 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
 /*
- *
- *   Copyright (c) International Business Machines  Corp., 2001
- *
- *   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
- */
-
-/*
- * Test Name: fchmod05
+ * Copyright (c) International Business Machines  Corp., 2001
+ * Author: Wayne Boyer
  *
  * Test Description:
  *  Verify that, fchmod(2) will succeed to change the mode of a directory
@@ -31,197 +14,80 @@
  * Expected Result:
  *  fchmod() should return value 0 on success and though succeeds to change
  *  the mode of a directory but fails to set setgid bit on it.
- *
- * Algorithm:
- *  Setup:
- *   Setup signal handling.
- *   Create temporary directory.
- *   Pause for SIGUSR1 if option specified.
- *
- *  Test:
- *   Loop if the proper options are given.
- *   Execute system call
- *   Check return code, if system call failed (return=-1)
- *	Log the errno and Issue a FAIL message.
- *   Otherwise,
- *	Verify the Functionality of system call
- *      if successful,
- *		Issue Functionality-Pass message.
- *      Otherwise,
- *		Issue Functionality-Fail message.
- *  Cleanup:
- *   Print errno log and/or timing stats if options given
- *   Delete the temporary directory created.
- *
- * Usage:  <for command-line>
- *  fchmod05 [-c n] [-f] [-i n] [-I x] [-P x] [-t]
- *     where,  -c n : Run n copies concurrently.
- *             -f   : Turn off functionality Testing.
- *	       -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.
- *
- * HISTORY
- *	07/2001 Ported by Wayne Boyer
- *
- * RESTRICTIONS:
- *  This test should be run by 'non-super-user' only.
- *
  */
 
 #ifndef _GNU_SOURCE
 #define _GNU_SOURCE
 #endif
 
-#include <stdio.h>
-#include <stdlib.h>
-#include <sys/types.h>
-#include <sys/stat.h>
-#include <fcntl.h>
-#include <errno.h>
-#include <string.h>
-#include <signal.h>
-#include <unistd.h>
-#include <grp.h>
 #include <pwd.h>
+#include <grp.h>
+#include <errno.h>
 
-#include "test.h"
-#include "safe_macros.h"
+#include "tst_test.h"
 #include "fchmod.h"
 
-#define PERMS_GID		043777
+#define PERMS_DIR	043777
 
-int fd;				/* file descriptor for test directory */
-char *TCID = "fchmod05";
-int TST_TOTAL = 1;
+static int fd;
 
-void setup();			/* Main setup function for test */
-void cleanup();			/* Main cleanup function for test */
-
-int main(int ac, char **av)
+static void verify_fchmod(void)
 {
-	struct stat stat_buf;	/* stat struct */
-	int lc;
-	mode_t dir_mode;	/* mode permissions set on test directory */
-
-	tst_parse_opts(ac, av, NULL, NULL);
-
-	setup();
-
-	for (lc = 0; TEST_LOOPING(lc); lc++) {
-
-		tst_count = 0;
-
-		/*
-		 * Call fchmod(2) with mode argument
-		 * to set setgid bit on TESTDIR.
-		 */
-
-		TEST(fchmod(fd, PERMS_GID));
-
-		if (TEST_RETURN == -1) {
-			tst_resm(TFAIL, "fchmod(%d, %#o) Failed, errno=%d : %s",
-				 fd, PERMS, TEST_ERRNO, strerror(TEST_ERRNO));
-			continue;
-		}
-		/*
-		 * Get the directory information using
-		 * fstat(2).
-		 */
-		if (fstat(fd, &stat_buf) < 0) {
-			tst_brkm(TFAIL, cleanup,
-				 "fstat(2) of %s failed, errno:%d",
-				 TESTDIR, TEST_ERRNO);
-		}
-		dir_mode = stat_buf.st_mode;
-		if ((PERMS_GID & ~S_ISGID) != dir_mode) {
-			tst_resm(TFAIL, "%s: Incorrect modes 0%03o, "
-				 "Expected 0%03o",
-				 TESTDIR, dir_mode, PERMS_GID & ~S_ISGID);
-		} else {
-			tst_resm(TPASS, "Functionality of fchmod(%d, "
-				 "%#o) successful", fd,
-				 PERMS_GID & ~S_ISGID);
-		}
+	struct stat stat_buf;
+	mode_t dir_mode;
+
+	TEST(fchmod(fd, PERMS_DIR));
+	if (TEST_RETURN == -1)
+		tst_res(TFAIL | TTERRNO, "fchmod() failed unexpectly");
+
+	SAFE_FSTAT(fd, &stat_buf);
+	dir_mode = stat_buf.st_mode;
+
+	if ((PERMS_DIR & ~S_ISGID) != dir_mode) {
+		tst_res(TFAIL, "%s: Incorrect modes 0%03o, Expected 0%03o",
+			TESTDIR, dir_mode & ~S_ISGID, PERMS_DIR);
+	} else {
+		tst_res(TPASS, "Functionality of fchmod(%d, %#o) successful",
+			fd, PERMS_DIR);
 	}
-
-	cleanup();
-	tst_exit();
 }
 
-/*
- * void
- * setup() - performs all ONE TIME setup for this test.
- *  Create a temporary directory and cd to it.
- *  Create a test directory under temporary directory.
- *  Invoke setuid to root program to modify group ownership
- *  on test directory.
- *  Open the test directory for reading.
- */
-void setup(void)
+static void setup(void)
 {
-	struct passwd *nobody_u;
-	struct group *bin_group;
-
-	tst_require_root();
-
-	tst_sig(FORK, DEF_HANDLER, cleanup);
-
-	TEST_PAUSE;
-
-	tst_tmpdir();
+	struct passwd *ltpuser;
+	struct group *ltpgroup;
 
-	nobody_u = getpwnam("nobody");
-	if (!nobody_u)
-		tst_brkm(TBROK, cleanup,
-			 "Couldn't find uid of nobody: %s", strerror(errno));
+	ltpuser = SAFE_GETPWNAM("nobody");
+	ltpgroup = SAFE_GETGRNAM("bin");
 
-	bin_group = getgrnam("bin");
-	if (!bin_group)
-		tst_brkm(TBROK, cleanup,
-			 "Couldn't find gid of bin: %s", strerror(errno));
+	SAFE_MKDIR(TESTDIR, DIR_MODE);
 
-	/*
-	 * Create a test directory under temporary directory with specified
-	 * mode permissions and change the gid of test directory to that of
-	 * guest user.
-	 */
-	SAFE_MKDIR(cleanup, TESTDIR, DIR_MODE);
-
-	if (setgroups(1, &nobody_u->pw_gid) == -1)
-		tst_brkm(TBROK, cleanup,
-			 "Couldn't change supplementary group Id: %s",
-			 strerror(errno));
+	if (setgroups(1, &ltpuser->pw_gid) == -1) {
+		tst_brk(TBROK, "Couldn't change supplementary group Id: %s",
+			tst_strerrno(TEST_ERRNO));
+	}
 
-	SAFE_CHOWN(cleanup, TESTDIR, nobody_u->pw_uid, bin_group->gr_gid);
+	SAFE_CHOWN(TESTDIR, ltpuser->pw_uid, ltpgroup->gr_gid);
 
-	/* change to nobody:nobody */
-	if (setegid(nobody_u->pw_gid) == -1 || seteuid(nobody_u->pw_uid) == -1)
-		tst_brkm(TBROK, cleanup, "Couldn't switch to nobody:nobody: %s",
-			 strerror(errno));
+	SAFE_SETEGID(ltpuser->pw_gid);
+	SAFE_SETEUID(ltpuser->pw_uid);
 
-	/* Open the test directory for reading */
-	fd = SAFE_OPEN(cleanup, TESTDIR, O_RDONLY);
+	fd = SAFE_OPEN(TESTDIR, O_RDONLY);
 }
 
-/*
- * void
- * cleanup() - performs all ONE TIME cleanup for this test at
- *		completion or premature exit.
- *  Close the test directory opened in the setup().
- *  Remove the test directory and temporary directory created in
- *  in the setup().
- */
-void cleanup(void)
+static void cleanup(void)
 {
+	if (fd > 0)
+		SAFE_CLOSE(fd);
 
-	/* Close the test directory opened in the setup() */
-	SAFE_CLOSE(NULL, fd);
-
-	SAFE_SETEGID(NULL, 0);
-	SAFE_SETEUID(NULL, 0);
-
-	tst_rmdir();
-
+	SAFE_SETEGID(0);
+	SAFE_SETEUID(0);
 }
+
+static struct tst_test test = {
+	.test_all = verify_fchmod,
+	.needs_root = 1,
+	.setup = setup,
+	.cleanup = cleanup,
+	.needs_tmpdir = 1,
+};
Signature · 1 line
1.8.3.1
#2 lore raw

PATCH 2/2 syscalls/fchmod05.c: Cleanup && convert to new API

Hi Jinhui,

Quoted 3 lines
---
 testcases/kernel/syscalls/fchmod/fchmod05.c | 238 ++++++----------------------

thanks for your patch!
Merged, with following diff (variables renamed, _GNU_SOURCE and include not needed).

Kind regards,
Petr

@@ -16,12 +16,7 @@
* the mode of a directory but fails to set setgid bit on it. */

-#ifndef _GNU_SOURCE
-#define _GNU_SOURCE
-#endif
-
#include <pwd.h>
-#include <grp.h>
#include <errno.h>

#include "tst_test.h"
@@ -37,7 +32,7 @@ static void verify_fchmod(void)
mode_t dir_mode;

TEST(fchmod(fd, PERMS_DIR));
- if (TEST_RETURN == -1)
+ if (TST_RET == -1)
tst_res(TFAIL | TTERRNO, "fchmod() failed unexpectly");

SAFE_FSTAT(fd, &stat_buf);
@@ -64,7 +59,7 @@ static void setup(void)

if (setgroups(1, &ltpuser->pw_gid) == -1) {
tst_brk(TBROK, "Couldn't change supplementary group Id: %s",
- tst_strerrno(TEST_ERRNO));
+ tst_strerrno(TST_ERR));
}

SAFE_CHOWN(TESTDIR, ltpuser->pw_uid, ltpgroup->gr_gid);

#3 lore raw

PATCH 1/2 syscalls/fchmod03.c: Convert to new API && merge fchmod04 into fchmod03

In reply to Jinhui huang

Hi!

testcases/kernel/syscalls/fchmod/fchmod03.c line 32

Quoted 4 lines
testcases/kernel/syscalls/fchmod/fchmod03.c +4 -0
+	struct stat stat_buf;
+	mode_t modes;
+
+	for (i = 0; i < 2; i++) {

Huh, wouldn't it be easier if we set .tcnt = 2 in the test structure?

Then the test function would be executed twice and you will get the i as a parameter...

Quoted 27 lines
testcases/kernel/syscalls/fchmod/fchmod03.c +15 -11
+		TEST(fchmod(fd[i], PERMS));
+		if (TEST_RETURN == -1)
+			tst_res(TFAIL | TTERRNO, "fchmod() failed unexpectly");
+
+		SAFE_FSTAT(fd[i], &stat_buf);
+		modes = stat_buf.st_mode;
+
+		if ((modes & PERMS) != PERMS) {
+			tst_res(TFAIL,
+				"%s: Incorrect modes 0%3o, Expected 0%03o",
+				TESTFILE, modes, PERMS);
+		} else {
+			tst_res(TPASS,
+				"Functionality of fchmod(%d, %#o) successful",
+				fd[i], PERMS);
 		}
-		/*
-		 * Get the file information using
-		 * fstat(2).
-		 */
-		if (fstat(fd, &stat_buf) == -1)
-			tst_brkm(TFAIL | TERRNO, cleanup,
-				 "fstat failed");
-		file_mode = stat_buf.st_mode;
-
-		/* Verify STICKY BIT set on testfile */
-		if ((file_mode & PERMS) != PERMS)

testcases/kernel/syscalls/fchmod/fchmod03.c line 50

Quoted 10 lines
testcases/kernel/syscalls/fchmod/fchmod03.c +0 -8
-			tst_resm(TFAIL, "%s: Incorrect modes 0%3o, "
-				 "Expected 0777", TESTFILE, file_mode);
-		else
-			tst_resm(TPASS, "Functionality of fchmod(%d, "
-				 "%#o) successful", fd, PERMS);
 	}
-
-	cleanup();
-	tst_exit();
 }

Also the test only checks for "the process is owner of the file" we should test for the second case the description lists, i.e. "the effective group ID or one of the supplementary group ID's of the process is equal to the group ID of the file." as well, shouldn't we? Well since that does not seem to be part of the original tests it would be better if we added that in a subseqent patch though.

Signature · 2 lines
Cyril Hrubis
[email protected]