From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeff King Subject: Re: [PATCH] index-pack: always zero-initialize object_entry list Date: Tue, 19 Mar 2013 11:52:44 -0400 Message-ID: <20130319155244.GA16532@sigill.intra.peff.net> References: <20130319102422.GB6341@sigill.intra.peff.net> <20130319105852.GA15182@sigill.intra.peff.net> <8738vr5rqh.fsf@pctrast.inf.ethz.ch> <20130319154353.GA10010@sigill.intra.peff.net> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Cc: Junio C Hamano , Stefan Zager , git@vger.kernel.org, =?utf-8?B?Tmd1eeG7hW4gVGjDoWkgTmfhu41j?= Duy To: Thomas Rast X-From: git-owner@vger.kernel.org Tue Mar 19 16:53:18 2013 Return-path: Envelope-to: gcvg-git-2@plane.gmane.org Received: from vger.kernel.org ([209.132.180.67]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1UHyqk-0007pT-1h for gcvg-git-2@plane.gmane.org; Tue, 19 Mar 2013 16:53:18 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932566Ab3CSPwu (ORCPT ); Tue, 19 Mar 2013 11:52:50 -0400 Received: from 75-15-5-89.uvs.iplsin.sbcglobal.net ([75.15.5.89]:58518 "EHLO peff.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932489Ab3CSPwt (ORCPT ); Tue, 19 Mar 2013 11:52:49 -0400 Received: (qmail 30969 invoked by uid 107); 19 Mar 2013 15:54:32 -0000 Received: from sigill.intra.peff.net (HELO sigill.intra.peff.net) (10.0.0.7) (smtp-auth username relayok, mechanism cram-md5) by peff.net (qpsmtpd/0.84) with ESMTPA; Tue, 19 Mar 2013 11:54:32 -0400 Received: by sigill.intra.peff.net (sSMTP sendmail emulation); Tue, 19 Mar 2013 11:52:44 -0400 Content-Disposition: inline In-Reply-To: <20130319154353.GA10010@sigill.intra.peff.net> Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: On Tue, Mar 19, 2013 at 11:43:53AM -0400, Jeff King wrote: > On Tue, Mar 19, 2013 at 04:33:42PM +0100, Thomas Rast wrote: > > > Jeff King writes: > > > > > Commit 38a4556 (index-pack: start learning to emulate > > > "verify-pack -v", 2011-06-03) added a "delta_depth" counter > > > to each "struct object_entry". Initially, all object entries > > > have their depth set to 0; in resolve_delta, we then set the > > > depth of each delta to "base + 1". Base entries never have > > > their depth touched, and remain at 0. > > > > This patch causes index-pack to fail on the pack that triggered the > > whole discussion. More in a minute in another side thread, but > > meanwhile: NAK until we understand what is really going on here. > > Odd; that's what I was testing with, and it worked fine. Ah, interesting. I built the fix on top of d1a0ed1, the first commit that shows the problem. And it works fine there. But when it is forward-ported to the current master, it breaks as you saw. More bisection fun. -Peff