GNU bug report logs - #54352
[PATCH] services: dnsmasq: Add more options.

Previous Next

Full log

Message #59 received at 54352 <at> debbugs.gnu.org (full text, mbox):

From: Maxime Devos <maximedevos <at> telenet.be>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 54352 <at> debbugs.gnu.org, Remco van 't Veer <remco <at> remworks.net>
Subject: Re: bug#54352: [PATCH] services: dnsmasq: Add more options.
Date: Mon, 21 Mar 2022 19:36:41 +0100

[Message part 1 (text/plain, inline)]

Ludovic Courtès schreef op ma 21-03-2022 om 16:22 [+0100]:
> 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.

Seems reasonable to me, given that the specifics weren't discussed yet,
although _everywhere_ (for all procedures, records, ...) seems a bit
much, unless you meant every field of the dnsmasq record.

Greetings,
Maxime.

[signature.asc (application/pgp-signature, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.