Odds and ends
Mark Atwood
fallenpegasus at gmail.com
Wed Jun 6 14:44:29 UTC 2018
>> The code for freeing up key strings zeros them out first. How do we do
that in python or go?
> Same way you would in C, iterate over them writing zeros. Am I missing
something?
I may be not understanding something.
I thought that strings were non-mutatable in Python, and every time one is
written to, the interpreter actually allocates a new string and eventually
GCs the old value.
..m
On Wed, Jun 6, 2018 at 4:35 AM Eric S. Raymond via devel <devel at ntpsec.org>
wrote:
> Hal Murray via devel <devel at ntpsec.org>:
> > I occasionally add msyslog lines when debugging. The DPRINTF stuff
> isn't
> > interesting - too much junk I don't want. When I'm cleaning up, should
> I
> > disable them with "if (0)", or delete them? Is there a better way? ...
>
> What I do is delete them unless I think they might have continuing value,
> in which case I put them under DPRINTF.
>
> > Some/many of the prototypes in our header files are not very useful. I
> think the problem is the lack of names on the parameters. If the types are
> all different, that's enough, but if there are several integer parameters
> that doesn't help. Here is an example:
> > extern int clocktime (int, int, int, int, int, time_t,
> uint32_t, uint32_t *, uint32_t *);
>
> Fair point. Are you advocating outting the formal names back in?
>
> > There is still some broadcast/multicast stuff around. grep for CAST
> will find them. Should we leave enough around so that our ntpq can decode
> things when talking to old servers? If so, I vote we move them to
> ntpclients/
>
> I agree with moving them to ntpclients. Anything that reeduces the
> amount of
> ceuft ubthe ntpd code is good.
>
> > Some of the counters that ntpq can display get written to a log file
> every hour. That resets the counters so ntpq can only see what has
> happened since then rather than the totals since startup. We should
> probably maintain and display both.
>
> Reasonable. Would you make ntpq parse the logfile?
>
> > It would be nice if ntpq would show something if a server is using
> authentication. I'd be willing to steal a character from the name column.
>
> No need. Just boldface the ones with authentication.
>
> > The code for freeing up key strings zeros them out first. How do we do
> that in python or go?
>
> Same way you would in C, iterate over them writing zeros. Am I
> missing something?
> --
> <a href="http://www.catb.org/~esr/">Eric S. Raymond</a>
>
> My work is funded by the Internet Civil Engineering Institute:
> https://icei.org
> Please visit their site and donate: the civilization you save might be
> your own.
>
>
> _______________________________________________
> devel mailing list
> devel at ntpsec.org
> http://lists.ntpsec.org/mailman/listinfo/devel
>
--
Mark Atwood
http://about.me/markatwood
+1-206-604-2198
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ntpsec.org/pipermail/devel/attachments/20180606/1c89b733/attachment.html>
More information about the devel
mailing list