Patrick Steinhardt <ps@xxxxxx> writes:
> On Tue, Apr 13, 2021 at 06:11:39PM -0700, Junio C Hamano wrote:
>> On 'seen' there are many topics that have not seen adequately
>> reviews; some tests are broken near its tip (I think they pass the
>> selftests by themselves).
>>
>> 0e76df05ca Merge branch 'ps/rev-list-object-type-filter' into seen
>> 956fbceb2e ### breaks 6112 6113
>> c007303ad4 Merge branch 'bc/hash-transition-interop-part-1' into seen
>> 4813f16161 ### breaks 0031
>
> Test breakage for the rev-list filter series has been a bad interaction
> of d8883ed540 (object.c: stop supporting len == -1 in
> type_from_string_gently(), 2021-03-28) and
> ps/rev-list-object-type-filter.
Yuck. The commit d8883ed5 (object.c: stop supporting len == -1 in
type_from_string_gently(), 2021-03-28) by itself may have meant
well, but in an environment where concurrent group development is
the norm, it is a change that can unnecessarily break other topics
easily, like it just did.
Perhaps a band-aid like this may be necessary.
diff --git i/object.c w/object.c
index 68de229f1a..b4bde9e444 100644
--- i/object.c
+++ w/object.c
@@ -39,6 +39,9 @@ int type_from_string_gently(const char *str, ssize_t len, int gentle)
{
int i;
+ if (len < 0)
+ BUG("type-from-string-gently no longer allows unspecified length");
+
for (i = 1; i < ARRAY_SIZE(object_type_strings); i++)
if (!strncmp(str, object_type_strings[i], len) &&
object_type_strings[i][len] == '\0')
> The following patch fixes this, which
> I'll include in my next reroll of this series.
Thanks.
> diff --git a/list-objects-filter-options.c b/list-objects-filter-options.c
> index 19e128ef11..33c7718a7a 100644
> --- a/list-objects-filter-options.c
> +++ b/list-objects-filter-options.c
> @@ -100,7 +100,7 @@ static int gently_parse_list_objects_filter(
> return 1;
>
> } else if (skip_prefix(arg, "object:type=", &v0)) {
> - int type = type_from_string_gently(v0, -1);
> + int type = type_from_string_gently(v0, strlen(v0));
> if (type < 0) {
> strbuf_addstr(errbuf, _("expected 'object:type=<type>'"));
> return 1;
