On Thu, 1 Jun 2023, Todd C. Miller wrote: >My inclination would be to just make the timeout signed to match >time_t. No need for ugly casts then and parse_timeout() parses the For OpenSSH itself, sure. For portable OpenSSH that’s not an option; time_t can be unsigned, signed, or even float. Another option would be to make the variable also time_t, but that would waste much space on contemporary platforms due to Y2038, so I think that, here, the cast is better. bye, //mirabilos -- Infrastrukturexperte • tarent solutions GmbH Am Dickobskreuz 10, D-53121 Bonn • http://www.tarent.de/ Telephon +49 228 54881-393 • Fax: +49 228 54881-235 HRB AG Bonn 5168 • USt-ID (VAT): DE122264941 Geschäftsführer: Dr. Stefan Barth, Kai Ebenrett, Boris Esser, Alexander Steeg **************************************************** /⁀\ The UTF-8 Ribbon ╲ ╱ Campaign against Mit dem tarent-Newsletter nichts mehr verpassen: ╳ HTML eMail! Also, https://www.tarent.de/newsletter ╱ ╲ header encryption! **************************************************** _______________________________________________ openssh-unix-dev mailing list openssh-unix-dev@xxxxxxxxxxx https://lists.mindrot.org/mailman/listinfo/openssh-unix-dev
