From 3ca97e8f0f08c85aecc08f5f171e46729e04911d Mon Sep 17 00:00:00 2001 From: Ulrich Drepper Date: Tue, 17 Dec 2002 06:52:45 +0000 Subject: [PATCH] Update. * sysdeps/posix/system.c (do_system): Add support for installation of cancellation handlers. * sysdeps/unix/sysv/linux/i386/system.c: New file. * stdlib/Makefile (tests): Add tst-system. * stdlib/tst-system.c: New file. --- ChangeLog | 7 +++ stdlib/Makefile | 11 +++-- stdlib/tst-system.c | 31 +++++++++++++ sysdeps/posix/system.c | 12 +++++ sysdeps/unix/sysv/linux/i386/system.c | 67 +++++++++++++++++++++++++++ 5 files changed, 123 insertions(+), 5 deletions(-) create mode 100644 stdlib/tst-system.c create mode 100644 sysdeps/unix/sysv/linux/i386/system.c diff --git a/ChangeLog b/ChangeLog index acf7205eb1..9ba4e4bcb6 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,12 @@ 2002-12-16 Ulrich Drepper + * sysdeps/posix/system.c (do_system): Add support for installation + of cancellation handlers. + * sysdeps/unix/sysv/linux/i386/system.c: New file. + + * stdlib/Makefile (tests): Add tst-system. + * stdlib/tst-system.c: New file. + * sysdeps/unix/sysv/linux/i386/socket.S [NEED_CANCELLATION && CENABLE]: Don't lose content of %ebx. diff --git a/stdlib/Makefile b/stdlib/Makefile index 90b6d8b1c8..4d471c911b 100644 --- a/stdlib/Makefile +++ b/stdlib/Makefile @@ -55,13 +55,14 @@ routines := \ # linked against when the shared library will be used. static-only-routines = atexit -distribute := exit.h grouping.h abort-instr.h isomac.c tst-fmtmsg.sh \ +distribute := exit.h grouping.h abort-instr.h isomac.c tst-fmtmsg.sh \ allocalim.h test-srcs := tst-fmtmsg -tests := tst-strtol tst-strtod testmb testrand testsort testdiv \ - test-canon test-canon2 tst-strtoll tst-environ \ - tst-xpg-basename tst-random tst-bsearch tst-limits \ - tst-rand48 bug-strtod tst-setcontext test-a64l tst-qsort +tests := tst-strtol tst-strtod testmb testrand testsort testdiv \ + test-canon test-canon2 tst-strtoll tst-environ \ + tst-xpg-basename tst-random tst-bsearch tst-limits \ + tst-rand48 bug-strtod tst-setcontext test-a64l tst-qsort \ + tst-system # Several mpn functions from GNU MP are used by the strtod function. diff --git a/stdlib/tst-system.c b/stdlib/tst-system.c new file mode 100644 index 0000000000..e6121c412c --- /dev/null +++ b/stdlib/tst-system.c @@ -0,0 +1,31 @@ +/* Copyright (C) 2002 Free Software Foundation, Inc. + This file is part of the GNU C Library. + Contributed by Ulrich Drepper , 2002. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library 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 + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, write to the Free + Software Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA + 02111-1307 USA. */ + +#include + + +static int +do_test (void) +{ + return system (":"); +} + + +#define TEST_FUNCTION do_test () +#include "../test-skeleton.c" diff --git a/sysdeps/posix/system.c b/sysdeps/posix/system.c index bca1c2ec3e..08852801a6 100644 --- a/sysdeps/posix/system.c +++ b/sysdeps/posix/system.c @@ -112,7 +112,15 @@ do_system (const char *line) } } +#ifdef CLEANUP_HANDLER + CLEANUP_HANDLER; +#endif + +#ifdef FORK + pid = FORK (); +#else pid = __fork (); +#endif if (pid == (pid_t) 0) { /* Child side. */ @@ -158,6 +166,10 @@ do_system (const char *line) #endif } +#ifdef CLEANUP_HANDLER + CLEANUP_RESET; +#endif + save = errno; DO_LOCK (); if ((SUB_REF () == 0 diff --git a/sysdeps/unix/sysv/linux/i386/system.c b/sysdeps/unix/sysv/linux/i386/system.c new file mode 100644 index 0000000000..1107fd6668 --- /dev/null +++ b/sysdeps/unix/sysv/linux/i386/system.c @@ -0,0 +1,67 @@ +/* Copyright (C) 2002 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library 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 + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, write to the Free + Software Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA + 02111-1307 USA. */ + +#include +#include +#include +#include +#include +#include + +/* We have to and actually can handle cancelable system(). The big + problem: we have to kill the child process if necessary. To do + this a cleanup handler has to be registered and is has to be able + to find the PID of the child. The main problem is to reliable have + the PID when needed. It is not necessary for the parent thread to + return. It might still be in the kernel when the cancellation + request comes. Therefore we have to use the clone() calls ability + to have the kernel write the PID into the user-level variable. */ +#define FORK() \ + INLINE_SYSCALL (clone, 3, CLONE_PARENT_SETTID | SIGCHLD, 0, &pid) + +static void cancel_handler (void *arg); + +#define CLEANUP_HANDLER \ + __libc_cleanup_region_start (0, cancel_handler, &pid) + +#define CLEANUP_RESET \ + __libc_cleanup_region_end (0) + + +/* Linux has waitpid(), so override the generic unix version. */ +#include + + +/* The cancellation handler. */ +static void +cancel_handler (void *arg) +{ + __kill (SIGKILL, *(pid_t *) arg); + + TEMP_FAILURE_RETRY (waitpid (*(pid_t *) arg, NULL, 0)); + + DO_LOCK (); + + if (SUB_REF () == 0) + { + (void) __sigaction (SIGQUIT, &quit, (struct sigaction *) NULL); + (void) __sigaction (SIGINT, &intr, (struct sigaction *) NULL); + } + + DO_UNLOCK (); +}