about summary refs log tree commit diff
diff options
context:
space:
mode:
authorPeter Stephenson <pws@users.sourceforge.net>2006-05-26 22:45:45 +0000
committerPeter Stephenson <pws@users.sourceforge.net>2006-05-26 22:45:45 +0000
commitc0810e3c0226efaf362419ea3a8d1f6d7f74ca20 (patch)
tree78dda92affa0498e082f9ce1ac9c236644ee8c34
parentfe1ddacd822e1a279dee902dd3f9a364c68f33de (diff)
downloadzsh-c0810e3c0226efaf362419ea3a8d1f6d7f74ca20.tar.gz
zsh-c0810e3c0226efaf362419ea3a8d1f6d7f74ca20.tar.xz
zsh-c0810e3c0226efaf362419ea3a8d1f6d7f74ca20.zip
22464: bad recursion if error in prompt substitution
-rw-r--r--ChangeLog5
-rw-r--r--Src/Zle/zle_main.c13
2 files changed, 14 insertions, 4 deletions
diff --git a/ChangeLog b/ChangeLog
index a3f394aab..24108151d 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2006-05-26  Peter Stephenson  <p.w.stephenson@ntlworld.com>
+
+	* 22464: Src/Zle/zle_main.c: an error in prompt substitution could
+	cause bad recursion karma.
+
 2006-01-10  Peter Stephenson  <pws@csr.com>
 
 	* 22151: Src/text.c: a here-string got too many quotes when
diff --git a/Src/Zle/zle_main.c b/Src/Zle/zle_main.c
index c6e374bc7..69e88c8b4 100644
--- a/Src/Zle/zle_main.c
+++ b/Src/Zle/zle_main.c
@@ -1342,10 +1342,15 @@ recursiveedit(UNUSED(char **args))
 void
 reexpandprompt(void)
 {
-    free(lpromptbuf);
-    lpromptbuf = promptexpand(raw_lp ? *raw_lp : NULL, 1, NULL, NULL);
-    free(rpromptbuf);
-    rpromptbuf = promptexpand(raw_rp ? *raw_rp : NULL, 1, NULL, NULL);
+    static reexpanding;
+
+    if (!reexpanding++) {
+	free(lpromptbuf);
+	lpromptbuf = promptexpand(raw_lp ? *raw_lp : NULL, 1, NULL, NULL);
+	free(rpromptbuf);
+	rpromptbuf = promptexpand(raw_rp ? *raw_rp : NULL, 1, NULL, NULL);
+    }
+    reexpanding--;
 }
 
 /**/