From: Junio C Hamano Date: Tue, 09 Jan 2024 22:34:38 GMT Subject: Re: [PATCH v7 1/1] mingw: give more details about unsafe directory's ownership Message-ID: In-Reply-To: Johannes Sixt writes: >> Because len_domain includes the terminating NUL for the domain part, >> (*str)[len_domain-1] is that NUL, no? And that is what you want to >> overwrite to make the two strings into a single >> one . So... > > But after a successful call, len_domain and len_user have been modified > to contain the lengths of the names (not counting the NULs), ... Ah, that is what I missed. Thanks.