stash: fix incorrect quoting in cleanup of temporary files
authorJohannes Sixt <j6t@kdbg.org>
Wed, 16 Mar 2011 08:14:33 +0000 (09:14 +0100)
committerJunio C Hamano <gitster@pobox.com>
Thu, 17 Mar 2011 21:53:10 +0000 (14:53 -0700)
The * was inside the quotes, so that the pattern was never expanded and the
temporary files were never removed. As a consequence, 'stash -p' left a
.git-stash-*-patch file in $GIT_DIR. Other code paths did not leave files
behind because they removed the temporary file themselves, at least in
non-error paths.

Signed-off-by: Johannes Sixt <j6t@kdbg.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
git-stash.sh

index 7561b374d2ec90fe774e98e39a121a1a3a3cdfa9..7c0d563c633bb1ace8536585797da2f93759f1e9 100755 (executable)
@@ -17,7 +17,7 @@ require_work_tree
 cd_to_toplevel
 
 TMP="$GIT_DIR/.git-stash.$$"
-trap 'rm -f "$TMP-*"' 0
+trap 'rm -f "$TMP-"*' 0
 
 ref_stash=refs/stash