On 08/09/2021 22:10, Gustaf Ullberg wrote:
> Thanks Petr for having a look at this.
> Since IDN processing do turns uppercase letters into lowercase, I
> deliberately left uppercase letters out.
> I think your approach to put everything in check_name() makes sense,
> even if that function grows and maybe is starting to become a bit hard
> to read.
> 
> I took your patch and made a couple of changes:
> - IDN processing is performed if there are uppercase letters present in
> the name (unless we have an old version of libidn2 and there is an
> underscore in the name).
> - IDN processing is always performed if there are non-ascii characters
> in the name, no matter if there are underscores or not (a non-processed
> name containing non-ascii characters sounds dangerous).
> 

That's an improvement. Applied in place of Petr's patch.

Cheers,


Simon.

_______________________________________________
Dnsmasq-discuss mailing list
Dnsmasq-discuss@lists.thekelleys.org.uk
https://lists.thekelleys.org.uk/cgi-bin/mailman/listinfo/dnsmasq-discuss

Reply via email to