Johannes Schindelin <Johannes.Schindelin@xxxxxx> wrote: > +static int cmd_whatchanged(int argc, const char **argv, char **envp) > +{ > + memmove(argv + 2, argv + 1, argc - 1); Shouldn't the size be sizeof(char *) * argc (NULL terminated array)? There's also overflow... -- http://onion.dynserv.net/~timo/ - : 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