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 --- ChangeLog | 5 +++++ Src/hist.c | 9 +++++++-- 2 files changed, 12 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index f4172a83e..a9240e419 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +2008-09-18 Peter Stephenson + + * 25682: Src/hist.c: terminate history line string before passing + down to zshaddhistory hook. + 2008-09-16 Peter Stephenson * 25677: Doc/Zsh/prompt.yo, Src/exec.c, Src/init.c, Src/math.c, 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