https://bugzilla.mindrot.org/show_bug.cgi?id=2464

--- Comment #2 from Darren Tucker <[email protected]> ---
Comment on attachment 2740
  --> https://bugzilla.mindrot.org/attachment.cgi?id=2740
adding timestamp to debug messages

>+#include <sys/timeb.h>

style(9) says this should go with the other <sys/...> includes. 

>+              ftime (&tb);

ftime is obsolete and has been removed from POSIX.1-2008.  Should use
gettimeofday instead.

>+              localtime_r (&tb.time, &tm);
>+              p += strftime (p, 16, "%Y%m%d.%H%M%S", &tm);

I'm not a fan of unnecessary pointer arithmetic.  Since this is neither
time nor space critical, I'd rather strftime into a static buffer then
use snprintf to assemble the final string.

>+              p += sprintf (p, ".%03d ", tb.millitm);

also not a fan of unbounded string functions even if this particular
one is safe.

-- 
You are receiving this mail because:
You are watching someone on the CC list of the bug.
You are watching the assignee of the bug.
_______________________________________________
openssh-bugs mailing list
[email protected]
https://lists.mindrot.org/mailman/listinfo/openssh-bugs

Reply via email to