[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#54352] [PATCH] services: dnsmasq: Add more options.
From: |
Ludovic Courtès |
Subject: |
[bug#54352] [PATCH] services: dnsmasq: Add more options. |
Date: |
Mon, 21 Mar 2022 16:22:36 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
Hi Maxime,
Maxime Devos <maximedevos@telenet.be> skribis:
> Remco van 't Veer schreef op zo 20-03-2022 om 12:44 [+0100]:
>> + (forward-private-reverse-lookup?
>> + dnsmasq-configuration-forward-private-reverse-lookup?
>> + (default #t)) ;boolean
>> + (strict-order? dnsmasq-configuration-strict-order?
>> + (default #f)) ;boolean
>
> It would be nice to verify that these are, in fact, booleans,
> using field sanitizers. See, e.g., ensure-setuid-program-list
> in (gnu system).
I think this suggestion is beyond the scope of this review: we’ve never
used sanitizers like this before (or almost), and this particular piece
of code doesn’t use them.
Also, with the recent discussion about the introduction of contracts,
I’d rather wait an use contracts everywhere once they’re available.
Thoughts?
Thanks,
Ludo’.
- [bug#54352] [PATCH] services: dnsmasq: Add more options., Remco van 't Veer, 2022/03/12
- [bug#54352] [PATCH] services: dnsmasq: Add more options., Ludovic Courtès, 2022/03/19
- [bug#54352] [PATCH v2] services: dnsmasq: Add more options., Remco van 't Veer, 2022/03/20
- [bug#54352] [PATCH] services: dnsmasq: Add more options.,
Ludovic Courtès <=
- [bug#54352] [PATCH] services: dnsmasq: Add more options., Maxime Devos, 2022/03/21
- [bug#54352] [PATCH] services: dnsmasq: Add more options., Remco van 't Veer, 2022/03/22
- [bug#54352] [PATCH] services: dnsmasq: Add more options., Ludovic Courtès, 2022/03/22
- [bug#54352] [PATCH] services: dnsmasq: Add more options., Remco van 't Veer, 2022/03/23
[bug#54352] [PATCH v2] services: dnsmasq: Add more options., Maxime Devos, 2022/03/20
[bug#54352] [PATCH v2] services: dnsmasq: Add more options., Maxime Devos, 2022/03/20