rebase -i: do not "echo" random user-supplied strings
commit47be06602656ee9cac860f675d2c8d1f0deabdbe
authorUwe Storbeck <uwe@ibr.ch>
Fri, 14 Mar 2014 23:56:43 +0000 (15 00:56 +0100)
committerJunio C Hamano <gitster@pobox.com>
Mon, 17 Mar 2014 19:24:14 +0000 (17 12:24 -0700)
tree4541122272ed41cbee9ec6152632ad8bd25b34d7
parent16216b6ab1073b7aaa8225c32208758e6ea16629
rebase -i: do not "echo" random user-supplied strings

In some places we "echo" a string that comes from a commit log
message, which may have a backslash sequence that is interpreted by
the command (POSIX.1 allows this), most notably "dash"'s built-in
'echo'.

A commit message which contains the string '\n' (or ends with the
string '\c') may result in a garbage line in the todo list of an
interactive rebase which causes the rebase to fail.

To reproduce the behavior (with dash as /bin/sh):

  mkdir test && cd test && git init
  echo 1 >foo && git add foo
  git commit -m"this commit message ends with '\n'"
  echo 2 >foo && git commit -a --fixup HEAD
  git rebase -i --autosquash --root

Now the editor opens with garbage in line 3 which has to be
removed or the rebase fails.

Signed-off-by: Uwe Storbeck <uwe@ibr.ch>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
git-rebase--interactive.sh