From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Mon, 10 Apr 2006 08:10:41 GMT
Subject: Re: [PATCH] git log [diff-tree options]...
Message-ID: <Pine.LNX.4.63.0604101010001.31612@wbgn013.biozentrum.uni-wuerzburg.de>
In-Reply-To: <20060410012258.589f1581.tihirvon@gmail.com>

Hi,

On Mon, 10 Apr 2006, Timo Hirvonen wrote:

> Johannes Schindelin <Johannes.Schindelin@gmx.de> 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...

Yeah, this was some useless late-night coding. But you get the idea.

Ciao,
Dscho