ebl Fix #1226 about bconsole segfault when using readline()

git-svn-id: https://bacula.svn.sourceforge.net/svnroot/bacula/trunk@8481 91ce42f0-d328-0410-95d8-f526ca767f89

diff --git a/bacula/src/console/console.c b/bacula/src/console/console.c
index 4150407e7d303f1fac2f4f7b12995b6fc58e8653..c1f0d2c128602db4abd8dcc6a2706d1446732ff1 100644 (file)
--- a/bacula/src/console/console.c
+++ b/bacula/src/console/console.c
@@ -344,7 +344,6 @@ static int tls_pem_callback(char *buf, int size, const void *userdata)
 
 #ifdef HAVE_READLINE
 #define READLINE_LIBRARY 1
-#undef free
 #include "readline.h"
 #include "history.h"
 
@@ -372,6 +371,9 @@ get_cmd(FILE *input, const char *prompt, BSOCK *sock, int sec)
    if (line == NULL) {
       do_history = 0;
       rl_catch_signals = 0;              /* do it ourselves */
+      /* Here, readline does ***real*** malloc
+       * so, be we have to use the real free
+       */
       line = readline((char *)prompt);   /* cast needed for old readlines */
       if (!line) {
          exit(1);
@@ -412,7 +414,7 @@ get_cmd(FILE *input, const char *prompt, BSOCK *sock, int sec)
       if (do_history) {
         add_history(line);
       }
-      free(line);
+      actuallyfree(line);       /* allocated by readline() malloc */
       line = NULL;
    }
    return 1;

diff --git a/bacula/technotes-2.5 b/bacula/technotes-2.5
index 387cc3dc97b56a89961c5b88ba37cd13c2cc6c7a..6bd07b42f50afe28d9e71173646e1a08868b0075 100644 (file)
--- a/bacula/technotes-2.5
+++ b/bacula/technotes-2.5
@@ -25,6 +25,8 @@ filepattern (restore with regex in bsr)
 mixed priorities
 
 General:
+19Feb09
+ebl  Fix #1226 about bconsole segfault when using readline()
 18Feb09
 kes  Apply Eric's next-beta.patch that enables 64 bit FileIds and
      adds new columns to the catalog.