changeset 12463:d6e56e6a980f

fprintf-posix: Check against memory leak fixed on 2009-12-15.
author Bruno Haible <bruno@clisp.org>
date Sat, 19 Dec 2009 22:15:32 +0100
parents f7383de75775
children 5bd91471b957
files ChangeLog modules/fprintf-posix-tests tests/test-fprintf-posix3.c tests/test-fprintf-posix3.sh
diffstat 4 files changed, 151 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2009-12-19  Bruno Haible  <bruno@clisp.org>
+
+	fprintf-posix: Check against memory leak fixed on 2009-12-15.
+	* tests/test-fprintf-posix3.sh: New file.
+	* tests/test-fprintf-posix3.c: New file.
+	* modules/fprintf-posix-tests (Files): Add them.
+	(Makefile.am): Augment TESTS and CHECK_PROGRAMS.
+
 2009-12-19  Eric Blake  <ebb9@byu.net>
 
 	dirfd: fix prototype
--- a/modules/fprintf-posix-tests
+++ b/modules/fprintf-posix-tests
@@ -5,6 +5,8 @@
 tests/test-printf-posix.output
 tests/test-fprintf-posix2.sh
 tests/test-fprintf-posix2.c
+tests/test-fprintf-posix3.sh
+tests/test-fprintf-posix3.c
 
 Depends-on:
 stdint
@@ -13,6 +15,6 @@
 AC_CHECK_FUNCS_ONCE([getrlimit setrlimit])
 
 Makefile.am:
-TESTS += test-fprintf-posix.sh test-fprintf-posix2.sh
+TESTS += test-fprintf-posix.sh test-fprintf-posix2.sh test-fprintf-posix3.sh
 TESTS_ENVIRONMENT += EXEEXT='@EXEEXT@' srcdir='$(srcdir)'
-check_PROGRAMS += test-fprintf-posix test-fprintf-posix2
+check_PROGRAMS += test-fprintf-posix test-fprintf-posix2 test-fprintf-posix3
new file mode 100644
--- /dev/null
+++ b/tests/test-fprintf-posix3.c
@@ -0,0 +1,108 @@
+/* Test of POSIX compatible fprintf() function.
+   Copyright (C) 2009 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
+   the Free Software Foundation; either version 3 of the License, or
+   (at your option) any later version.
+
+   This program 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 program.  If not, see <http://www.gnu.org/licenses/>.  */
+
+/* Written by Bruno Haible <bruno@clisp.org>, 2009.  */
+
+#include <config.h>
+
+#include <stdio.h>
+
+#if HAVE_GETRLIMIT && HAVE_SETRLIMIT
+
+#include <stdlib.h>
+#include <sys/types.h>
+#include <sys/time.h>
+#include <sys/resource.h>
+#include <string.h>
+#include <errno.h>
+
+/* Test against a memory leak in the fprintf replacement.  */
+
+/* Number of iterations across the loop.  */
+#define NUM_ROUNDS 1000
+
+/* Number of bytes that are allowed to escape per round.  */
+#define MAX_ALLOC_ROUND 10000
+
+/* Number of bytes that are allowed to escape in total.
+   This should be at least 10 MB, since it includes the normal memory
+   or address space of the test program.  */
+#define MAX_ALLOC_TOTAL (NUM_ROUNDS * MAX_ALLOC_ROUND)
+
+int
+main (int argc, char *argv[])
+{
+  struct rlimit limit;
+  int arg;
+  int repeat;
+
+  /* Limit the amount of malloc()ed memory to MAX_ALLOC_TOTAL or less.  */
+
+  /* On BSD systems, malloc() is limited by RLIMIT_DATA.  */
+#ifdef RLIMIT_DATA
+  if (getrlimit (RLIMIT_DATA, &limit) < 0)
+    return 77;
+  if (limit.rlim_max == RLIM_INFINITY || limit.rlim_max > MAX_ALLOC_TOTAL)
+    limit.rlim_max = MAX_ALLOC_TOTAL;
+  limit.rlim_cur = limit.rlim_max;
+  if (setrlimit (RLIMIT_DATA, &limit) < 0)
+    return 77;
+#endif
+  /* On Linux systems, malloc() is limited by RLIMIT_AS.  */
+#ifdef RLIMIT_AS
+  if (getrlimit (RLIMIT_AS, &limit) < 0)
+    return 77;
+  if (limit.rlim_max == RLIM_INFINITY || limit.rlim_max > MAX_ALLOC_TOTAL)
+    limit.rlim_max = MAX_ALLOC_TOTAL;
+  limit.rlim_cur = limit.rlim_max;
+  if (setrlimit (RLIMIT_AS, &limit) < 0)
+    return 77;
+#endif
+
+  arg = atoi (argv[1]);
+  if (arg == 0)
+    {
+      void *memory = malloc (MAX_ALLOC_TOTAL);
+      if (memory == NULL)
+        return 1;
+      memset (memory, 17, MAX_ALLOC_TOTAL);
+      return 78;
+    }
+
+  /* Perform the test and test whether it triggers a permanent memory
+     allocation of more than MAX_ALLOC_TOTAL bytes.  */
+
+  for (repeat = 0; repeat < NUM_ROUNDS; repeat++)
+    {
+      /* This may produce a temporary memory allocation of 11000 bytes.
+         but should not result in a permanent memory allocation.  */
+      if (fprintf (stdout, "%011000d\n", 17) == -1
+          && errno == ENOMEM)
+        return 1;
+    }
+
+  return 0;
+}
+
+#else
+
+int
+main (int argc, char *argv[])
+{
+  return 77;
+}
+
+#endif
new file mode 100755
--- /dev/null
+++ b/tests/test-fprintf-posix3.sh
@@ -0,0 +1,31 @@
+#!/bin/sh
+
+# Test against a memory leak.
+
+(./test-fprintf-posix3${EXEEXT} 0
+ result=$?
+ if test $result != 77 && test $result != 78; then result=1; fi
+ exit $result
+) 2>/dev/null
+malloc_result=$?
+if test $malloc_result = 77; then
+  echo "Skipping test: getrlimit and setrlimit don't work"
+  exit 77
+fi
+
+./test-fprintf-posix3${EXEEXT} 1 > /dev/null
+result=$?
+if test $result = 77; then
+  echo "Skipping test: getrlimit and setrlimit don't work"
+  exit 77
+fi
+if test $result != 0; then
+  exit 1
+fi
+
+if test $malloc_result = 78; then
+  echo "Skipping test: getrlimit and setrlimit don't work"
+  exit 77
+fi
+
+exit 0