setup_readline(): Added a comment about memory leak (reported by

Purify) being caused by a bug in the readline library.  Nothing we can
do about it.

Cause: readline_initialize_everything() throws away the return value
from rl_read_init_file(), but that happens to be the last reference to
a dynamically allocated char*.

diff --git a/Modules/readline.c b/Modules/readline.c
index 608dd88f518af87d02846b050cc0738860b9f183..b2b56a2279cdaaf035ea2b57129fa1dcb38966b9 100644 (file)
--- a/Modules/readline.c
+++ b/Modules/readline.c
@@ -244,7 +244,11 @@ setup_readline()
        rl_completer_word_break_characters =
                " \t\n`~!@#$%^&*()-=+[{]}\\|;:'\",<>/?";
                /* All nonalphanums except '.' */
-       /* Initialize (allows .inputrc to override) */
+       /* Initialize (allows .inputrc to override)
+        *
+        * XXX: A bug in the readline-2.2 library causes a memory leak
+        * inside this function.  Nothing we can do about it.
+        */
        rl_initialize();
 }