Re: git-blame: handling of revisions and filenames
From: Junio C Hamano <hidden>
Date: 2016-08-11 20:42:31
fork0@t-online.de (Alex Riesen) writes:
And I afraid the patch has a small chance of crashing: I don't check if there is enough space in argv (don't actually even know how to),
Actually this is a safe thing to do because we already know (and removed) the "path" argument from the command line at that point. If the original command line did not have "path" we have already rejected it. So unk at that point never exceeds argc.
quoted hunk
diff --git a/builtin-blame.c b/builtin-blame.c index 066dee7..83c8905 100644 --- a/builtin-blame.c +++ b/builtin-blame.c@@ -1787,6 +1787,7 @@ int cmd_blame(int argc, const char **arg /* Now we got rev and path. We do not want the path pruning * but we may want "bottom" processing. */ + argv[unk++] = "--"; argv[unk] = NULL; init_revisions(&revs, NULL);
Thanks.