[Bug 39737] LogFormat "%{tid}P" reports wrong thread id on Windows

[bu...@apache.org]

https://bz.apache.org/bugzilla/show_bug.cgi?id=39737

--- Comment #5 from William A. Rowe Jr. <wr...@apache.org> ---
apr_os_thread_current returns a handle (OS representation of a manipulatable
thread.) Similarly apr_os_thread_get. That behavior is correct.

mod_log_custom is doing this;

   apr_os_thread_t tid = apr_os_thread_current();

That isn't portable, but OS specific.

core.c is grabbing this for the pid, not using apr;

    return apr_psprintf(p, "%s%s%" APR_PID_T_FMT, string,
                        delim, getpid());

It seems we simply should switch to gettid(), and presume it is portable?

Interesting there is no APR_TID_T_FMT.

Also odd that apr_pid_t doesn't exist, but APR_PID_T_FMT is defined.

Perhaps it is possible to presume sizeof(pid_t) == sizeof(tid_t)?

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org