procyberian/emacs
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Don't let unusual after-change-functions overwrite " prin1" buffer in use.

Browse source 
src/doc.c (Fsubstitute_command_keys): Inhibit modification hooks
 while we are using Vprin1_to_string_buffer.

Fixes: debbugs:15907
This commit is contained in:
Eli Zaretskii 2013-11-16 12:23:09 +02:00
parent 3d6dec8d1e
commit d17f4dba9c
2 changed files with 9 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/ChangeLog
View file

@ -1,5 +1,8 @@
2013-11-16 Eli Zaretskii <eliz@gnu.org>
* doc.c (Fsubstitute_command_keys): Inhibit modification hooks
while we are using Vprin1_to_string_buffer.
* keymap.c (describe_map): Don't crash if PT is 1 both before and
after inserting the description string. (Bug#15907)

6
src/doc.c
View file

@ -850,6 +850,7 @@ Otherwise, return a new string, without any text properties. */)
/* This is for computing the SHADOWS arg for describe_map_tree. */
Lisp_Object active_maps = Fcurrent_active_maps (Qnil, Qnil);
Lisp_Object earlier_maps;
ptrdiff_t count = SPECPDL_INDEX ();
changed = 1;
strp += 2; /* skip \{ or \< */
@ -886,6 +887,10 @@ Otherwise, return a new string, without any text properties. */)
/* Now switch to a temp buffer. */
oldbuf = current_buffer;
set_buffer_internal (XBUFFER (Vprin1_to_string_buffer));
/* This is for an unusual case where some after-change
function uses 'format' or 'prin1' or something else that
will thrash Vprin1_to_string_buffer we are using. */
specbind (Qinhibit_modification_hooks, Qt);
if (NILP (tem))
{
@ -910,6 +915,7 @@ Otherwise, return a new string, without any text properties. */)
tem = Fbuffer_string ();
Ferase_buffer ();
set_buffer_internal (oldbuf);
unbind_to (count, Qnil);
subst_string:
start = SDATA (tem);