# HG changeset patch # User Bruno Haible # Date 1189342904 0 # Node ID 1050562322a42dc0699da62440a808824e548864 # Parent c7875dab3c3eb9ce6bfd7d73d6771d5ab4e8c223 New module 'malloc-posix'. diff -r c7875dab3c3e -r 1050562322a4 ChangeLog --- a/ChangeLog Sun Sep 09 12:56:27 2007 +0000 +++ b/ChangeLog Sun Sep 09 13:01:44 2007 +0000 @@ -1,3 +1,19 @@ +2007-09-09 Bruno Haible + + * modules/malloc-posix: New file. + * modules/malloc (Depends-on): Add malloc-posix. + * lib/malloc.c: Include errno.h. + (rpl_malloc): Merge the requirements of a glibc-compatible malloc + and a POSIX-compatible malloc into a single function. Set ENOMEM + when returning NULL. + * m4/malloc.m4: New file. + * doc/functions/malloc.texi: Mention the malloc-posix module. + * lib/stdlib_.h (malloc): New declaration. + * m4/stdlib_h.m4 (gl_STDLIB_H_DEFAULTS): Initialize + GNULIB_MALLOC_POSIX and HAVE_MALLOC_POSIX. + * modules/stdlib (stdlib.h): Substitute also GNULIB_MALLOC_POSIX + and HAVE_MALLOC_POSIX. + 2007-09-09 Bruno Haible * modules/realloc-posix: New file. diff -r c7875dab3c3e -r 1050562322a4 doc/functions/malloc.texi --- a/doc/functions/malloc.texi Sun Sep 09 12:56:27 2007 +0000 +++ b/doc/functions/malloc.texi Sun Sep 09 13:01:44 2007 +0000 @@ -4,10 +4,14 @@ POSIX specification: @url{http://www.opengroup.org/susv3xsh/malloc.html} -Gnulib module: --- +Gnulib module: malloc-posix Portability problems fixed by Gnulib: @itemize +@item +Upon failure, the function does not set @code{errno} to @code{ENOMEM} on +some platforms: +mingw. @end itemize Portability problems not fixed by Gnulib: diff -r c7875dab3c3e -r 1050562322a4 lib/malloc.c --- a/lib/malloc.c Sun Sep 09 12:56:27 2007 +0000 +++ b/lib/malloc.c Sun Sep 09 13:01:44 2007 +0000 @@ -1,6 +1,6 @@ /* malloc() function that is glibc compatible. - Copyright (C) 1997, 1998, 2006 Free Software Foundation, Inc. + Copyright (C) 1997, 1998, 2006, 2007 Free Software Foundation, Inc. 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 @@ -16,20 +16,39 @@ along with this program; if not, write to the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. */ -/* written by Jim Meyering */ +/* written by Jim Meyering and Bruno Haible */ #include -#undef malloc +/* Only the AC_FUNC_MALLOC macro defines 'malloc' already in config.h. */ +#ifdef malloc +# define NEED_MALLOC_GNU +# undef malloc +#endif +/* Specification. */ #include +#include + /* Allocate an N-byte block of memory from the heap. If N is zero, allocate a 1-byte block. */ void * rpl_malloc (size_t n) { + void *result; + +#ifdef NEED_MALLOC_GNU if (n == 0) n = 1; - return malloc (n); +#endif + + result = malloc (n); + +#if !HAVE_MALLOC_POSIX + if (result == NULL) + errno = ENOMEM; +#endif + + return result; } diff -r c7875dab3c3e -r 1050562322a4 lib/stdlib_.h --- a/lib/stdlib_.h Sun Sep 09 12:56:27 2007 +0000 +++ b/lib/stdlib_.h Sun Sep 09 13:01:44 2007 +0000 @@ -55,6 +55,21 @@ #endif +#if @GNULIB_MALLOC_POSIX@ +# if !@HAVE_MALLOC_POSIX@ +# undef malloc +# define malloc rpl_malloc +extern void * malloc (size_t size); +# endif +#elif defined GNULIB_POSIXCHECK +# undef malloc +# define malloc(s) \ + (GL_LINK_WARNING ("malloc is not POSIX compliant everywhere - " \ + "use gnulib module malloc-posix for portability"), \ + malloc (s)) +#endif + + #if @GNULIB_REALLOC_POSIX@ # if !@HAVE_REALLOC_POSIX@ # undef realloc diff -r c7875dab3c3e -r 1050562322a4 m4/malloc.m4 --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/m4/malloc.m4 Sun Sep 09 13:01:44 2007 +0000 @@ -0,0 +1,34 @@ +# malloc.m4 serial 8 +dnl Copyright (C) 2007 Free Software Foundation, Inc. +dnl This file is free software; the Free Software Foundation +dnl gives unlimited permission to copy and/or distribute it, +dnl with or without modifications, as long as this notice is preserved. + +# gl_FUNC_MALLOC_POSIX +# -------------------- +# Test whether 'malloc' is POSIX compliant (sets errno to ENOMEM when it +# fails), and replace malloc if it is not. +AC_DEFUN([gl_FUNC_MALLOC_POSIX], +[ + AC_CACHE_CHECK([whether malloc, realloc, calloc are POSIX compliant], + [gl_cv_func_malloc_posix], + [ + dnl It is too dangerous to try to allocate a large amount of memory: + dnl some systems go to their knees when you do that. So assume that + dnl all Unix implementations of the function are POSIX compliant. + AC_TRY_COMPILE([], + [#if !((defined _WIN32 || defined __WIN32__) && ! defined __CYGWIN__) + choke me + #endif + ], [gl_cv_func_malloc_posix=yes], [gl_cv_func_malloc_posix=no]) + ]) + if test $gl_cv_func_malloc_posix = yes; then + HAVE_MALLOC_POSIX=1 + AC_DEFINE([HAVE_MALLOC_POSIX], 1, + [Define if the 'malloc' function is POSIX compliant.]) + else + AC_LIBOBJ([malloc]) + HAVE_MALLOC_POSIX=0 + fi + AC_SUBST([HAVE_MALLOC_POSIX]) +]) diff -r c7875dab3c3e -r 1050562322a4 m4/stdlib_h.m4 --- a/m4/stdlib_h.m4 Sun Sep 09 12:56:27 2007 +0000 +++ b/m4/stdlib_h.m4 Sun Sep 09 13:01:44 2007 +0000 @@ -19,6 +19,7 @@ AC_DEFUN([gl_STDLIB_H_DEFAULTS], [ + GNULIB_MALLOC_POSIX=0; AC_SUBST([GNULIB_MALLOC_POSIX]) GNULIB_REALLOC_POSIX=0; AC_SUBST([GNULIB_REALLOC_POSIX]) GNULIB_CALLOC_POSIX=0; AC_SUBST([GNULIB_CALLOC_POSIX]) GNULIB_GETSUBOPT=0; AC_SUBST([GNULIB_GETSUBOPT]) @@ -27,6 +28,7 @@ dnl Assume proper GNU behavior unless another module says otherwise. HAVE_CALLOC_POSIX=1; AC_SUBST([HAVE_CALLOC_POSIX]) HAVE_GETSUBOPT=1; AC_SUBST([HAVE_GETSUBOPT]) + HAVE_MALLOC_POSIX=1; AC_SUBST([HAVE_MALLOC_POSIX]) HAVE_MKDTEMP=1; AC_SUBST([HAVE_MKDTEMP]) HAVE_REALLOC_POSIX=1; AC_SUBST([HAVE_REALLOC_POSIX]) REPLACE_MKSTEMP=0; AC_SUBST([REPLACE_MKSTEMP]) diff -r c7875dab3c3e -r 1050562322a4 modules/malloc --- a/modules/malloc Sun Sep 09 12:56:27 2007 +0000 +++ b/modules/malloc Sun Sep 09 13:01:44 2007 +0000 @@ -5,6 +5,7 @@ lib/malloc.c Depends-on: +malloc-posix configure.ac: AC_FUNC_MALLOC diff -r c7875dab3c3e -r 1050562322a4 modules/malloc-posix --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/modules/malloc-posix Sun Sep 09 13:01:44 2007 +0000 @@ -0,0 +1,25 @@ +Description: +malloc() function: allocate memory with indefinite extent. + +Files: +lib/malloc.c +m4/malloc.m4 + +Depends-on: +stdlib + +configure.ac: +gl_FUNC_MALLOC_POSIX +gl_STDLIB_MODULE_INDICATOR([malloc-posix]) + +Makefile.am: + +Include: + + +License: +LGPL + +Maintainer: +Bruno Haible + diff -r c7875dab3c3e -r 1050562322a4 modules/stdlib --- a/modules/stdlib Sun Sep 09 12:56:27 2007 +0000 +++ b/modules/stdlib Sun Sep 09 13:01:44 2007 +0000 @@ -23,6 +23,7 @@ { echo '/* DO NOT EDIT! GENERATED AUTOMATICALLY! */' && \ sed -e 's/@''INCLUDE_NEXT''@/$(INCLUDE_NEXT)/g' \ -e 's|@''NEXT_STDLIB_H''@|$(NEXT_STDLIB_H)|g' \ + -e 's|@''GNULIB_MALLOC_POSIX''@|$(GNULIB_MALLOC_POSIX)|g' \ -e 's|@''GNULIB_REALLOC_POSIX''@|$(GNULIB_REALLOC_POSIX)|g' \ -e 's|@''GNULIB_CALLOC_POSIX''@|$(GNULIB_CALLOC_POSIX)|g' \ -e 's|@''GNULIB_GETSUBOPT''@|$(GNULIB_GETSUBOPT)|g' \ @@ -30,6 +31,7 @@ -e 's|@''GNULIB_MKSTEMP''@|$(GNULIB_MKSTEMP)|g' \ -e 's|@''HAVE_CALLOC_POSIX''@|$(HAVE_CALLOC_POSIX)|g' \ -e 's|@''HAVE_GETSUBOPT''@|$(HAVE_GETSUBOPT)|g' \ + -e 's|@''HAVE_MALLOC_POSIX''@|$(HAVE_MALLOC_POSIX)|g' \ -e 's|@''HAVE_MKDTEMP''@|$(HAVE_MKDTEMP)|g' \ -e 's|@''HAVE_REALLOC_POSIX''@|$(HAVE_REALLOC_POSIX)|g' \ -e 's|@''REPLACE_MKSTEMP''@|$(REPLACE_MKSTEMP)|g' \