[PATCH] resolv: AAAA (28) is valid, no fallthrough to default
Florian Weimer
fweimer@redhat.com
Thu Jan 10 11:01:00 GMT 2019
* Andreas K. Hüttel:
> The test resolv/tst-resolv-ai_idn-nolibidn2 kept failing, with the puzzling
> output "invalid qtype: 28". Puzzling since 28 is an AAAA record.
> Turns out the switch statement is missing a break, so the "T_AAAA" case
> continues with "default".
> ---
> resolv/tst-resolv-ai_idn-common.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/resolv/tst-resolv-ai_idn-common.c b/resolv/tst-resolv-ai_idn-common.c
> index bb81c2ca23..61cc81b71f 100644
> --- a/resolv/tst-resolv-ai_idn-common.c
> +++ b/resolv/tst-resolv-ai_idn-common.c
> @@ -241,6 +241,7 @@ response (const struct resolv_response_context *ctx,
> qname[0] };
> resolv_response_add_data (b, &addr, sizeof (addr));
> }
> + break;
> default:
> FAIL_EXIT1 ("invalid qtype: %d", qtype);
> }
The change is obviously correct. I will commit it for you.
However, how do you encounter this issue? Did you change some test?
The current IDN tests do not actually send AAAA queries, I think.
Thanks,
Florian
More information about the Libc-alpha
mailing list