Skip to content

Commit d0d61a7

Browse files
author
Steve Hay
committed
Subject: [PATCH] Don't call modperl_threaded_mpm() et al. from XS code
From: Niko Tyni <[email protected]> Date: Wed, 15 Oct 2014 11:20:02 +0300 Message-ID: <[email protected]> Fixes: https://bugs.debian.org/765174 git-svn-id: https://svn.apache.org/repos/asf/perl/modperl/trunk@1632172 13f79535-47bb-0310-9956-ffa450edef68
1 parent 7589bca commit d0d61a7

File tree

2 files changed

+5
-0
lines changed

2 files changed

+5
-0
lines changed

Changes

+3
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,9 @@ Also refer to the Apache::Test changes log file, at Apache-Test/Changes
1212

1313
=item 2.0.9-dev
1414

15+
Don't call modperl_threaded_mpm() et al. from XS code. Fixes Debian Bug
16+
#765174. [Niko Tyni <[email protected]>]
17+
1518
Make sure modperl_interp_select uses r->server rather than the passed s
1619
parameter to find the interpreter pool to pull an interpreter from. This
1720
fixes an issue with vhosts with a separate interpreter pool and runtime

src/modules/perl/modperl_common_log.c

+2
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,7 @@ void modperl_trace(const char *func, const char *fmt, ...)
5353
http://apr.apache.org/docs/apr/1.4/group__apr__lib.html#gad2cd3594aeaafd45931d1034965f48c1
5454
*/
5555

56+
#ifndef MP_IN_XS
5657
/* PERL_GET_CONTEXT yields nonsense until the first interpreter is
5758
* created. Hence the modperl_is_running() question. */
5859
if (modperl_threaded_mpm()) {
@@ -77,6 +78,7 @@ void modperl_trace(const char *func, const char *fmt, ...)
7778
apr_file_printf(logfile, "[pid=%lu] ", (unsigned long)getpid());
7879
#endif
7980
}
81+
#endif
8082

8183
if (func && *func) {
8284
apr_file_printf(logfile, "%s: ", func);

0 commit comments

Comments
 (0)