Till Wegmüller
2024-04-07 6ca8c2ef2bffdfec8cc95c049ac3f2230dc6ca82
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
# DP: bash43-021 upstream patch
 
                 BASH PATCH REPORT
                 =================
 
Bash-Release:    4.3
Patch-ID:    bash43-021
 
Bug-Reported-by:    Jared Yanovich <slovichon@gmail.com>
Bug-Reference-ID:    <20140625225019.GJ17044@nightderanger.psc.edu>
Bug-Reference-URL:    http://lists.gnu.org/archive/html/bug-bash/2014-06/msg00070.html
 
Bug-Description:
 
When the readline `revert-all-at-newline' option is set, pressing newline
when the current line is one retrieved from history results in a double free
and a segmentation fault.
 
Patch (apply with `patch -p0'):
 
Index: b/misc.c
===================================================================
--- a/misc.c
+++ b/misc.c
@@ -461,6 +461,7 @@ _rl_revert_all_lines ()
         saved_undo_list = 0;
       /* Set up rl_line_buffer and other variables from history entry */
       rl_replace_from_history (entry, 0);    /* entry->line is now current */
+      entry->data = 0;            /* entry->data is now current undo list */
       /* Undo all changes to this history entry */
       while (rl_undo_list)
         rl_do_undo ();
@@ -468,7 +469,6 @@ _rl_revert_all_lines ()
          the timestamp. */
       FREE (entry->line);
       entry->line = savestring (rl_line_buffer);
-      entry->data = 0;
     }
       entry = previous_history ();
     }