* [LTP] [PATCH] setfsgid01: Rewrite using new LTP API
@ 2022-10-12 6:49 Avinesh Kumar
2022-10-21 22:58 ` Petr Vorel
0 siblings, 1 reply; 3+ messages in thread
From: Avinesh Kumar @ 2022-10-12 6:49 UTC (permalink / raw)
To: ltp
Rewrite using new API and add test setfsgid(-1) which will always
fail but will determine if the preceding setfsgid() call changed
the filesystem group ID as expected.
Signed-off-by: Avinesh Kumar <akumar@suse.de>
---
.../kernel/syscalls/setfsgid/setfsgid01.c | 97 ++++++-------------
1 file changed, 29 insertions(+), 68 deletions(-)
diff --git a/testcases/kernel/syscalls/setfsgid/setfsgid01.c b/testcases/kernel/syscalls/setfsgid/setfsgid01.c
index 45d89c0cb..b97b21281 100644
--- a/testcases/kernel/syscalls/setfsgid/setfsgid01.c
+++ b/testcases/kernel/syscalls/setfsgid/setfsgid01.c
@@ -1,86 +1,47 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
/*
* Copyright (c) International Business Machines Corp., 2001
* Ported by Wayne Boyer
- *
- * 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 check the basic functionality of setfsgid(2) system call.
+/*\
+ * [Description]
+ *
+ * Verify that setfsgid() correctly updates the filesystem group ID
+ * to the value given in fsgid argument.
*/
-#include <stdio.h>
-#include <unistd.h>
-#include <sys/fsuid.h>
-#include <sys/types.h>
-#include <errno.h>
-
-#include "test.h"
-#include "compat_16.h"
+#include <pwd.h>
-TCID_DEFINE(setfsgid01);
-int TST_TOTAL = 1;
+#include "tst_test.h"
+#include "compat_tst_16.h"
-static void setup(void);
-static void cleanup(void);
+static gid_t nobody_gid;
-int main(int ac, char **av)
+static void setup(void)
{
- int lc;
-
- gid_t gid;
-
- tst_parse_opts(ac, av, NULL, NULL);
-
- setup();
+ struct passwd *nobody;
- for (lc = 0; TEST_LOOPING(lc); lc++) {
- tst_count = 0;
-
- gid = getegid();
- GID16_CHECK(gid, setfsgid, cleanup);
-
- TEST(SETFSGID(cleanup, gid));
-
- if (TEST_RETURN == -1) {
- tst_resm(TFAIL | TTERRNO,
- "setfsgid() failed unexpectedly");
- continue;
- }
-
- if (TEST_RETURN != gid) {
- tst_resm(TFAIL, "setfsgid() returned %ld, expected %d",
- TEST_RETURN, gid);
- } else {
- tst_resm(TPASS,
- "setfsgid() returned expected value : %ld",
- TEST_RETURN);
- }
- }
-
- cleanup();
- tst_exit();
+ nobody = SAFE_GETPWNAM("nobody");
+ nobody_gid = nobody->pw_gid;
}
-static void setup(void)
+static void run(void)
{
- tst_sig(NOFORK, DEF_HANDLER, cleanup);
+ gid_t gid;
- TEST_PAUSE;
-}
+ gid = getegid();
+ GID16_CHECK(gid, setfsgid);
-static void cleanup(void)
-{
+ SAFE_SETEUID(0);
+ TST_EXP_VAL(setfsgid(nobody_gid), gid);
+ TST_EXP_VAL(setfsgid(-1), nobody_gid);
+ TST_EXP_VAL_SILENT(setfsgid(gid), nobody_gid);
}
+
+static struct tst_test test = {
+ .setup = setup,
+ .test_all = run,
+ .needs_root = 1
+};
--
2.37.3
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [LTP] [PATCH] setfsgid01: Rewrite using new LTP API
2022-10-12 6:49 [LTP] [PATCH] setfsgid01: Rewrite using new LTP API Avinesh Kumar
@ 2022-10-21 22:58 ` Petr Vorel
2022-10-25 6:37 ` Avinesh Kumar
0 siblings, 1 reply; 3+ messages in thread
From: Petr Vorel @ 2022-10-21 22:58 UTC (permalink / raw)
To: Avinesh Kumar; +Cc: ltp
Hi Avinesh,
> Rewrite using new API and add test setfsgid(-1) which will always
> fail but will determine if the preceding setfsgid() call changed
> the filesystem group ID as expected.
+1
Thanks, merged with fix below.
> - gid = getegid();
> - GID16_CHECK(gid, setfsgid, cleanup);
> -
> - TEST(SETFSGID(cleanup, gid));
> -
> - if (TEST_RETURN == -1) {
> - tst_resm(TFAIL | TTERRNO,
> - "setfsgid() failed unexpectedly");
> - continue;
> - }
> -
> - if (TEST_RETURN != gid) {
> - tst_resm(TFAIL, "setfsgid() returned %ld, expected %d",
> - TEST_RETURN, gid);
> - } else {
> - tst_resm(TPASS,
> - "setfsgid() returned expected value : %ld",
> - TEST_RETURN);
> - }
> - }
...
> + gid = getegid();
> + GID16_CHECK(gid, setfsgid);
> -static void cleanup(void)
> -{
> + SAFE_SETEUID(0);
> + TST_EXP_VAL(setfsgid(nobody_gid), gid);
> + TST_EXP_VAL(setfsgid(-1), nobody_gid);
> + TST_EXP_VAL_SILENT(setfsgid(gid), nobody_gid);
NOTE: all of these needed to be SETFSGID(), otherwise setfsgid01_16 would
happily test 32-bit version instead of 16-bit version.
Kind regards,
Petr
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [LTP] [PATCH] setfsgid01: Rewrite using new LTP API
2022-10-21 22:58 ` Petr Vorel
@ 2022-10-25 6:37 ` Avinesh Kumar
0 siblings, 0 replies; 3+ messages in thread
From: Avinesh Kumar @ 2022-10-25 6:37 UTC (permalink / raw)
To: Petr Vorel; +Cc: ltp
Hi Petr,
On Saturday, October 22, 2022 4:28:39 AM IST Petr Vorel wrote:
> Hi Avinesh,
>
> > Rewrite using new API and add test setfsgid(-1) which will always
> > fail but will determine if the preceding setfsgid() call changed
> > the filesystem group ID as expected.
> +1
>
> Thanks, merged with fix below.
>
> > - gid = getegid();
> > - GID16_CHECK(gid, setfsgid, cleanup);
> > -
> > - TEST(SETFSGID(cleanup, gid));
> > -
> > - if (TEST_RETURN == -1) {
> > - tst_resm(TFAIL | TTERRNO,
> > - "setfsgid() failed unexpectedly");
> > - continue;
> > - }
> > -
> > - if (TEST_RETURN != gid) {
> > - tst_resm(TFAIL, "setfsgid() returned %ld, expected %d",
> > - TEST_RETURN, gid);
> > - } else {
> > - tst_resm(TPASS,
> > - "setfsgid() returned expected value : %ld",
> > - TEST_RETURN);
> > - }
> > - }
> ...
> > + gid = getegid();
> > + GID16_CHECK(gid, setfsgid);
>
> > -static void cleanup(void)
> > -{
> > + SAFE_SETEUID(0);
> > + TST_EXP_VAL(setfsgid(nobody_gid), gid);
> > + TST_EXP_VAL(setfsgid(-1), nobody_gid);
> > + TST_EXP_VAL_SILENT(setfsgid(gid), nobody_gid);
>
> NOTE: all of these needed to be SETFSGID(), otherwise setfsgid01_16 would
> happily test 32-bit version instead of 16-bit version.
Thank you for this correction, will keep 16-bit thing in mind.
>
> Kind regards,
> Petr
>
Regards,
Avinesh
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-10-25 6:37 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-10-12 6:49 [LTP] [PATCH] setfsgid01: Rewrite using new LTP API Avinesh Kumar
2022-10-21 22:58 ` Petr Vorel
2022-10-25 6:37 ` Avinesh Kumar
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.