On Tue, 2009-05-12 at 19:06 +0200, Jiri Palecek wrote:
> Cleanup has to be made only once in the parent process; when the
> child makes the cleanup itself, the parent has nothing to cleanup,
> which makes him sad.
>
> Signed-off-by: Jiri Palecek <[email protected]>
Thanks.
Regards--
Subrata
> ---
> testcases/kernel/syscalls/setuid/setuid04.c | 7 +++----
> 1 files changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/testcases/kernel/syscalls/setuid/setuid04.c
> b/testcases/kernel/syscalls/setuid/setuid04.c
> index 18f1d90..26a3868 100644
> --- a/testcases/kernel/syscalls/setuid/setuid04.c
> +++ b/testcases/kernel/syscalls/setuid/setuid04.c
> @@ -114,7 +114,7 @@ void do_master_child()
> int status;
>
> if (setuid(ltpuser->pw_uid) == -1) {
> - tst_brkm(TBROK, cleanup,
> + tst_brkm(TBROK, tst_exit,
> "setuid failed to set the effective uid to %d",
> ltpuser->pw_uid);
> }
> @@ -143,7 +143,7 @@ void do_master_child()
>
> pid = FORK_OR_VFORK();
> if (pid < 0)
> - tst_brkm(TBROK, cleanup, "Fork failed");
> + tst_brkm(TBROK, tst_exit, "Fork failed");
>
> if (pid == 0) {
> int tst_fd2;
> @@ -160,13 +160,12 @@ void do_master_child()
>
> if (TEST_ERRNO == EACCES) {
> tst_resm(TPASS, "open returned errno EACCES");
> - exit(0);
> } else {
> tst_resm(TFAIL,
> "open returned unexpected errno - %d",
> TEST_ERRNO);
> - exit(-1);
> }
> + tst_exit();
> } else {
> /* Wait for son completion */
> waitpid(pid, &status, 0);
------------------------------------------------------------------------------
Are you an open source citizen? Join us for the Open Source Bridge conference!
Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250.
Need another reason to go? 24-hour hacker lounge. Register today!
http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list
