Re: [PATCH] Speedup recursive by flushing index only once for all entries

From: Sergey Vlasov <vsu@altlinux.ru>
Date: Jan 12, 2007, 20:37 UTC
Message-ID: <20070112203721.GA4562@procyon.home>
In-Reply-To: <81b0412b0701120938o1606dcachf2553a83b47921b1@mail.gmail.com>

On Fri, Jan 12, 2007 at 06:38:26PM +0100, Alex Riesen wrote:

Show 12 quoted lines

> On 1/12/07, Sergey Vlasov <vsu@altlinux.ru> wrote:
> >> Subject: [PATCH] merge-recursive: do not use on-file index when not 
> >needed.
> >
> >This commit broke t3401-rebase-partial.sh:
> >
> >...
> >*   ok 3: rebase topic branch against new master and check git-am did not 
> >get halted
> >
> 
> Hmm... Can't reproduce. Do you have your own patches in the tree?

I had when I encountered the problem, but then retested with clean 'master' (4494c656e2e29c468c48c9c2b20595342056e9dc) and got the same crash. But since the problem is an uninitialized stack variable, you can get anything depending on the phase of the moon from it (however, Valgrind should still be able to catch it).