patchSubject: [PATCH] object-file: fix sparse 'plain integer as NULL pointer' error

6 messages between Mar 19, 2026 and Mar 20, 2026, from Ramsay Jones, Junio C Hamano, Patrick Steinhardt.

Plain Markdown or JSON for tools and agents. Diffs are folded; open one to read it.

Ramsay JonesMar 19, 2026, 22:49 UTC on lore

Signed-off-by: Ramsay Jones <ramsay@ramsayjones.plus.com>
---

Hi Patrick,

When you next re-roll your 'ps/object-counting' branch, could you please squash this into the patch equivalent to the commit 2b24db1110 ("object-file: generalize counting objects",2026-03-12) in tonight's 'seen' branch.

Thanks,

ATB, Ramsay Jones

 object-file.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Show changes to object-file.c +1 −1

diff --git a/object-file.c b/object-file.c
index 35be7e58cb..3708b8f18d 100644
--- a/object-file.c
+++ b/object-file.c
@@ -1941,7 +1941,7 @@ int odb_source_loose_count_objects(struct odb_source *source,
 	} else {
 		*out = 0;
 		ret = odb_source_loose_for_each_object(source, NULL, count_loose_object,
-						       out, 0);
+						       out, NULL);
 	}
 
 out:

-- 
2.53.0

Junio C HamanoMar 20, 2026, 01:35 UTC in reply to Ramsay Jones on lore

Re: Subject: [PATCH] object-file: fix sparse 'plain integer as NULL pointer' error

Ramsay Jones <ramsay@ramsayjones.plus.com> writes:

Show 8 quoted lines

> Signed-off-by: Ramsay Jones <ramsay@ramsayjones.plus.com>
> ---
>
> Hi Patrick,
>
> When you next re-roll your 'ps/object-counting' branch, could you please squash this
> into the patch equivalent to the commit 2b24db1110 ("object-file: generalize counting
> objects",2026-03-12) in tonight's 'seen' branch.

The topic being in 'next' since March 17th, that is a bit awkward to arrange. I can queue the fix on top instead.

Show 23 quoted lines

>
> Thanks,
>
> ATB,
> Ramsay Jones
>
>
>  object-file.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/object-file.c b/object-file.c
> index 35be7e58cb..3708b8f18d 100644
> --- a/object-file.c
> +++ b/object-file.c
> @@ -1941,7 +1941,7 @@ int odb_source_loose_count_objects(struct odb_source *source,
>  	} else {
>  		*out = 0;
>  		ret = odb_source_loose_for_each_object(source, NULL, count_loose_object,
> -						       out, 0);
> +						       out, NULL);
>  	}
>  
>  out:

Ramsay JonesMar 20, 2026, 02:04 UTC in reply to Junio C Hamano on lore

Re: Subject: [PATCH] object-file: fix sparse 'plain integer as NULL pointer' error

On 20/03/2026 1:35 am, Junio C Hamano wrote:

Show 13 quoted lines

> Ramsay Jones <ramsay@ramsayjones.plus.com> writes:
> 
>> Signed-off-by: Ramsay Jones <ramsay@ramsayjones.plus.com>
>> ---
>>
>> Hi Patrick,
>>
>> When you next re-roll your 'ps/object-counting' branch, could you please squash this
>> into the patch equivalent to the commit 2b24db1110 ("object-file: generalize counting
>> objects",2026-03-12) in tonight's 'seen' branch.
> 
> The topic being in 'next' since March 17th, that is a bit awkward to
> arrange.  I can queue the fix on top instead.

Hmm, odd. I didn't do a 'branch --contains', obviously, but this only started tonight (and I built git yesterday - master, next and seen), so I just assumed ... ;)

Sorry about that.

Thanks.

ATB, Ramsay Jones

Ramsay JonesMar 20, 2026, 03:44 UTC in reply to Ramsay Jones on lore

Re: Subject: [PATCH] object-file: fix sparse 'plain integer as NULL pointer' error

On 20/03/2026 02:04, Ramsay Jones wrote:

Show 19 quoted lines

>
> On 20/03/2026 1:35 am, Junio C Hamano wrote:
>> Ramsay Jones <ramsay@ramsayjones.plus.com> writes:
>>
>>> Signed-off-by: Ramsay Jones <ramsay@ramsayjones.plus.com>
>>> ---
>>>
>>> Hi Patrick,
>>>
>>> When you next re-roll your 'ps/object-counting' branch, could you please squash this
>>> into the patch equivalent to the commit 2b24db1110 ("object-file: generalize counting
>>> objects",2026-03-12) in tonight's 'seen' branch.
>> The topic being in 'next' since March 17th, that is a bit awkward to
>> arrange.  I can queue the fix on top instead.
> Hmm, odd. I didn't do a 'branch --contains', obviously, but this only started tonight
> (and I built git yesterday - master, next and seen), so I just assumed ... ;)
>
> Sorry about that.
>

Heh, I could not sleep, so took another look! :)

The problem was actually caused by the 'ps/odb-generic-object-name-handling' branch and commit 936c2bfecb ("odb: introduce

`struct odb_for_each_object_options`", 2026-03-19). In particular, that commit changes the type of the last parameter of the

odb_source_loose_for_each_object() function from 'unsigned' to 'const struct odb_for_each_object_options *', without

changing one call site from '0' to 'NULL'.

OK, I can go to bed now!

Thanks.

ATB,

Ramsay Jones

Junio C HamanoMar 20, 2026, 06:05 UTC in reply to Ramsay Jones on lore

Re: Subject: [PATCH] object-file: fix sparse 'plain integer as NULL pointer' error

Ramsay Jones <ramsay@ramsayjones.plus.com> writes:

Show 19 quoted lines

> On 20/03/2026 1:35 am, Junio C Hamano wrote:
>> Ramsay Jones <ramsay@ramsayjones.plus.com> writes:
>> 
>>> Signed-off-by: Ramsay Jones <ramsay@ramsayjones.plus.com>
>>> ---
>>>
>>> Hi Patrick,
>>>
>>> When you next re-roll your 'ps/object-counting' branch, could you please squash this
>>> into the patch equivalent to the commit 2b24db1110 ("object-file: generalize counting
>>> objects",2026-03-12) in tonight's 'seen' branch.
>> 
>> The topic being in 'next' since March 17th, that is a bit awkward to
>> arrange.  I can queue the fix on top instead.
>
> Hmm, odd. I didn't do a 'branch --contains', obviously, but this only started tonight
> (and I built git yesterday - master, next and seen), so I just assumed ... ;)
>
> Sorry about that.

FWIW, my local build & test did not catch this until today, only because the version of sparse I was running was a tad old. After updating it today, I started seeing the breakage there.

Patrick SteinhardtMar 20, 2026, 06:14 UTC in reply to Ramsay Jones on lore

Re: Subject: [PATCH] object-file: fix sparse 'plain integer as NULL pointer' error

On Fri, Mar 20, 2026 at 03:44:08AM +0000, Ramsay Jones wrote:

Show 38 quoted lines

> 
> On 20/03/2026 02:04, Ramsay Jones wrote:
> > 
> > On 20/03/2026 1:35 am, Junio C Hamano wrote:
> > > Ramsay Jones <ramsay@ramsayjones.plus.com> writes:
> > > 
> > > > Signed-off-by: Ramsay Jones <ramsay@ramsayjones.plus.com>
> > > > ---
> > > > 
> > > > Hi Patrick,
> > > > 
> > > > When you next re-roll your 'ps/object-counting' branch, could you please squash this
> > > > into the patch equivalent to the commit 2b24db1110 ("object-file: generalize counting
> > > > objects",2026-03-12) in tonight's 'seen' branch.
> > > The topic being in 'next' since March 17th, that is a bit awkward to
> > > arrange.  I can queue the fix on top instead.
> > Hmm, odd. I didn't do a 'branch --contains', obviously, but this only started tonight
> > (and I built git yesterday - master, next and seen), so I just assumed ... ;)
> > 
> > Sorry about that.
> > 
> 
> Heh, I could not sleep, so took another look! :)
> 
> 
> The problem was actually caused by the 'ps/odb-generic-object-name-handling'
> branch and commit 936c2bfecb ("odb: introduce
> 
> `struct odb_for_each_object_options`", 2026-03-19). In particular,  that
> commit changes the type of the last parameter of the
> 
> odb_source_loose_for_each_object() function from 'unsigned' to 'const struct
> odb_for_each_object_options *', without
> 
> changing one call site from '0' to 'NULL'.
> 
> 
> OK, I can go to bed now!

Thanks for catching this! Fixed locally now.

Patrick