Today's patches
Matt Seitz (matseitz)
matseitz at cisco.com
Tue May 5 23:58:48 UTC 2015
Maybe somebody wanted something that looked like "fifo feed"?
-----Original Message-----
From: inn-workers-bounces at lists.isc.org [mailto:inn-workers-bounces at lists.isc.org] On Behalf Of Julien ÉLIE
Sent: Tuesday, May 05, 2015 12:11
To: inn-workers at lists.isc.org
Subject: Re: Today's patches
Hi Richard,
> /* A magic number for the group.index file so that we can later change the
> format in a backward-compatible fashion. */
> #define TDX_MAGIC (~(0xf1f0f33d))
>
> Why is it defined in the inverse? i.e. why not the much clearer:
>
> #define TDX_MAGIC 0x0e0f0cc2
I admit I do not know. Leaving this as-is does no harm, though.
> Subject: [PATCH 1/3] Initialize ARTHANDLE objects before passing by
> value
> Subject: [PATCH 3/3] Correct remap check in tradindexed group lookup
Both committed. Thanks!
> Subject: [PATCH 2/3] Remove redundant (broken!) code
>
> The check was (i) off by one and (ii) can never happen, given the loop
> condition.
At the end of the loop, we have:
parent = &entry->next.recno;
current = *parent;
Can't we imagine that the group index still has not been remaped after the creation of a new newsgroup (therefore situated at index->count)?
Then, index->count is the next.recno and we should check that the same way you did for patch #3.
"while (current >= 0)" only and check whether "current >= index->count"?
Wouldn't it make sense to do that?
--
Julien ÉLIE
« Et devant cette levée de boucliers, moi, je lève le camp ! »
(Astérix)
_______________________________________________
inn-workers mailing list
inn-workers at lists.isc.org
https://lists.isc.org/mailman/listinfo/inn-workers
More information about the inn-workers
mailing list