[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#54352] [PATCH v2] services: dnsmasq: Add more options.
From: |
Remco van 't Veer |
Subject: |
[bug#54352] [PATCH v2] services: dnsmasq: Add more options. |
Date: |
Sun, 20 Mar 2022 13:22:00 +0100 |
User-agent: |
mu4e 1.6.10; emacs 27.2 |
2022/03/20 12:56, Maxime Devos:
> 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 agree but the same could be said about the other fields and types in
this record, and those of other services. In this case, the names of
the fields ending with "?" should be enough for somebody to realise this
is a boolean, IMHO. The ";boolean" comments I've added are just me
trying to blend in.
- [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, 2022/03/21
- [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