Allowing the user full control over the values of MALLOC_CHECK_ and

MALLOC_PERTURB_; nano shouldn't override them.


git-svn-id: svn://svn.savannah.gnu.org/nano/trunk/nano@5446 35c25a1d-7b9e-4130-9fde-d3aeb78583b8

ChangeLog

+2015-11-28  Benno Schulenberg  <bensberg@justemail.net>
+	* src/nano.c (main): Allow the user full control over the values of
+	MALLOC_CHECK_ and MALLOC_PERTURB_; nano shouldn't override these.
+	This reverts r5344 from August 6.
+
 2015-11-26  Benno Schulenberg  <bensberg@justemail.net>
 	* src/text.c (do_redo): Not just the undoing, also the redoing of a
 	Backspace at EOF is a special case.  This fixes Savannah bug #46532.
@@ -213,7 +218,7 @@ GNU nano 2.4.3 - 2015.11.18
 2015-08-06  Benno Schulenberg  <bensberg@justemail.net>
 	* src/nano.c (main): For --enable-debug builds, let malloc() help
 	to find initialization failures and uses-after-free.  Suggested by
-	Mike Frysinger.
+	Mike Frysinger.  [Reverted in r5446 on November 28.]
 	* doc/texinfo/nano.texi: Improve the formatting, using @t to mark
 	double-quoted literal strings, @: to mark periods that do not end
 	sentences, and @. to mark a finishing period after a capital.

src/nano.c

@@ -2189,12 +2189,6 @@ int main(int argc, char **argv)
     };
 #endif
 
-#ifdef DEBUG
-    /* Help to find failed-to-initialize and use-after-free problems. */
-    mallopt(M_CHECK_ACTION, 3);
-    mallopt(M_PERTURB, 0x4c);
-#endif
-
 #ifdef ENABLE_UTF8
     {
 	/* If the locale set exists and uses UTF-8, we should use

src/nano.h

@@ -54,10 +54,6 @@
 #include <stdarg.h>
 #endif
 
-#ifdef DEBUG
-#include <malloc.h>
-#endif
-
 /* Suppress warnings for __attribute__((warn_unused_result)). */
 #define IGNORE_CALL_RESULT(call) do { if (call) {} } while(0)