# [PATCH] name-hash.c: always initialize dir_next pointer

3 messages from 2011-11-01 to 2011-11-01. Participants: Johannes Sixt, Junio C Hamano, Jeff King.
Thread: https://gitlist.dev/t/28819

## Johannes Sixt, 2011-11-01 22:21

Subject: [PATCH] name-hash.c: always initialize dir_next pointer
Message-ID: <4EB070D2.4040709@kdbg.org>
URL: https://gitlist.dev/e/4EB070D2.4040709%40kdbg.org

```
Test t2021-checkout-overwrite.sh reveals a segfault in 'git add' on a
case-insensitive file system when git is compiled with XMALLOC_POISON
defined. The reason is that 2548183b (fix phantom untracked files when
core.ignorecase is set) added a new member dir_next to struct cache_entry,
but forgot to initialize it in all cases.

Signed-off-by: Johannes Sixt <j6t@kdbg.org>
---
 You can also insert git config core.ignorecase true before the first
 test script in t2021 to see the segfault.

 I actually found the crash with an MSVC debug build, which has something
 like XMALLOC_POISON built-in.

 name-hash.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/name-hash.c b/name-hash.c
index 225dd76..d8d25c2 100644
--- a/name-hash.c
+++ b/name-hash.c
@@ -74,7 +74,7 @@ static void hash_index_entry(struct index_state *istate, struct cache_entry *ce)
 	if (ce->ce_flags & CE_HASHED)
 		return;
 	ce->ce_flags |= CE_HASHED;
-	ce->next = NULL;
+	ce->next = ce->dir_next = NULL;
 	hash = hash_name(ce->name, ce_namelen(ce));
 	pos = insert_hash(hash, ce, &istate->name_hash);
 	if (pos) {
-- 
1.7.7.1.586.ga0958b

```

## Junio C Hamano, 2011-11-01 22:31

Subject: Re: [PATCH] name-hash.c: always initialize dir_next pointer
Message-ID: <7vbosvlbjt.fsf@alter.siamese.dyndns.org>
URL: https://gitlist.dev/e/7vbosvlbjt.fsf%40alter.siamese.dyndns.org
In-Reply-To: <4EB070D2.4040709@kdbg.org>

```
Thanks.

```

## Jeff King, 2011-11-01 22:39

Subject: Re: [PATCH] name-hash.c: always initialize dir_next pointer
Message-ID: <20111101223949.GA15578@sigill.intra.peff.net>
URL: https://gitlist.dev/e/20111101223949.GA15578%40sigill.intra.peff.net
In-Reply-To: <4EB070D2.4040709@kdbg.org>

```
On Tue, Nov 01, 2011 at 11:21:06PM +0100, Johannes Sixt wrote:

> Test t2021-checkout-overwrite.sh reveals a segfault in 'git add' on a
> case-insensitive file system when git is compiled with XMALLOC_POISON
> defined. The reason is that 2548183b (fix phantom untracked files when
> core.ignorecase is set) added a new member dir_next to struct cache_entry,
> but forgot to initialize it in all cases.
> 
> Signed-off-by: Johannes Sixt <j6t@kdbg.org>
> ---
>  You can also insert git config core.ignorecase true before the first
>  test script in t2021 to see the segfault.

Yikes, thanks.

-Peff

```