fork0@xxxxxxxxxxx (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. > 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. - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html