diff options
Diffstat (limited to 'libio')
49 files changed, 257 insertions, 124 deletions
diff --git a/libio/Makefile b/libio/Makefile index e4df6b545e..355a79a0e5 100644 --- a/libio/Makefile +++ b/libio/Makefile @@ -36,6 +36,15 @@ routines := \ \ libc_fatal +include ../Makeconfig + +ifneq (,$(filter %REENTRANT, $(defines))) +routines += clearerr_u feof_u ferror_u fputc_u getc_u getchar_u \ + iofflush_u putc_u putchar_u ioflockfile + +CPPFLAGS += -D_IO_MTSAFE_IO +endif + aux := \ cleanup fileops genops stdfiles stdio strops diff --git a/libio/clearerr.c b/libio/clearerr.c index 669a9b0d74..9dfdb11530 100644 --- a/libio/clearerr.c +++ b/libio/clearerr.c @@ -24,9 +24,9 @@ clearerr (fp) FILE *fp; { CHECK_FILE (fp, /*nothing*/); - flockfile (fp); + _IO_flockfile (fp); _IO_clearerr (fp); - funlockfile (fp); + _IO_funlockfile (fp); } #ifdef _IO_MTSAFE_IO diff --git a/libio/clearerr_u.c b/libio/clearerr_u.c index 83ed65d8b1..4c8b6c29bd 100644 --- a/libio/clearerr_u.c +++ b/libio/clearerr_u.c @@ -27,4 +27,4 @@ __clearerr_unlocked (fp) _IO_clearerr (fp); } -weak_alias (clearerr_unlocked, __clearerr_unlocked) +weak_alias (__clearerr_unlocked, clearerr_unlocked) diff --git a/libio/feof_u.c b/libio/feof_u.c index 5ce5583613..6ed8b7d4fe 100644 --- a/libio/feof_u.c +++ b/libio/feof_u.c @@ -24,6 +24,8 @@ the executable file might be covered by the GNU General Public License. */ #include "libioP.h" #include "stdio.h" +#undef feof_unlocked + int feof_unlocked (fp) _IO_FILE* fp; diff --git a/libio/ferror_u.c b/libio/ferror_u.c index bee668d6d9..df550d9d4d 100644 --- a/libio/ferror_u.c +++ b/libio/ferror_u.c @@ -24,6 +24,8 @@ the executable file might be covered by the GNU General Public License. */ #include "libioP.h" #include "stdio.h" +#undef ferror_unlocked + int ferror_unlocked (fp) _IO_FILE* fp; diff --git a/libio/fgetc.c b/libio/fgetc.c index 2aaea05cfe..07fd89a516 100644 --- a/libio/fgetc.c +++ b/libio/fgetc.c @@ -31,8 +31,9 @@ fgetc (fp) { int result; CHECK_FILE (fp, EOF); - flockfile (fp); + __libc_cleanup_region_start (_IO_funlockfile, fp); + _IO_flockfile (fp); result = _IO_getc_unlocked (fp); - funlockfile (fp); + __libc_cleanup_region_end (1); return result; } diff --git a/libio/fileno.c b/libio/fileno.c index 2dc67149c8..08e657b970 100644 --- a/libio/fileno.c +++ b/libio/fileno.c @@ -42,6 +42,6 @@ fileno (fp) it only accesses once a single variable and this is already atomic (at least at thread level). */ -weak_alias (fileno_unlocked, fileno) -weak_alias (fileno_locked, fileno) +weak_alias (fileno, fileno_unlocked) +weak_alias (fileno, fileno_locked) #endif diff --git a/libio/fputc.c b/libio/fputc.c index 339861119f..bd871ab047 100644 --- a/libio/fputc.c +++ b/libio/fputc.c @@ -32,8 +32,11 @@ fputc (c, fp) { int result; CHECK_FILE (fp, EOF); + __libc_cleanup_region_start (&_IO_funlockfile, fp); _IO_flockfile (fp); result = _IO_putc_unlocked (c, fp); - _IO_funlockfile (fp); + __libc_cleanup_region_end (1); return result; } + +weak_alias (fputc, fputc_locked) diff --git a/libio/fputc_u.c b/libio/fputc_u.c index 211e206605..97e4d94eb3 100644 --- a/libio/fputc_u.c +++ b/libio/fputc_u.c @@ -36,4 +36,4 @@ __fputc_unlocked (c, fp) return _IO_putc_unlocked (c, fp); } -weak_alias (fputc_unlocked, __fputc_unlocked) +weak_alias (__fputc_unlocked, fputc_unlocked) diff --git a/libio/freopen.c b/libio/freopen.c index 829af31dc1..5821e84c6f 100644 --- a/libio/freopen.c +++ b/libio/freopen.c @@ -35,8 +35,9 @@ freopen (filename, mode, fp) CHECK_FILE (fp, NULL); if (!(fp->_flags & _IO_IS_FILEBUF)) return NULL; - flockfile (fp); + __libc_cleanup_region_start (&_IO_funlockfile, fp); + _IO_flockfile (fp); result = _IO_freopen (filename, mode, fp); - funlockfile (fp); + __libc_cleanup_region_end (1); return result; } diff --git a/libio/fseek.c b/libio/fseek.c index 5255e9defe..c33927a95f 100644 --- a/libio/fseek.c +++ b/libio/fseek.c @@ -1,5 +1,5 @@ /* -Copyright (C) 1993, 1995 Free Software Foundation +Copyright (C) 1993, 1995, 1996 Free Software Foundation, Inc. This file is part of the GNU IO Library. This library is free software; you can redistribute it and/or modify it under the @@ -22,8 +22,8 @@ the resulting executable to be covered by the GNU General Public License. This exception does not however invalidate any other reasons why the executable file might be covered by the GNU General Public License. */ -#include "stdio.h" #include "libioP.h" +#include "stdio.h" int fseek (fp, offset, whence) @@ -33,8 +33,9 @@ fseek (fp, offset, whence) { int result; CHECK_FILE (fp, -1); - flockfile (fp); + __libc_cleanup_region_start (&_IO_funlockfile, fp); + _IO_flockfile (fp); result = _IO_fseek (fp, offset, whence); - funlockfile (fp); + __libc_cleanup_region_end (1); return result; } diff --git a/libio/genops.c b/libio/genops.c index 98eef94b58..818e740851 100644 --- a/libio/genops.c +++ b/libio/genops.c @@ -466,7 +466,7 @@ DEFUN(_IO_init, (fp, flags), fp->_markers = NULL; fp->_cur_column = 0; #ifdef _IO_MTSAFE_IO - _IO_mutex_init (fp->_lock); + __libc_lock_init (fp->_lock); #endif } @@ -501,7 +501,7 @@ DEFUN(_IO_default_finish, (fp), } #ifdef _IO_MTSAFE_IO - _IO_mutex_destroy (fp->_lock); + __libc_lock_fini (fp->_lock); #endif _IO_un_link(fp); diff --git a/libio/getc.c b/libio/getc.c index 89abc0dac0..82e5756ad3 100644 --- a/libio/getc.c +++ b/libio/getc.c @@ -33,14 +33,15 @@ getc (fp) { int result; CHECK_FILE (fp, EOF); + __libc_cleanup_region_start (&_IO_funlockfile, fp); _IO_flockfile (fp); result = _IO_getc_unlocked (fp); - _IO_funlockfile (fp); + __libc_cleanup_region_end (1); return result; } #ifdef _IO_MTSAFE_IO # undef getc_locked -weak_alias (getc_locked, getc) +weak_alias (getc, getc_locked) #endif diff --git a/libio/getc_u.c b/libio/getc_u.c index e6491bb944..4aa5cd5ccd 100644 --- a/libio/getc_u.c +++ b/libio/getc_u.c @@ -35,4 +35,4 @@ __getc_unlocked (fp) return _IO_getc_unlocked (fp); } -weak_alias (getc_unlocked, __getc_unlocked) +weak_alias (__getc_unlocked, getc_unlocked) diff --git a/libio/getchar.c b/libio/getchar.c index fa5c11f731..e7a236cf2e 100644 --- a/libio/getchar.c +++ b/libio/getchar.c @@ -31,14 +31,15 @@ int getchar () { int result; + __libc_cleanup_region_start (&_IO_funlockfile, stdin); _IO_flockfile (stdin); result = _IO_getc_unlocked (stdin); - _IO_funlockfile (stdin); + __libc_cleanup_region_end (1); return result; } #ifdef _IO_MTSAFE_IO # undef getchar_locked -weak_alias (getchar_locked, getchar) +weak_alias (getchar, getchar_locked) #endif diff --git a/libio/getchar_u.c b/libio/getchar_u.c index 40cfbf6625..0430f8119c 100644 --- a/libio/getchar_u.c +++ b/libio/getchar_u.c @@ -33,4 +33,4 @@ __getchar_unlocked () return _IO_getc_unlocked (stdin); } -weak_alias (getchar_unlocked, __getchar_unlocked) +weak_alias (__getchar_unlocked, getchar_unlocked) diff --git a/libio/iofclose.c b/libio/iofclose.c index e5cae966fd..9d537377a4 100644 --- a/libio/iofclose.c +++ b/libio/iofclose.c @@ -35,6 +35,7 @@ _IO_fclose (fp) CHECK_FILE(fp, EOF); + __libc_cleanup_region_start (&_IO_funlockfile, fp); _IO_flockfile (fp); if (fp->_IO_file_flags & _IO_IS_FILEBUF) status = _IO_file_close_it (fp); @@ -46,7 +47,7 @@ _IO_fclose (fp) fp->_IO_file_flags = 0; free(fp); } - _IO_funlockfile (fp); + __libc_cleanup_region_end (1); return status; } diff --git a/libio/iofdopen.c b/libio/iofdopen.c index 6bb19fa214..c49387b1a0 100644 --- a/libio/iofdopen.c +++ b/libio/iofdopen.c @@ -39,7 +39,11 @@ _IO_fdopen (fd, mode) { int read_write; int posix_mode = 0; - struct _IO_FILE_plus *fp; + struct locked_FILE + { + struct _IO_FILE_plus fp; + _IO_lock_t lock; + } *new_f; int fd_flags; switch (*mode++) @@ -97,28 +101,29 @@ _IO_fdopen (fd, mode) } #endif - fp = (struct _IO_FILE_plus *) malloc (sizeof (struct _IO_FILE_plus)); - if (fp == NULL) + new_f = (struct locked_FILE *) malloc (sizeof (struct locked_FILE)); + if (new_f == NULL) return NULL; - _IO_init (&fp->file, 0); - _IO_JUMPS (&fp->file) = &_IO_file_jumps; - _IO_file_init (&fp->file); + new_f->fp.file._lock = &new_f->lock; + _IO_init (&new_f->fp.file, 0); + _IO_JUMPS (&new_f->fp.file) = &_IO_file_jumps; + _IO_file_init (&new_f->fp.file); #if !_IO_UNIFIED_JUMPTABLES - fp->vtable = NULL; + new_f->fp.vtable = NULL; #endif - if (_IO_file_attach (&fp->file, fd) == NULL) + if (_IO_file_attach (&new_f->fp.file, fd) == NULL) { - _IO_un_link (&fp->file); - free (fp); + _IO_un_link (&new_f->fp.file); + free (new_f); return NULL; } - fp->file._flags &= ~_IO_DELETE_DONT_CLOSE; + new_f->fp.file._flags &= ~_IO_DELETE_DONT_CLOSE; - fp->file._IO_file_flags = - _IO_mask_flags (&fp->file, read_write, + new_f->fp.file._IO_file_flags = + _IO_mask_flags (&new_f->fp.file, read_write, _IO_NO_READS+_IO_NO_WRITES+_IO_IS_APPENDING); - return (_IO_FILE *) fp; + return (_IO_FILE *) &new_f->fp; } weak_alias (_IO_fdopen, fdopen) diff --git a/libio/iofflush.c b/libio/iofflush.c index 6fe2d5262d..cbc5b1f6d7 100644 --- a/libio/iofflush.c +++ b/libio/iofflush.c @@ -1,5 +1,5 @@ /* -Copyright (C) 1993, 1995 Free Software Foundation +Copyright (C) 1993, 1995, 1996 Free Software Foundation, Inc. This file is part of the GNU IO Library. This library is free software; you can redistribute it and/or modify it under the @@ -33,10 +33,11 @@ _IO_fflush (fp) else { int result; - _IO_flockfile (fp); CHECK_FILE (fp, EOF); + __libc_cleanup_region_start (&_IO_funlockfile, fp); + _IO_flockfile (fp); result = _IO_SYNC (fp) ? EOF : 0; - _IO_funlockfile (fp); + __libc_cleanup_region_end (1); return result; } } diff --git a/libio/iofgetpos.c b/libio/iofgetpos.c index 50014765f9..c45cfacb06 100644 --- a/libio/iofgetpos.c +++ b/libio/iofgetpos.c @@ -1,5 +1,5 @@ /* -Copyright (C) 1993, 1995 Free Software Foundation +Copyright (C) 1993, 1995, 1996 Free Software Foundation, Inc. This file is part of the GNU IO Library. This library is free software; you can redistribute it and/or modify it under the @@ -33,9 +33,10 @@ _IO_fgetpos (fp, posp) { _IO_fpos_t pos; CHECK_FILE (fp, EOF); + __libc_cleanup_region_start (&_IO_funlockfile, fp); _IO_flockfile (fp); pos = _IO_seekoff (fp, 0, _IO_seek_cur, 0); - _IO_funlockfile (fp); + __libc_cleanup_region_end (1); if (pos == _IO_pos_BAD) { #ifdef EIO diff --git a/libio/iofgets.c b/libio/iofgets.c index 88d4bd518e..369251f22a 100644 --- a/libio/iofgets.c +++ b/libio/iofgets.c @@ -35,6 +35,7 @@ _IO_fgets (buf, n, fp) CHECK_FILE (fp, NULL); if (n <= 0) return NULL; + __libc_cleanup_region_start (&_IO_funlockfile, fp); _IO_flockfile (fp); count = _IO_getline (fp, buf, n - 1, '\n', 1); if (count == 0 || (fp->_IO_file_flags & _IO_ERR_SEEN)) @@ -44,7 +45,7 @@ _IO_fgets (buf, n, fp) buf[count] = '\0'; result = buf; } - _IO_funlockfile (fp); + __libc_cleanup_region_end (1); return result; } diff --git a/libio/ioflockfile.c b/libio/ioflockfile.c new file mode 100644 index 0000000000..a68f4b662d --- /dev/null +++ b/libio/ioflockfile.c @@ -0,0 +1,43 @@ +/* +Copyright (C) 1996 Free Software Foundation + +This file is part of the GNU IO Library. This library 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, or (at your option) +any later version. + +This 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 General Public License for more details. + +You should have received a copy of the GNU General Public License +along with this library; see the file COPYING. If not, write to the Free +Software Foundation, 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. + +As a special exception, if you link this library with files +compiled with a GNU compiler to produce an executable, this does not cause +the resulting executable to be covered by the GNU General Public License. +This exception does not however invalidate any other reasons why +the executable file might be covered by the GNU General Public License. */ + +#include "libioP.h" +#include <libc-lock.h> + +void +_IO_flockfile (s) + _IO_FILE *s; +{ + __libc_lock_lock (s->_lock); +} +weak_alias (_IO_flockfile, flockfile) + + +void +_IO_funlockfile (s) + _IO_FILE *s; +{ + __libc_lock_unlock (s->_lock); +} +weak_alias (_IO_funlockfile, funlockfile) diff --git a/libio/iofopen.c b/libio/iofopen.c index 14cce5336a..334df4652a 100644 --- a/libio/iofopen.c +++ b/libio/iofopen.c @@ -32,20 +32,25 @@ _IO_fopen (filename, mode) const char *filename; const char *mode; { - struct _IO_FILE_plus *fp = - (struct _IO_FILE_plus *) malloc (sizeof (struct _IO_FILE_plus)); - if (fp == NULL) + struct locked_FILE + { + struct _IO_FILE_plus fp; + _IO_lock_t lock; + } *new_f = (struct locked_FILE *) malloc (sizeof (struct locked_FILE)); + + if (new_f == NULL) return NULL; - _IO_init (&fp->file, 0); - _IO_JUMPS (&fp->file) = &_IO_file_jumps; - _IO_file_init (&fp->file); + new_f->fp.file._lock = &new_f->lock; + _IO_init (&new_f->fp.file, 0); + _IO_JUMPS (&new_f->fp.file) = &_IO_file_jumps; + _IO_file_init (&new_f->fp.file); #if !_IO_UNIFIED_JUMPTABLES - fp->vtable = NULL; + new_f->fp.vtable = NULL; #endif - if (_IO_file_fopen (&fp->file, filename, mode) != NULL) - return (_IO_FILE *) fp; - _IO_un_link (&fp->file); - free (fp); + if (_IO_file_fopen (&new_f->fp.file, filename, mode) != NULL) + return (_IO_FILE *) &new_f->fp; + _IO_un_link (&new_f->fp.file); + free (new_f); return NULL; } diff --git a/libio/iofopncook.c b/libio/iofopncook.c index 989f6d6fe7..4193c2447d 100644 --- a/libio/iofopncook.c +++ b/libio/iofopncook.c @@ -128,7 +128,11 @@ fopencookie (cookie, mode, io_functions) _IO_cookie_io_functions_t io_functions; { int read_write; - struct _IO_cookie_file *cfile; + struct locked_FILE + { + struct _IO_cookie_file cfile; + _IO_lock_t lock; + } *new_f; switch (*mode++) { @@ -147,21 +151,21 @@ fopencookie (cookie, mode, io_functions) if (mode[0] == '+' || (mode[0] == 'b' && mode[1] == '+')) read_write &= _IO_IS_APPENDING; - cfile = (struct _IO_cookie_file *) malloc (sizeof (struct _IO_cookie_file)); - if (cfile == NULL) + new_f = (struct locked_FILE *) malloc (sizeof (struct locked_FILE)); + if (new_f == NULL) return NULL; + new_f->cfile.file._lock = &new_f->lock; - _IO_init (&cfile->file, 0); - _IO_JUMPS (&cfile->file) = &_IO_cookie_jumps; - cfile->cookie = cookie; - cfile->io_functions = io_functions; + _IO_init (&new_f->cfile.file, 0); + _IO_JUMPS (&new_f->cfile.file) = &_IO_cookie_jumps; + new_f->cfile.cookie = cookie; + new_f->cfile.io_functions = io_functions; - _IO_file_init(&cfile->file); + _IO_file_init(&new_f->cfile.file); - cfile->file._IO_file_flags = - _IO_mask_flags (&cfile->file, read_write, + new_f->cfile.file._IO_file_flags = + _IO_mask_flags (&new_f->cfile.file, read_write, _IO_NO_READS+_IO_NO_WRITES+_IO_IS_APPENDING); - return &cfile->file; + return &new_f->cfile.file; } - diff --git a/libio/iofputs.c b/libio/iofputs.c index 3359515ac8..a9fd5f29a3 100644 --- a/libio/iofputs.c +++ b/libio/iofputs.c @@ -33,12 +33,13 @@ _IO_fputs (str, fp) _IO_size_t len = strlen (str); int result; CHECK_FILE (fp, EOF); + __libc_cleanup_region_start (&_IO_funlockfile, fp); _IO_flockfile (fp); if (_IO_sputn (fp, str, len) != len) result = EOF; else result = 1; - _IO_funlockfile (fp); + __libc_cleanup_region_end (1); return result; } diff --git a/libio/iofread.c b/libio/iofread.c index bad94ca17b..af6bc33f24 100644 --- a/libio/iofread.c +++ b/libio/iofread.c @@ -36,9 +36,10 @@ _IO_fread (buf, size, count, fp) CHECK_FILE (fp, 0); if (bytes_requested == 0) return 0; + __libc_cleanup_region_start (&_IO_funlockfile, fp); _IO_flockfile (fp); bytes_read = _IO_sgetn (fp, (char *) buf, bytes_requested); - _IO_funlockfile (fp); + __libc_cleanup_region_end (1); return bytes_requested == bytes_read ? count : bytes_read / size; } weak_alias (_IO_fread, fread) diff --git a/libio/iofsetpos.c b/libio/iofsetpos.c index c8897522d6..96c3a7a2a0 100644 --- a/libio/iofsetpos.c +++ b/libio/iofsetpos.c @@ -32,6 +32,7 @@ _IO_fsetpos (fp, posp) { int result; CHECK_FILE (fp, EOF); + __libc_cleanup_region_start (&_IO_funlockfile, fp); _IO_flockfile (fp); if (_IO_seekpos (fp, *posp, _IOS_INPUT|_IOS_OUTPUT) == _IO_pos_BAD) { @@ -44,7 +45,7 @@ _IO_fsetpos (fp, posp) } else result = 0; - _IO_funlockfile (fp); + __libc_cleanup_region_end (1); return result; } diff --git a/libio/ioftell.c b/libio/ioftell.c index d49f8c3553..12b90b37f5 100644 --- a/libio/ioftell.c +++ b/libio/ioftell.c @@ -1,5 +1,5 @@ /* -Copyright (C) 1993, 1995 Free Software Foundation +Copyright (C) 1993, 1995, 1996 Free Software Foundation, Inc. This file is part of the GNU IO Library. This library is free software; you can redistribute it and/or modify it under the @@ -32,9 +32,10 @@ _IO_ftell (fp) { _IO_pos_t pos; CHECK_FILE (fp, -1L); + __libc_cleanup_region_start (&_IO_funlockfile, fp); _IO_flockfile (fp); pos = _IO_seekoff (fp, 0, _IO_seek_cur, 0); - _IO_funlockfile (fp); + __libc_cleanup_region_end (1); if (pos == _IO_pos_BAD) { #ifdef EIO diff --git a/libio/iofwrite.c b/libio/iofwrite.c index 0bbeef2646..4ee489d2b1 100644 --- a/libio/iofwrite.c +++ b/libio/iofwrite.c @@ -1,5 +1,5 @@ /* -Copyright (C) 1993 Free Software Foundation +Copyright (C) 1993, 1996 Free Software Foundation, Inc. This file is part of the GNU IO Library. This library is free software; you can redistribute it and/or modify it under the @@ -36,9 +36,10 @@ _IO_fwrite (buf, size, count, fp) CHECK_FILE (fp, 0); if (request == 0) return 0; + __libc_cleanup_region_start (&_IO_funlockfile, fp); _IO_flockfile (fp); written = _IO_sputn (fp, (const char *) buf, request); - _IO_funlockfile (fp); + __libc_cleanup_region_end (1); /* Many traditional implementations return 0 if size==0 && count > 0, but ANSI seems to require us to return count in this case. */ if (written == request) diff --git a/libio/iogetdelim.c b/libio/iogetdelim.c index 1d14551615..0acd76e6c5 100644 --- a/libio/iogetdelim.c +++ b/libio/iogetdelim.c @@ -1,5 +1,5 @@ /* -Copyright (C) 1994 Free Software Foundation +Copyright (C) 1994, 1996 Free Software Foundation, Inc. This file is part of the GNU IO Library. This library is free software; you can redistribute it and/or modify it under the @@ -54,6 +54,7 @@ _IO_getdelim (lineptr, n, delimiter, fp) return -1; } CHECK_FILE (fp, -1); + __libc_cleanup_region_start (&_IO_funlockfile, fp); _IO_flockfile (fp); if (_IO_ferror_unlocked (fp)) { @@ -115,7 +116,7 @@ _IO_getdelim (lineptr, n, delimiter, fp) result = cur_len; unlock_return: - _IO_funlockfile (fp); + __libc_cleanup_region_end (1); return result; } diff --git a/libio/iogets.c b/libio/iogets.c index 352e80244b..8fdc4350a9 100644 --- a/libio/iogets.c +++ b/libio/iogets.c @@ -1,5 +1,5 @@ /* -Copyright (C) 1993 Free Software Foundation +Copyright (C) 1993, 1996 Free Software Foundation, Inc. This file is part of the GNU IO Library. This library is free software; you can redistribute it and/or modify it under the @@ -32,6 +32,7 @@ _IO_gets (buf) _IO_size_t count; int ch; + __libc_cleanup_region_start (&_IO_funlockfile, _IO_stdin); _IO_flockfile (_IO_stdin); ch = _IO_getc_unlocked (_IO_stdin); if (ch == EOF) @@ -45,7 +46,7 @@ _IO_gets (buf) if (_IO_stdin->_IO_file_flags & _IO_ERR_SEEN) return NULL; } - _IO_funlockfile (_IO_stdin); + __libc_cleanup_region_end (1); buf[count] = 0; return buf; } diff --git a/libio/iopopen.c b/libio/iopopen.c index 5b9bd4844d..ca2a18edac 100644 --- a/libio/iopopen.c +++ b/libio/iopopen.c @@ -149,10 +149,18 @@ _IO_FILE * DEFUN(_IO_popen, (command, mode), const char *command AND const char *mode) { - _IO_proc_file *fpx = (_IO_proc_file*)malloc(sizeof(_IO_proc_file)); - _IO_FILE *fp = (_IO_FILE*)fpx; - if (fp == NULL) + struct locked_FILE + { + struct _IO_proc_file fpx; + _IO_lock_t lock; + } *new_f; + _IO_FILE *fp; + + new_f = (struct locked_FILE *) malloc (sizeof (struct locked_FILE)); + if (new_f == NULL) return NULL; + new_f->fpx.file.file._lock = &new_f->lock; + fp = (_IO_FILE*)&new_f->fpx; _IO_init(fp, 0); _IO_JUMPS(fp) = &_IO_proc_jumps; _IO_file_init(fp); @@ -161,7 +169,7 @@ DEFUN(_IO_popen, (command, mode), #endif if (_IO_proc_open (fp, command, mode) != NULL) return fp; - free (fpx); + free (new_f); return NULL; } diff --git a/libio/ioputs.c b/libio/ioputs.c index 6c52f7521c..2770ab2935 100644 --- a/libio/ioputs.c +++ b/libio/ioputs.c @@ -30,13 +30,14 @@ _IO_puts (str) { int result; _IO_size_t len = strlen (str); - _IO_flockfile (fp); + __libc_cleanup_region_start (&_IO_funlockfile, _IO_stdout); + _IO_flockfile (_IO_stdout); if (_IO_sputn (_IO_stdout, str, len) == len && _IO_putc_unlocked ('\n', _IO_stdout) != EOF) result = len + 1; else result = EOF; - _IO_funlockfile (fp); + __libc_cleanup_region_end (1); return result; } weak_alias (_IO_puts, puts) diff --git a/libio/iosetbuffer.c b/libio/iosetbuffer.c index d3fc7e5917..e44f33962f 100644 --- a/libio/iosetbuffer.c +++ b/libio/iosetbuffer.c @@ -31,12 +31,13 @@ _IO_setbuffer (fp, buf, size) _IO_size_t size; { CHECK_FILE (fp, ); + __libc_cleanup_region_start (&_IO_funlockfile, fp); _IO_flockfile (fp); fp->_flags &= ~_IO_LINE_BUF; if (!buf) size = 0; (void) _IO_SETBUF (fp, buf, size); - _IO_funlockfile (fp); + __libc_cleanup_region_end (1); } weak_alias (_IO_setbuffer, setbuffer) diff --git a/libio/iosetvbuf.c b/libio/iosetvbuf.c index 6d4bcff2bc..e9dccae529 100644 --- a/libio/iosetvbuf.c +++ b/libio/iosetvbuf.c @@ -1,5 +1,5 @@ /* -Copyright (C) 1993 Free Software Foundation +Copyright (C) 1993, 1996 Free Software Foundation, Inc. This file is part of the GNU IO Library. This library is free software; you can redistribute it and/or modify it under the @@ -37,6 +37,7 @@ _IO_setvbuf (fp, buf, mode, size) { int result; CHECK_FILE (fp, EOF); + __libc_cleanup_region_start (&_IO_funlockfile, fp); _IO_flockfile (fp); switch (mode) { @@ -88,7 +89,7 @@ _IO_setvbuf (fp, buf, mode, size) } result = _IO_SETBUF (fp, buf, size) == NULL ? EOF : 0; unlock_return: - _IO_funlockfile (fp); + __libc_cleanup_region_end (1); return result; } diff --git a/libio/ioungetc.c b/libio/ioungetc.c index c1237b5f9b..cc414083f8 100644 --- a/libio/ioungetc.c +++ b/libio/ioungetc.c @@ -33,9 +33,10 @@ _IO_ungetc (c, fp) CHECK_FILE (fp, EOF); if (c == EOF) return EOF; + __libc_cleanup_region_start (&_IO_funlockfile, fp); _IO_flockfile (fp); result = _IO_sputbackc (fp, (unsigned char) c); - _IO_funlockfile (fp); + __libc_cleanup_region_end (1); return result; } diff --git a/libio/iovsprintf.c b/libio/iovsprintf.c index 2c5d77503d..916804f63d 100644 --- a/libio/iovsprintf.c +++ b/libio/iovsprintf.c @@ -32,7 +32,10 @@ _IO_vsprintf (string, format, args) _IO_va_list args; { _IO_strfile sf; + _IO_lock_t lock; int ret; + + sf._f._lock = &lock; _IO_init ((_IO_FILE *) &sf, 0); _IO_JUMPS ((_IO_FILE *) &sf) = &_IO_str_jumps; _IO_str_init_static ((_IO_FILE *) &sf, string, -1, string); diff --git a/libio/iovsscanf.c b/libio/iovsscanf.c index dcd3e92902..5b4c790517 100644 --- a/libio/iovsscanf.c +++ b/libio/iovsscanf.c @@ -30,6 +30,8 @@ DEFUN(_IO_vsscanf, (string, format, args), const char *string AND const char *format AND _IO_va_list args) { _IO_strfile sf; + _IO_lock_t lock; + sf._f._lock = &lock; _IO_init((_IO_FILE*)&sf, 0); _IO_JUMPS((_IO_FILE*)&sf) = &_IO_str_jumps; _IO_str_init_static ((_IO_FILE*)&sf, (char*)string, 0, NULL); diff --git a/libio/libio.h b/libio/libio.h index b498f125eb..17cef91121 100644 --- a/libio/libio.h +++ b/libio/libio.h @@ -150,6 +150,7 @@ typedef struct /* Handle lock. */ #ifdef _IO_MTSAFE_IO +#include <pthread.h> typedef pthread_mutex_t _IO_lock_t; #else typedef void _IO_lock_t; @@ -272,10 +273,10 @@ extern int _IO_putc_locked __P ((int, _IO_FILE *)); extern void _IO_flockfile __P ((_IO_FILE *)); extern void _IO_funlockfile __P ((_IO_FILE *)); -#if !defined(_REENTRANT) && !defined(_THREAD_SAFE) +#ifndef _IO_MTSAFE_IO # define _IO_flockfile(FILE) /**/ # define _IO_funlockfile(FILE) /**/ -#endif /* _REENTRANT || _THREAD_SAFE */ +#endif /* !_IO_MTSAFE_IO */ extern int _IO_vfscanf __P((_IO_FILE*, const char*, _IO_va_list, int*)); diff --git a/libio/libioP.h b/libio/libioP.h index 8525b71827..ac4df17ce8 100644 --- a/libio/libioP.h +++ b/libio/libioP.h @@ -26,6 +26,7 @@ the executable file might be covered by the GNU General Public License. */ #ifndef errno extern int errno; #endif +#include <libc-lock.h> #include "iolibio.h" diff --git a/libio/memstream.c b/libio/memstream.c index a2fe8447cb..1a4f4b9e96 100644 --- a/libio/memstream.c +++ b/libio/memstream.c @@ -16,8 +16,8 @@ License along with the GNU C Library; see the file COPYING.LIB. If not, write to the Free Software Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. */ -#include "strfile.h" #include "libioP.h" +#include "strfile.h" #include <stdio.h> #include <stdlib.h> @@ -64,26 +64,30 @@ open_memstream (bufloc, sizeloc) char **bufloc; _IO_size_t *sizeloc; { - struct _IO_FILE_memstream *fp; + struct locked_FILE + { + struct _IO_FILE_memstream fp; + _IO_lock_t lock; + } *new_f; char *buf; - fp = (struct _IO_FILE_memstream *) - malloc (sizeof (struct _IO_FILE_memstream)); - if (fp == NULL) + new_f = (struct locked_FILE *) malloc (sizeof (struct locked_FILE)); + if (new_f == NULL) return NULL; + new_f->fp._sf._f._lock = &new_f->lock; buf = ALLOC_BUF (_IO_BUFSIZ); - _IO_init (&fp->_sf._f, 0); - _IO_JUMPS (&fp->_sf._f) = &_IO_mem_jumps; - _IO_str_init_static (&fp->_sf._f, buf, _IO_BUFSIZ, buf); - fp->_sf._f._flags &= ~_IO_USER_BUF; - fp->_sf._s._allocate_buffer = (_IO_alloc_type) malloc; - fp->_sf._s._free_buffer = (_IO_free_type) free; + _IO_init (&new_f->fp._sf._f, 0); + _IO_JUMPS (&new_f->fp._sf._f) = &_IO_mem_jumps; + _IO_str_init_static (&new_f->fp._sf._f, buf, _IO_BUFSIZ, buf); + new_f->fp._sf._f._flags &= ~_IO_USER_BUF; + new_f->fp._sf._s._allocate_buffer = (_IO_alloc_type) malloc; + new_f->fp._sf._s._free_buffer = (_IO_free_type) free; - fp->bufloc = bufloc; - fp->sizeloc = sizeloc; + new_f->fp.bufloc = bufloc; + new_f->fp.sizeloc = sizeloc; - return &fp->_sf._f; + return &new_f->fp._sf._f; } diff --git a/libio/putc.c b/libio/putc.c index 23595ab7b5..5dff3de304 100644 --- a/libio/putc.c +++ b/libio/putc.c @@ -28,14 +28,15 @@ putc (c, fp) { int result; CHECK_FILE (fp, EOF); + __libc_cleanup_region_start (&_IO_funlockfile, fp); _IO_flockfile (fp); result = _IO_putc_unlocked (c, fp); - _IO_funlockfile (fp); + __libc_cleanup_region_end (1); return result; } #ifdef _IO_MTSAFE_IO # undef putc_locked -weak_alias (putc_locked, putc) +weak_alias (putc, putc_locked) #endif diff --git a/libio/putchar.c b/libio/putchar.c index d4d52f60d2..a7891a5a2a 100644 --- a/libio/putchar.c +++ b/libio/putchar.c @@ -26,14 +26,15 @@ putchar (c) int c; { int result; + __libc_cleanup_region_start (&_IO_funlockfile, _IO_stdout); _IO_flockfile (_IO_stdout); result = _IO_putc_unlocked (c, _IO_stdout); - _IO_funlockfile (_IO_stdout); + __libc_cleanup_region_end (1); return result; } #ifdef _IO_MTSAFE_IO # undef putchar_locked -weak_alias (putchar_locked, putchar) +weak_alias (putchar, putchar_locked) #endif diff --git a/libio/putchar_u.c b/libio/putchar_u.c index d29652aadc..4c52d638b7 100644 --- a/libio/putchar_u.c +++ b/libio/putchar_u.c @@ -25,6 +25,6 @@ int putchar_unlocked (c) int c; { - CHECK_FILE (fp, EOF); - return _IO_putc_unlocked (c, fp); + CHECK_FILE (stdout, EOF); + return _IO_putc_unlocked (c, stdout); } diff --git a/libio/rewind.c b/libio/rewind.c index 2f57e0de83..d30233e17e 100644 --- a/libio/rewind.c +++ b/libio/rewind.c @@ -21,15 +21,16 @@ the resulting executable to be covered by the GNU General Public License. This exception does not however invalidate any other reasons why the executable file might be covered by the GNU General Public License. */ -#include "stdio.h" #include "libioP.h" +#include "stdio.h" void rewind (fp) _IO_FILE* fp; { CHECK_FILE (fp, ); + __libc_cleanup_region_start (&_IO_funlockfile, fp); _IO_flockfile (fp); _IO_rewind (fp); - _IO_funlockfile (fp); + __libc_cleanup_region_end (1); } diff --git a/libio/stdio.h b/libio/stdio.h index 048a11d60c..0d6c5e6741 100644 --- a/libio/stdio.h +++ b/libio/stdio.h @@ -206,39 +206,51 @@ extern const char *const _sys_errlist[]; #endif /* Handle locking of streams. */ -#if defined(_REENTRANT) || defined(_THREAD_SAFE) +#ifdef __USE_REENTRANT +extern void clearerr_locked __P ((FILE *)); extern void clearerr_unlocked __P ((FILE *)); -extern void fileno_unlocked __P ((FILE *)); +extern int feof_locked __P ((FILE *)); +extern int feof_unlocked __P ((FILE *)); +extern int ferror_locked __P ((FILE*)); +extern int ferror_unlocked __P ((FILE*)); +extern int fileno_locked __P ((FILE *)); +extern int fileno_unlocked __P ((FILE *)); extern void flockfile __P ((FILE *)); extern void funlockfile __P ((FILE *)); extern int fclose_unlocked __P ((FILE *)); +extern int fflush_locked __P ((FILE *)); extern int fflush_unlocked __P ((FILE *)); extern size_t fread_unlocked __P ((void *, size_t, size_t, FILE *)); extern size_t fwrite_unlocked __P ((const void *, size_t, size_t, FILE *)); +extern int fputc_locked __P ((int, FILE*)); +extern int fputc_unlocked __P ((int, FILE*)); +extern int getc_locked __P ((FILE *)); +extern int getc_unlocked __P ((FILE *)); +extern int getchar_locked __P ((void)); +extern int getchar_unlocked __P ((void)); +extern int putc_locked __P ((int, FILE *)); +extern int putc_unlocked __P ((int, FILE *)); +extern int putchar_locked __P ((int)); +extern int putchar_unlocked __P ((int)); + # define getc_unlocked(fp) _IO_getc_unlocked (fp) -# define getc_locked(fp) _IO_fgetc (fp) +# define getc_locked(fp) fgetc (fp) # define getchar_unlocked() getc_unlocked (stdin) # define getchar_locked() getc_locked (stdin) # define getc(fp) getc_locked (fp) -# define putc_unlocked(c, fp) _IO_putc_unlocked (c, fp) -# define putc_locked(c, fp) _IO_putc_locked (c, fp) +# define putc_unlocked(c, fp) putc_unlocked (c, fp) +# define putc_locked(c, fp) putc_locked (c, fp) # define putchar_unlocked(c) putc_unlocked (c, stdout) # define putchar_locked(c) putc_locked (c, stdout) # define putc(c, fp) putc_locked (c, fp) -# define feof_unlocked(fp) _IO_feof_unlocked (fp) -# define ferror_unlocked(fp) _IO_ferror_unlocked (fp) - #else # define getc(fp) _IO_getc_unlocked (fp) # define putc(c, fp) _IO_putc_unlocked (c, fp) -#endif /* _REENTRANT || _THREAD_SAFE */ - -#define flockfile(FILE) _IO_flockfile (FILE) -#define funlockfile(FILE) _IO_funlockfile (FILE) +#endif /* __USE_REENTRANT */ #define putchar(c) putc (c, stdout) #define getchar() getc (stdin) diff --git a/libio/vasprintf.c b/libio/vasprintf.c index ee92f83b1c..cf50603f0d 100644 --- a/libio/vasprintf.c +++ b/libio/vasprintf.c @@ -38,10 +38,12 @@ _IO_vasprintf (result_ptr, format, args) const _IO_size_t init_string_size = 100; char *string; _IO_strfile sf; + _IO_lock_t lock; int ret; string = ALLOC_BUF(init_string_size); if (string == NULL) return -1; + sf._f._lock = &lock; _IO_init((_IO_FILE*)&sf, 0); _IO_JUMPS((_IO_FILE*)&sf) = &_IO_str_jumps; _IO_str_init_static ((_IO_FILE*)&sf, string, init_string_size, string); diff --git a/libio/vdprintf.c b/libio/vdprintf.c index b92251eada..8c5833ccbf 100644 --- a/libio/vdprintf.c +++ b/libio/vdprintf.c @@ -32,8 +32,10 @@ _IO_vdprintf (d, format, arg) _IO_va_list arg; { struct _IO_FILE_plus tmpfil; + _IO_lock_t lock; int done; + tmpfil.file._lock = &lock; _IO_init (&tmpfil.file, 0); _IO_JUMPS (&tmpfil.file) = &_IO_file_jumps; _IO_file_init (&tmpfil.file); diff --git a/libio/vsnprintf.c b/libio/vsnprintf.c index 38f91c9c07..174640daac 100644 --- a/libio/vsnprintf.c +++ b/libio/vsnprintf.c @@ -33,7 +33,9 @@ _IO_vsnprintf (string, maxlen, format, args) _IO_va_list args; { _IO_strfile sf; + _IO_lock_t lock; int ret; + sf._f._lock = &lock; _IO_init ((_IO_FILE *) &sf, 0); _IO_JUMPS ((_IO_FILE *) &sf) = &_IO_str_jumps; _IO_str_init_static ((_IO_FILE *) &sf, string, maxlen - 1, string); |