Hi Sharyathi, This is the forwarded mail containing the patch for chmod05 in new LTP provided by Wei Yongjun <[email protected]>. Is this the functionality you are also looking for and trying to fix?
Thanks Muni
--- Begin Message ---Refer to the manpage: # man 2 chmod If the calling process is not privileged (Linux: does not have the CAP_FSETID capability), and the group of the file does not match the effective group ID of the process or one of its supplementary group IDs, the S_ISGID bit will be turned off, but this will not cause an error to be returned. So, if we want S_ISGID bit be turned off after chmod(), we can not have the CAP_FSETID capability and not match the effective group ID. The 'bin' group always has the CAP_FSETID capability, so we can not change the own of the TESTDIR to 'bin' group, instead, 'nobody' can be used. This patch fixed the problem by change gid of chown to 'nobody' group and change the gid of setegid() to 'bin' group. Signed-off-by: Wei Yongjun <[email protected]> --- testcases/kernel/syscalls/chmod/chmod05.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/testcases/kernel/syscalls/chmod/chmod05.c b/testcases/kernel/syscalls/chmod/chmod05.c index 4504aaa..c6f1225 100644 --- a/testcases/kernel/syscalls/chmod/chmod05.c +++ b/testcases/kernel/syscalls/chmod/chmod05.c @@ -177,7 +177,7 @@ int main(int ac, char **av) if ((PERMS & ~S_ISGID) != dir_mode) { tst_resm(TFAIL, "%s: Incorrect modes 0%03o, " "Expected 0%03o", TESTDIR, dir_mode, - PERMS); + PERMS & ~S_ISGID); } else { tst_resm(TPASS, "Functionality of chmod(%s, %#o) successful", @@ -241,12 +241,12 @@ void setup() strerror(errno)); } - if (chown(TESTDIR, nobody_u->pw_uid, bin_group->gr_gid) == -1) + if (chown(TESTDIR, nobody_u->pw_uid, nobody_u->pw_gid) == -1) tst_brkm(TBROK, cleanup, "Couldn't change owner of testdir: %s", strerror(errno)); - /* change to nobody:nobody */ - if (setegid(nobody_u->pw_gid) == -1 || + /* change to nobody:bin */ + if (setegid(bin_group->gr_gid) == -1 || seteuid(nobody_u->pw_uid) == -1) tst_brkm(TBROK, cleanup, "Couldn't switch to nobody:nobody: %s", strerror(errno)); -- 1.6.0.2.530.g67faa ------------------------------------------------------------------------------ OpenSolaris 2009.06 is a cutting edge operating system for enterprises looking to deploy the next generation of Solaris that includes the latest innovations from Sun and the OpenSource community. Download a copy and enjoy capabilities such as Networking, Storage and Virtualization. Go to: http://p.sf.net/sfu/opensolaris-get _______________________________________________ Ltp-list mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ltp-list
--- End Message ---
------------------------------------------------------------------------------ Enter the BlackBerry Developer Challenge This is your chance to win up to $100,000 in prizes! For a limited time, vendors submitting new applications to BlackBerry App World(TM) will have the opportunity to enter the BlackBerry Developer Challenge. See full prize details at: http://p.sf.net/sfu/Challenge
_______________________________________________ Ltp-list mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ltp-list
