From e4d3ee06b39043901054f1f20682a2e79c9f1924 Mon Sep 17 00:00:00 2001 From: Peter Stephenson Date: Thu, 18 Sep 2008 09:42:44 +0000 Subject: 25682: terminate history line before passing to zshaddhistory hook --- Src/hist.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) (limited to 'Src') diff --git a/Src/hist.c b/Src/hist.c index f3979dfc0..55131f06e 100644 --- a/Src/hist.c +++ b/Src/hist.c @@ -1129,7 +1129,13 @@ hend(Eprog prog) if (hist_ignore_all_dups != isset(HISTIGNOREALLDUPS) && (hist_ignore_all_dups = isset(HISTIGNOREALLDUPS)) != 0) histremovedups(); - + + /* + * Added the following in case the test "hptr < chline + 1" + * is more than just paranoia. + */ + DPUTS(hptr < chline, "History end pointer off start of line"); + *hptr = '\0'; addlinknode(hookargs, "zshaddhistory"); addlinknode(hookargs, chline); callhookfunc("zshaddhistory", hookargs, 1, &hookret); @@ -1144,7 +1150,6 @@ hend(Eprog prog) if (hptr < chline + 1) save = 0; else { - *hptr = '\0'; if (hptr[-1] == '\n') { if (chline[1]) { *--hptr = '\0'; -- cgit 1.4.1