From: Avinesh Kumar <akumar@suse.de>
To: ltp@lists.linux.it
Subject: [LTP] [PATCH] setfsuid02: Rewrite using new LTP API
Date: Wed, 19 Oct 2022 12:47:06 +0530 [thread overview]
Message-ID: <20221019071706.25416-1-akumar@suse.de> (raw)
Rewrite using new API and add setfsuid(-1) call to verify that preceding
setfsuid() call with an invalid fsuid had failed.
Signed-off-by: Avinesh Kumar <akumar@suse.de>
---
.../kernel/syscalls/setfsuid/setfsuid02.c | 96 ++++---------------
1 file changed, 21 insertions(+), 75 deletions(-)
diff --git a/testcases/kernel/syscalls/setfsuid/setfsuid02.c b/testcases/kernel/syscalls/setfsuid/setfsuid02.c
index 5807bb6c0..84f9fb504 100644
--- a/testcases/kernel/syscalls/setfsuid/setfsuid02.c
+++ b/testcases/kernel/syscalls/setfsuid/setfsuid02.c
@@ -1,91 +1,37 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
/*
* Copyright (C) International Business Machines Corp., 2001
* Ported by Wayne Boyer
* Adapted by Dustin Kirkland (k1rkland@us.ibm.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
+ * Copyright (c) 2022 SUSE LLC Avinesh Kumar <avinesh.kumar@suse.com>
*/
-/*
- * Testcase to test the basic functionality of the setfsuid(2) system
- * call to fail on invalid uid.
+/*\
+ * [Description]
+ *
+ * Verify that setfsuid() syscall fails if an invalid fsuid is given.
*/
-#include <stdio.h>
-#include <unistd.h>
#include <pwd.h>
-#include <sys/types.h>
-#include <errno.h>
-
-#include "test.h"
-#include "compat_16.h"
+#include "tst_test.h"
+#include "compat_tst_16.h"
-static void setup(void);
-static void cleanup(void);
-
-TCID_DEFINE(setfsuid02);
-int TST_TOTAL = 1;
-
-int main(int ac, char **av)
+static void run(void)
{
- int lc;
-
- uid_t uid;
-
- tst_parse_opts(ac, av, NULL, NULL);
-
- setup();
+ uid_t invalid_uid, current_uid;
- uid = 1;
- while (getpwuid(uid))
- uid++;
+ current_uid = geteuid();
+ invalid_uid = 1;
+ while (getpwuid(invalid_uid))
+ invalid_uid++;
- UID16_CHECK(uid, setfsuid, cleanup);
+ UID16_CHECK(invalid_uid, setfsuid);
- for (lc = 0; TEST_LOOPING(lc); lc++) {
-
- tst_count = 0;
-
- TEST(SETFSUID(cleanup, uid));
-
- if (TEST_RETURN == -1) {
- tst_resm(TFAIL | TTERRNO,
- "setfsuid() failed unexpectedly");
- continue;
- }
-
- if (TEST_RETURN == uid) {
- tst_resm(TFAIL, "setfsuid() returned %ld, expected %d",
- TEST_RETURN, uid);
- } else {
- tst_resm(TPASS, "setfsuid() returned expected value : "
- "%ld", TEST_RETURN);
- }
- }
-
- cleanup();
- tst_exit();
+ TST_EXP_VAL_SILENT(setfsuid(invalid_uid), current_uid);
+ TST_EXP_VAL(setfsuid(-1), current_uid,
+ "setfsuid(invalid_fsuid) test for expected failure:");
}
-static void setup(void)
-{
- tst_sig(NOFORK, DEF_HANDLER, cleanup);
-
- TEST_PAUSE;
-}
-
-static void cleanup(void)
-{
-}
+static struct tst_test test = {
+ .test_all = run
+};
--
2.37.3
--
Mailing list info: https://lists.linux.it/listinfo/ltp
next reply other threads:[~2022-10-19 7:17 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-19 7:17 Avinesh Kumar [this message]
2022-10-25 7:59 ` [LTP] [PATCH] setfsuid02: Rewrite using new LTP API Richard Palethorpe
2022-10-27 7:47 ` Avinesh Kumar
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20221019071706.25416-1-akumar@suse.de \
--to=akumar@suse.de \
--cc=ltp@lists.linux.it \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox