On Mon, Aug 5, 2013 at 7:19 AM, Faidon Liambotis <parav...@debian.org>wrote:
> "timeout" is nowadays a duration, not an absolute value. Fix the trace2
> output to print it as such, instead of subtracting time(NULL) and
> resulting in lines such as:
>
> [Mon Aug 05 03:51:07.369625 2013] [ssl:trace2] [pid 7916:tid
> 140003006699264] ssl_engine_kernel.c(1698): Inter-Process Session Cache:
> request=SET status=OK
> id=e53934c8b21017256ebe00327be265ccbb82bbaaef52adb900a884a07ad32eac
> timeout=-1375674367s (session caching)
>
> --- modules/ssl/ssl_engine_kernel.c (revision 1510425)
> +++ modules/ssl/ssl_engine_kernel.c (working copy)
> @@ -1687,7 +1687,7 @@
>
> if (timeout) {
> apr_snprintf(timeout_str, sizeof(timeout_str),
> - "timeout=%lds ", (timeout - time(NULL)));
> + "timeout=%lds ", timeout);
> }
>
> ap_log_error(APLOG_MARK, APLOG_TRACE2, 0, s,
>
Someone else committed this to trunk yesterday (
http://svn.apache.org/viewvc?view=revision&revision=r1510098), and it now
has 2 of 3 necessary votes to appear in the next 2.4.x release.
--
Born in Roswell... married an alien...
http://emptyhammock.com/
