Mercurial > hg > nginx
annotate src/http/modules/perl/ngx_http_perl_module.c @ 6820:eada22643e8b
Perl: added PERL_SET_INTERP().
For Perl compiled with threads, without PERL_SET_INTERP() the PL_curinterp
remains set to the first interpreter created (that is, one created at
original start). As a result after a reload Perl thinks that operations
are done withing a thread, and, most notably, denies to change environment.
For example, the following code properly works on original start,
but fails after a reload:
perl 'sub {
my $r = shift;
$r->send_http_header("text/plain");
$ENV{TZ} = "UTC";
$r->print("tz: " . $ENV{TZ} . " (localtime " . (localtime()) . ")\n");
$ENV{TZ} = "Europe/Moscow";
$r->print("tz: " . $ENV{TZ} . " (localtime " . (localtime()) . ")\n");
return OK;
}';
To fix this, PERL_SET_INTERP() added anywhere where PERL_SET_CONTEXT()
was previously used.
Note that PERL_SET_INTERP() doesn't seem to be documented anywhere.
Yet it is used in some other software, and also seems to be the only
solution possible.
| author | Maxim Dounin <mdounin@mdounin.ru> |
|---|---|
| date | Wed, 07 Dec 2016 19:03:19 +0300 |
| parents | cb4a4e9bba8e |
| children | 1c5e5e5b008d |
| rev | line source |
|---|---|
| 599 | 1 |
| 2 /* | |
| 3 * Copyright (C) Igor Sysoev | |
| 4412 | 4 * Copyright (C) Nginx, Inc. |
| 599 | 5 */ |
| 6 | |
| 7 | |
| 8 #include <ngx_config.h> | |
| 9 #include <ngx_core.h> | |
| 10 #include <ngx_http.h> | |
| 11 #include <ngx_http_perl_module.h> | |
| 12 | |
| 13 | |
| 14 typedef struct { | |
| 15 PerlInterpreter *perl; | |
|
873
f92ad15c2db1
fix segfaults in future workers those will use old configuration,
Igor Sysoev <igor@sysoev.ru>
parents:
872
diff
changeset
|
16 HV *nginx; |
| 3175 | 17 ngx_array_t *modules; |
|
3174
479fd46cd1c4
use ngx_conf_set_str_array_slot() for perl_require
Igor Sysoev <igor@sysoev.ru>
parents:
3173
diff
changeset
|
18 ngx_array_t *requires; |
| 599 | 19 } ngx_http_perl_main_conf_t; |
| 20 | |
| 21 | |
| 22 typedef struct { | |
| 23 SV *sub; | |
| 24 ngx_str_t handler; | |
| 25 } ngx_http_perl_loc_conf_t; | |
| 26 | |
| 27 | |
| 28 typedef struct { | |
| 29 SV *sub; | |
| 30 ngx_str_t handler; | |
| 31 } ngx_http_perl_variable_t; | |
| 32 | |
| 33 | |
| 617 | 34 #if (NGX_HTTP_SSI) |
| 599 | 35 static ngx_int_t ngx_http_perl_ssi(ngx_http_request_t *r, |
| 36 ngx_http_ssi_ctx_t *ssi_ctx, ngx_str_t **params); | |
| 617 | 37 #endif |
| 38 | |
| 599 | 39 static char *ngx_http_perl_init_interpreter(ngx_conf_t *cf, |
| 40 ngx_http_perl_main_conf_t *pmcf); | |
| 1069 | 41 static PerlInterpreter *ngx_http_perl_create_interpreter(ngx_conf_t *cf, |
| 42 ngx_http_perl_main_conf_t *pmcf); | |
| 653 | 43 static ngx_int_t ngx_http_perl_run_requires(pTHX_ ngx_array_t *requires, |
| 649 | 44 ngx_log_t *log); |
| 599 | 45 static ngx_int_t ngx_http_perl_call_handler(pTHX_ ngx_http_request_t *r, |
|
1899
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
46 HV *nginx, SV *sub, SV **args, ngx_str_t *handler, ngx_str_t *rv); |
| 599 | 47 static void ngx_http_perl_eval_anon_sub(pTHX_ ngx_str_t *handler, SV **sv); |
| 48 | |
| 49 static ngx_int_t ngx_http_perl_preconfiguration(ngx_conf_t *cf); | |
| 50 static void *ngx_http_perl_create_main_conf(ngx_conf_t *cf); | |
| 51 static char *ngx_http_perl_init_main_conf(ngx_conf_t *cf, void *conf); | |
| 52 static void *ngx_http_perl_create_loc_conf(ngx_conf_t *cf); | |
| 53 static char *ngx_http_perl_merge_loc_conf(ngx_conf_t *cf, void *parent, | |
| 54 void *child); | |
| 55 static char *ngx_http_perl(ngx_conf_t *cf, ngx_command_t *cmd, void *conf); | |
| 56 static char *ngx_http_perl_set(ngx_conf_t *cf, ngx_command_t *cmd, void *conf); | |
|
872
1c4a5b3f9110
Axe several perl interpreter instances: they may be useful in currently
Igor Sysoev <igor@sysoev.ru>
parents:
871
diff
changeset
|
57 |
|
1c4a5b3f9110
Axe several perl interpreter instances: they may be useful in currently
Igor Sysoev <igor@sysoev.ru>
parents:
871
diff
changeset
|
58 #if (NGX_HAVE_PERL_MULTIPLICITY) |
| 599 | 59 static void ngx_http_perl_cleanup_perl(void *data); |
| 649 | 60 #endif |
| 599 | 61 |
| 1257 | 62 static ngx_int_t ngx_http_perl_init_worker(ngx_cycle_t *cycle); |
|
1221
aa653367028e
PERL_SYS_TERM() should be called once on exit only, this fixes the message
Igor Sysoev <igor@sysoev.ru>
parents:
1165
diff
changeset
|
63 static void ngx_http_perl_exit(ngx_cycle_t *cycle); |
|
aa653367028e
PERL_SYS_TERM() should be called once on exit only, this fixes the message
Igor Sysoev <igor@sysoev.ru>
parents:
1165
diff
changeset
|
64 |
| 599 | 65 |
| 66 static ngx_command_t ngx_http_perl_commands[] = { | |
| 67 | |
| 68 { ngx_string("perl_modules"), | |
| 69 NGX_HTTP_MAIN_CONF|NGX_CONF_TAKE1, | |
| 3175 | 70 ngx_conf_set_str_array_slot, |
| 599 | 71 NGX_HTTP_MAIN_CONF_OFFSET, |
| 72 offsetof(ngx_http_perl_main_conf_t, modules), | |
| 73 NULL }, | |
| 74 | |
| 75 { ngx_string("perl_require"), | |
| 76 NGX_HTTP_MAIN_CONF|NGX_CONF_TAKE1, | |
|
3174
479fd46cd1c4
use ngx_conf_set_str_array_slot() for perl_require
Igor Sysoev <igor@sysoev.ru>
parents:
3173
diff
changeset
|
77 ngx_conf_set_str_array_slot, |
| 599 | 78 NGX_HTTP_MAIN_CONF_OFFSET, |
|
3174
479fd46cd1c4
use ngx_conf_set_str_array_slot() for perl_require
Igor Sysoev <igor@sysoev.ru>
parents:
3173
diff
changeset
|
79 offsetof(ngx_http_perl_main_conf_t, requires), |
| 599 | 80 NULL }, |
| 81 | |
| 82 { ngx_string("perl"), | |
|
922
a5b9cdfe3e19
allow "perl" inside "limit_except"
Igor Sysoev <igor@sysoev.ru>
parents:
913
diff
changeset
|
83 NGX_HTTP_LOC_CONF|NGX_HTTP_LMT_CONF|NGX_CONF_TAKE1, |
| 599 | 84 ngx_http_perl, |
| 85 NGX_HTTP_LOC_CONF_OFFSET, | |
| 86 0, | |
| 87 NULL }, | |
| 88 | |
| 89 { ngx_string("perl_set"), | |
| 90 NGX_HTTP_MAIN_CONF|NGX_CONF_TAKE2, | |
| 91 ngx_http_perl_set, | |
| 92 NGX_HTTP_LOC_CONF_OFFSET, | |
| 93 0, | |
| 94 NULL }, | |
| 95 | |
| 96 ngx_null_command | |
| 97 }; | |
| 98 | |
| 99 | |
| 100 static ngx_http_module_t ngx_http_perl_module_ctx = { | |
| 101 ngx_http_perl_preconfiguration, /* preconfiguration */ | |
| 102 NULL, /* postconfiguration */ | |
| 103 | |
| 104 ngx_http_perl_create_main_conf, /* create main configuration */ | |
| 105 ngx_http_perl_init_main_conf, /* init main configuration */ | |
| 106 | |
| 107 NULL, /* create server configuration */ | |
| 108 NULL, /* merge server configuration */ | |
| 109 | |
| 110 ngx_http_perl_create_loc_conf, /* create location configuration */ | |
| 111 ngx_http_perl_merge_loc_conf /* merge location configuration */ | |
| 112 }; | |
| 113 | |
| 114 | |
| 115 ngx_module_t ngx_http_perl_module = { | |
| 116 NGX_MODULE_V1, | |
| 117 &ngx_http_perl_module_ctx, /* module context */ | |
| 118 ngx_http_perl_commands, /* module directives */ | |
| 119 NGX_HTTP_MODULE, /* module type */ | |
| 120 NULL, /* init master */ | |
| 121 NULL, /* init module */ | |
| 1257 | 122 ngx_http_perl_init_worker, /* init process */ |
| 599 | 123 NULL, /* init thread */ |
| 124 NULL, /* exit thread */ | |
| 125 NULL, /* exit process */ | |
|
1221
aa653367028e
PERL_SYS_TERM() should be called once on exit only, this fixes the message
Igor Sysoev <igor@sysoev.ru>
parents:
1165
diff
changeset
|
126 ngx_http_perl_exit, /* exit master */ |
| 599 | 127 NGX_MODULE_V1_PADDING |
| 128 }; | |
| 129 | |
| 130 | |
| 617 | 131 #if (NGX_HTTP_SSI) |
| 132 | |
| 599 | 133 #define NGX_HTTP_PERL_SSI_SUB 0 |
| 134 #define NGX_HTTP_PERL_SSI_ARG 1 | |
| 135 | |
| 136 | |
| 137 static ngx_http_ssi_param_t ngx_http_perl_ssi_params[] = { | |
| 138 { ngx_string("sub"), NGX_HTTP_PERL_SSI_SUB, 1, 0 }, | |
| 139 { ngx_string("arg"), NGX_HTTP_PERL_SSI_ARG, 0, 1 }, | |
| 140 { ngx_null_string, 0, 0, 0 } | |
| 141 }; | |
| 142 | |
| 143 static ngx_http_ssi_command_t ngx_http_perl_ssi_command = { | |
| 667 | 144 ngx_string("perl"), ngx_http_perl_ssi, ngx_http_perl_ssi_params, 0, 0, 1 |
| 599 | 145 }; |
| 146 | |
| 617 | 147 #endif |
| 148 | |
| 599 | 149 |
|
3350
67da53a19e02
use global perl variable in perl_destruct()/perl_free()
Igor Sysoev <igor@sysoev.ru>
parents:
3175
diff
changeset
|
150 static ngx_str_t ngx_null_name = ngx_null_string; |
|
67da53a19e02
use global perl variable in perl_destruct()/perl_free()
Igor Sysoev <igor@sysoev.ru>
parents:
3175
diff
changeset
|
151 static HV *nginx_stash; |
| 681 | 152 |
| 2715 | 153 #if (NGX_HAVE_PERL_MULTIPLICITY) |
|
3350
67da53a19e02
use global perl variable in perl_destruct()/perl_free()
Igor Sysoev <igor@sysoev.ru>
parents:
3175
diff
changeset
|
154 static ngx_uint_t ngx_perl_term; |
|
67da53a19e02
use global perl variable in perl_destruct()/perl_free()
Igor Sysoev <igor@sysoev.ru>
parents:
3175
diff
changeset
|
155 #else |
|
67da53a19e02
use global perl variable in perl_destruct()/perl_free()
Igor Sysoev <igor@sysoev.ru>
parents:
3175
diff
changeset
|
156 static PerlInterpreter *perl; |
| 2715 | 157 #endif |
| 681 | 158 |
| 633 | 159 |
| 599 | 160 static void |
| 161 ngx_http_perl_xs_init(pTHX) | |
| 162 { | |
| 163 newXS("DynaLoader::boot_DynaLoader", boot_DynaLoader, __FILE__); | |
| 633 | 164 |
| 165 nginx_stash = gv_stashpv("nginx", TRUE); | |
| 599 | 166 } |
| 167 | |
| 168 | |
| 169 static ngx_int_t | |
| 170 ngx_http_perl_handler(ngx_http_request_t *r) | |
| 171 { | |
|
3082
5e8bf3e983d2
fix request counter handling for perl handler, introduced in r3050
Igor Sysoev <igor@sysoev.ru>
parents:
3050
diff
changeset
|
172 r->main->count++; |
|
5e8bf3e983d2
fix request counter handling for perl handler, introduced in r3050
Igor Sysoev <igor@sysoev.ru>
parents:
3050
diff
changeset
|
173 |
| 681 | 174 ngx_http_perl_handle_request(r); |
| 175 | |
| 176 return NGX_DONE; | |
| 629 | 177 } |
| 178 | |
| 179 | |
| 681 | 180 void |
| 629 | 181 ngx_http_perl_handle_request(ngx_http_request_t *r) |
| 182 { | |
| 681 | 183 SV *sub; |
| 599 | 184 ngx_int_t rc; |
| 681 | 185 ngx_str_t uri, args, *handler; |
| 599 | 186 ngx_http_perl_ctx_t *ctx; |
| 187 ngx_http_perl_loc_conf_t *plcf; | |
| 188 ngx_http_perl_main_conf_t *pmcf; | |
| 189 | |
| 190 ngx_log_debug0(NGX_LOG_DEBUG_HTTP, r->connection->log, 0, "perl handler"); | |
| 191 | |
| 192 ctx = ngx_http_get_module_ctx(r, ngx_http_perl_module); | |
| 193 | |
| 194 if (ctx == NULL) { | |
| 195 ctx = ngx_pcalloc(r->pool, sizeof(ngx_http_perl_ctx_t)); | |
| 196 if (ctx == NULL) { | |
| 681 | 197 ngx_http_finalize_request(r, NGX_ERROR); |
| 1025 | 198 return; |
| 599 | 199 } |
| 200 | |
| 201 ngx_http_set_ctx(r, ctx, ngx_http_perl_module); | |
| 202 } | |
| 203 | |
| 204 pmcf = ngx_http_get_module_main_conf(r, ngx_http_perl_module); | |
| 205 | |
| 206 { | |
| 207 | |
|
872
1c4a5b3f9110
Axe several perl interpreter instances: they may be useful in currently
Igor Sysoev <igor@sysoev.ru>
parents:
871
diff
changeset
|
208 dTHXa(pmcf->perl); |
|
873
f92ad15c2db1
fix segfaults in future workers those will use old configuration,
Igor Sysoev <igor@sysoev.ru>
parents:
872
diff
changeset
|
209 PERL_SET_CONTEXT(pmcf->perl); |
|
6820
eada22643e8b
Perl: added PERL_SET_INTERP().
Maxim Dounin <mdounin@mdounin.ru>
parents:
6791
diff
changeset
|
210 PERL_SET_INTERP(pmcf->perl); |
| 599 | 211 |
| 681 | 212 if (ctx->next == NULL) { |
| 213 plcf = ngx_http_get_module_loc_conf(r, ngx_http_perl_module); | |
| 214 sub = plcf->sub; | |
| 215 handler = &plcf->handler; | |
| 599 | 216 |
| 681 | 217 } else { |
| 218 sub = ctx->next; | |
| 219 handler = &ngx_null_name; | |
| 220 ctx->next = NULL; | |
| 221 } | |
| 222 | |
|
873
f92ad15c2db1
fix segfaults in future workers those will use old configuration,
Igor Sysoev <igor@sysoev.ru>
parents:
872
diff
changeset
|
223 rc = ngx_http_perl_call_handler(aTHX_ r, pmcf->nginx, sub, NULL, handler, |
|
f92ad15c2db1
fix segfaults in future workers those will use old configuration,
Igor Sysoev <igor@sysoev.ru>
parents:
872
diff
changeset
|
224 NULL); |
| 599 | 225 |
| 226 } | |
| 227 | |
|
3082
5e8bf3e983d2
fix request counter handling for perl handler, introduced in r3050
Igor Sysoev <igor@sysoev.ru>
parents:
3050
diff
changeset
|
228 ngx_log_debug1(NGX_LOG_DEBUG_HTTP, r->connection->log, 0, |
|
5e8bf3e983d2
fix request counter handling for perl handler, introduced in r3050
Igor Sysoev <igor@sysoev.ru>
parents:
3050
diff
changeset
|
229 "perl handler done: %i", rc); |
|
5e8bf3e983d2
fix request counter handling for perl handler, introduced in r3050
Igor Sysoev <igor@sysoev.ru>
parents:
3050
diff
changeset
|
230 |
|
1702
86bb52e28ce0
fix segfault when $r->has_request_body() is called with ready body
Igor Sysoev <igor@sysoev.ru>
parents:
1565
diff
changeset
|
231 if (rc == NGX_DONE) { |
|
3111
4a2d3f571de6
fix request counter handling in perl module for $r->internal_redirect()
Igor Sysoev <igor@sysoev.ru>
parents:
3082
diff
changeset
|
232 ngx_http_finalize_request(r, rc); |
|
1702
86bb52e28ce0
fix segfault when $r->has_request_body() is called with ready body
Igor Sysoev <igor@sysoev.ru>
parents:
1565
diff
changeset
|
233 return; |
|
86bb52e28ce0
fix segfault when $r->has_request_body() is called with ready body
Igor Sysoev <igor@sysoev.ru>
parents:
1565
diff
changeset
|
234 } |
|
86bb52e28ce0
fix segfault when $r->has_request_body() is called with ready body
Igor Sysoev <igor@sysoev.ru>
parents:
1565
diff
changeset
|
235 |
| 599 | 236 if (rc > 600) { |
| 237 rc = NGX_OK; | |
| 238 } | |
| 239 | |
| 240 if (ctx->redirect_uri.len) { | |
| 241 uri = ctx->redirect_uri; | |
| 242 args = ctx->redirect_args; | |
| 629 | 243 |
| 244 } else { | |
| 245 uri.len = 0; | |
| 599 | 246 } |
| 247 | |
| 633 | 248 ctx->filename.data = NULL; |
| 599 | 249 ctx->redirect_uri.len = 0; |
| 250 | |
| 681 | 251 if (ctx->done || ctx->next) { |
|
3111
4a2d3f571de6
fix request counter handling in perl module for $r->internal_redirect()
Igor Sysoev <igor@sysoev.ru>
parents:
3082
diff
changeset
|
252 ngx_http_finalize_request(r, NGX_DONE); |
| 681 | 253 return; |
| 254 } | |
| 255 | |
| 599 | 256 if (uri.len) { |
| 629 | 257 ngx_http_internal_redirect(r, &uri, &args); |
|
3111
4a2d3f571de6
fix request counter handling in perl module for $r->internal_redirect()
Igor Sysoev <igor@sysoev.ru>
parents:
3082
diff
changeset
|
258 ngx_http_finalize_request(r, NGX_DONE); |
| 629 | 259 return; |
| 599 | 260 } |
| 261 | |
| 262 if (rc == NGX_OK || rc == NGX_HTTP_OK) { | |
| 629 | 263 ngx_http_send_special(r, NGX_HTTP_LAST); |
| 681 | 264 ctx->done = 1; |
| 599 | 265 } |
| 266 | |
| 629 | 267 ngx_http_finalize_request(r, rc); |
| 599 | 268 } |
| 269 | |
| 270 | |
| 1897 | 271 void |
| 911 | 272 ngx_http_perl_sleep_handler(ngx_http_request_t *r) |
| 273 { | |
|
913
90ce4d0e3241
fix ngx_http_perl_sleep_handler()
Igor Sysoev <igor@sysoev.ru>
parents:
911
diff
changeset
|
274 ngx_event_t *wev; |
|
90ce4d0e3241
fix ngx_http_perl_sleep_handler()
Igor Sysoev <igor@sysoev.ru>
parents:
911
diff
changeset
|
275 |
| 911 | 276 ngx_log_debug0(NGX_LOG_DEBUG_HTTP, r->connection->log, 0, |
| 277 "perl sleep handler"); | |
| 278 | |
|
913
90ce4d0e3241
fix ngx_http_perl_sleep_handler()
Igor Sysoev <igor@sysoev.ru>
parents:
911
diff
changeset
|
279 wev = r->connection->write; |
|
90ce4d0e3241
fix ngx_http_perl_sleep_handler()
Igor Sysoev <igor@sysoev.ru>
parents:
911
diff
changeset
|
280 |
|
90ce4d0e3241
fix ngx_http_perl_sleep_handler()
Igor Sysoev <igor@sysoev.ru>
parents:
911
diff
changeset
|
281 if (wev->timedout) { |
|
90ce4d0e3241
fix ngx_http_perl_sleep_handler()
Igor Sysoev <igor@sysoev.ru>
parents:
911
diff
changeset
|
282 wev->timedout = 0; |
|
90ce4d0e3241
fix ngx_http_perl_sleep_handler()
Igor Sysoev <igor@sysoev.ru>
parents:
911
diff
changeset
|
283 ngx_http_perl_handle_request(r); |
|
90ce4d0e3241
fix ngx_http_perl_sleep_handler()
Igor Sysoev <igor@sysoev.ru>
parents:
911
diff
changeset
|
284 return; |
|
90ce4d0e3241
fix ngx_http_perl_sleep_handler()
Igor Sysoev <igor@sysoev.ru>
parents:
911
diff
changeset
|
285 } |
|
90ce4d0e3241
fix ngx_http_perl_sleep_handler()
Igor Sysoev <igor@sysoev.ru>
parents:
911
diff
changeset
|
286 |
|
2388
722b5aff05ae
use "!= NGX_OK" instead of "== NGX_ERROR"
Igor Sysoev <igor@sysoev.ru>
parents:
2049
diff
changeset
|
287 if (ngx_handle_write_event(wev, 0) != NGX_OK) { |
|
913
90ce4d0e3241
fix ngx_http_perl_sleep_handler()
Igor Sysoev <igor@sysoev.ru>
parents:
911
diff
changeset
|
288 ngx_http_finalize_request(r, NGX_HTTP_INTERNAL_SERVER_ERROR); |
|
90ce4d0e3241
fix ngx_http_perl_sleep_handler()
Igor Sysoev <igor@sysoev.ru>
parents:
911
diff
changeset
|
289 } |
| 911 | 290 } |
| 291 | |
| 292 | |
| 599 | 293 static ngx_int_t |
| 294 ngx_http_perl_variable(ngx_http_request_t *r, ngx_http_variable_value_t *v, | |
| 295 uintptr_t data) | |
| 296 { | |
| 297 ngx_http_perl_variable_t *pv = (ngx_http_perl_variable_t *) data; | |
| 298 | |
| 299 ngx_int_t rc; | |
| 300 ngx_str_t value; | |
| 301 ngx_http_perl_ctx_t *ctx; | |
| 302 ngx_http_perl_main_conf_t *pmcf; | |
| 303 | |
| 304 ngx_log_debug0(NGX_LOG_DEBUG_HTTP, r->connection->log, 0, | |
| 305 "perl variable handler"); | |
| 306 | |
| 307 ctx = ngx_http_get_module_ctx(r, ngx_http_perl_module); | |
| 308 | |
| 309 if (ctx == NULL) { | |
| 310 ctx = ngx_pcalloc(r->pool, sizeof(ngx_http_perl_ctx_t)); | |
| 311 if (ctx == NULL) { | |
| 312 return NGX_ERROR; | |
| 313 } | |
| 314 | |
| 315 ngx_http_set_ctx(r, ctx, ngx_http_perl_module); | |
|
872
1c4a5b3f9110
Axe several perl interpreter instances: they may be useful in currently
Igor Sysoev <igor@sysoev.ru>
parents:
871
diff
changeset
|
316 } |
| 599 | 317 |
|
872
1c4a5b3f9110
Axe several perl interpreter instances: they may be useful in currently
Igor Sysoev <igor@sysoev.ru>
parents:
871
diff
changeset
|
318 pmcf = ngx_http_get_module_main_conf(r, ngx_http_perl_module); |
| 599 | 319 |
| 320 value.data = NULL; | |
| 321 | |
| 322 { | |
| 323 | |
|
872
1c4a5b3f9110
Axe several perl interpreter instances: they may be useful in currently
Igor Sysoev <igor@sysoev.ru>
parents:
871
diff
changeset
|
324 dTHXa(pmcf->perl); |
|
873
f92ad15c2db1
fix segfaults in future workers those will use old configuration,
Igor Sysoev <igor@sysoev.ru>
parents:
872
diff
changeset
|
325 PERL_SET_CONTEXT(pmcf->perl); |
|
6820
eada22643e8b
Perl: added PERL_SET_INTERP().
Maxim Dounin <mdounin@mdounin.ru>
parents:
6791
diff
changeset
|
326 PERL_SET_INTERP(pmcf->perl); |
| 599 | 327 |
|
873
f92ad15c2db1
fix segfaults in future workers those will use old configuration,
Igor Sysoev <igor@sysoev.ru>
parents:
872
diff
changeset
|
328 rc = ngx_http_perl_call_handler(aTHX_ r, pmcf->nginx, pv->sub, NULL, |
| 599 | 329 &pv->handler, &value); |
| 330 | |
| 331 } | |
| 332 | |
| 333 if (value.data) { | |
| 334 v->len = value.len; | |
| 335 v->valid = 1; | |
| 1565 | 336 v->no_cacheable = 0; |
| 599 | 337 v->not_found = 0; |
| 338 v->data = value.data; | |
| 339 | |
| 340 } else { | |
| 341 v->not_found = 1; | |
| 342 } | |
| 343 | |
| 633 | 344 ctx->filename.data = NULL; |
| 599 | 345 ctx->redirect_uri.len = 0; |
| 346 | |
| 871 | 347 ngx_log_debug0(NGX_LOG_DEBUG_HTTP, r->connection->log, 0, |
| 348 "perl variable done"); | |
| 349 | |
| 599 | 350 return rc; |
| 351 } | |
| 352 | |
| 353 | |
| 617 | 354 #if (NGX_HTTP_SSI) |
| 355 | |
| 599 | 356 static ngx_int_t |
| 357 ngx_http_perl_ssi(ngx_http_request_t *r, ngx_http_ssi_ctx_t *ssi_ctx, | |
| 358 ngx_str_t **params) | |
| 359 { | |
|
1899
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
360 SV *sv, **asv; |
| 599 | 361 ngx_int_t rc; |
|
1899
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
362 ngx_str_t *handler, **args; |
|
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
363 ngx_uint_t i; |
| 599 | 364 ngx_http_perl_ctx_t *ctx; |
| 365 ngx_http_perl_main_conf_t *pmcf; | |
| 366 | |
| 367 ngx_log_debug0(NGX_LOG_DEBUG_HTTP, r->connection->log, 0, | |
| 368 "perl ssi handler"); | |
| 369 | |
| 370 ctx = ngx_http_get_module_ctx(r, ngx_http_perl_module); | |
| 371 | |
| 372 if (ctx == NULL) { | |
| 373 ctx = ngx_pcalloc(r->pool, sizeof(ngx_http_perl_ctx_t)); | |
| 374 if (ctx == NULL) { | |
| 375 return NGX_ERROR; | |
| 376 } | |
| 377 | |
| 378 ngx_http_set_ctx(r, ctx, ngx_http_perl_module); | |
| 379 } | |
| 380 | |
|
872
1c4a5b3f9110
Axe several perl interpreter instances: they may be useful in currently
Igor Sysoev <igor@sysoev.ru>
parents:
871
diff
changeset
|
381 pmcf = ngx_http_get_module_main_conf(r, ngx_http_perl_module); |
| 599 | 382 |
| 383 ctx->ssi = ssi_ctx; | |
| 384 | |
| 385 handler = params[NGX_HTTP_PERL_SSI_SUB]; | |
| 386 handler->data[handler->len] = '\0'; | |
| 387 | |
| 388 { | |
| 389 | |
|
872
1c4a5b3f9110
Axe several perl interpreter instances: they may be useful in currently
Igor Sysoev <igor@sysoev.ru>
parents:
871
diff
changeset
|
390 dTHXa(pmcf->perl); |
|
873
f92ad15c2db1
fix segfaults in future workers those will use old configuration,
Igor Sysoev <igor@sysoev.ru>
parents:
872
diff
changeset
|
391 PERL_SET_CONTEXT(pmcf->perl); |
|
6820
eada22643e8b
Perl: added PERL_SET_INTERP().
Maxim Dounin <mdounin@mdounin.ru>
parents:
6791
diff
changeset
|
392 PERL_SET_INTERP(pmcf->perl); |
| 599 | 393 |
| 651 | 394 #if 0 |
| 395 | |
| 396 /* the code is disabled to force the precompiled perl code using only */ | |
| 397 | |
| 398 ngx_http_perl_eval_anon_sub(aTHX_ handler, &sv); | |
| 399 | |
| 400 if (sv == &PL_sv_undef) { | |
| 401 ngx_log_error(NGX_LOG_ERR, r->connection->log, 0, | |
| 402 "eval_pv(\"%V\") failed", handler); | |
| 403 return NGX_ERROR; | |
| 404 } | |
| 405 | |
| 406 if (sv == NULL) { | |
| 407 sv = newSVpvn((char *) handler->data, handler->len); | |
| 408 } | |
| 409 | |
| 410 #endif | |
| 411 | |
| 599 | 412 sv = newSVpvn((char *) handler->data, handler->len); |
| 413 | |
|
1899
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
414 args = ¶ms[NGX_HTTP_PERL_SSI_ARG]; |
|
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
415 |
|
6791
cb4a4e9bba8e
Perl: fixed optimization in SSI command handler.
Maxim Dounin <mdounin@mdounin.ru>
parents:
5800
diff
changeset
|
416 if (args[0]) { |
|
1899
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
417 |
|
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
418 for (i = 0; args[i]; i++) { /* void */ } |
|
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
419 |
|
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
420 asv = ngx_pcalloc(r->pool, (i + 1) * sizeof(SV *)); |
|
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
421 |
|
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
422 if (asv == NULL) { |
|
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
423 SvREFCNT_dec(sv); |
|
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
424 return NGX_ERROR; |
|
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
425 } |
|
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
426 |
|
5362
79b9101cecf4
Handling of ngx_int_t != intptr_t case.
Maxim Dounin <mdounin@mdounin.ru>
parents:
5330
diff
changeset
|
427 asv[0] = (SV *) (uintptr_t) i; |
|
1899
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
428 |
|
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
429 for (i = 0; args[i]; i++) { |
|
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
430 asv[i + 1] = newSVpvn((char *) args[i]->data, args[i]->len); |
|
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
431 } |
|
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
432 |
|
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
433 } else { |
|
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
434 asv = NULL; |
|
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
435 } |
|
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
436 |
|
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
437 rc = ngx_http_perl_call_handler(aTHX_ r, pmcf->nginx, sv, asv, handler, |
|
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
438 NULL); |
| 599 | 439 |
| 440 SvREFCNT_dec(sv); | |
| 441 | |
| 442 } | |
| 443 | |
| 633 | 444 ctx->filename.data = NULL; |
| 599 | 445 ctx->redirect_uri.len = 0; |
| 446 ctx->ssi = NULL; | |
| 447 | |
| 871 | 448 ngx_log_debug0(NGX_LOG_DEBUG_HTTP, r->connection->log, 0, "perl ssi done"); |
| 449 | |
| 599 | 450 return rc; |
| 451 } | |
| 452 | |
| 617 | 453 #endif |
| 454 | |
| 599 | 455 |
| 456 static char * | |
| 457 ngx_http_perl_init_interpreter(ngx_conf_t *cf, ngx_http_perl_main_conf_t *pmcf) | |
| 458 { | |
|
3350
67da53a19e02
use global perl variable in perl_destruct()/perl_free()
Igor Sysoev <igor@sysoev.ru>
parents:
3175
diff
changeset
|
459 ngx_str_t *m; |
|
67da53a19e02
use global perl variable in perl_destruct()/perl_free()
Igor Sysoev <igor@sysoev.ru>
parents:
3175
diff
changeset
|
460 ngx_uint_t i; |
|
872
1c4a5b3f9110
Axe several perl interpreter instances: they may be useful in currently
Igor Sysoev <igor@sysoev.ru>
parents:
871
diff
changeset
|
461 #if (NGX_HAVE_PERL_MULTIPLICITY) |
|
3350
67da53a19e02
use global perl variable in perl_destruct()/perl_free()
Igor Sysoev <igor@sysoev.ru>
parents:
3175
diff
changeset
|
462 ngx_pool_cleanup_t *cln; |
| 599 | 463 |
| 464 cln = ngx_pool_cleanup_add(cf->pool, 0); | |
| 465 if (cln == NULL) { | |
| 466 return NGX_CONF_ERROR; | |
| 467 } | |
| 468 | |
| 649 | 469 #endif |
| 470 | |
| 599 | 471 #ifdef NGX_PERL_MODULES |
| 3175 | 472 if (pmcf->modules == NGX_CONF_UNSET_PTR) { |
| 473 | |
| 474 pmcf->modules = ngx_array_create(cf->pool, 1, sizeof(ngx_str_t)); | |
| 475 if (pmcf->modules == NULL) { | |
| 476 return NGX_CONF_ERROR; | |
| 477 } | |
| 478 | |
| 479 m = ngx_array_push(pmcf->modules); | |
| 480 if (m == NULL) { | |
| 481 return NGX_CONF_ERROR; | |
| 482 } | |
| 483 | |
|
3516
dd1570b6f237
ngx_str_set() and ngx_str_null()
Igor Sysoev <igor@sysoev.ru>
parents:
3448
diff
changeset
|
484 ngx_str_set(m, NGX_PERL_MODULES); |
| 599 | 485 } |
| 486 #endif | |
| 487 | |
| 3175 | 488 if (pmcf->modules != NGX_CONF_UNSET_PTR) { |
| 489 m = pmcf->modules->elts; | |
| 490 for (i = 0; i < pmcf->modules->nelts; i++) { | |
|
5330
314c3d7cc3a5
Backed out f1a91825730a and 7094bd12c1ff.
Maxim Dounin <mdounin@mdounin.ru>
parents:
5317
diff
changeset
|
491 if (ngx_conf_full_name(cf->cycle, &m[i], 0) != NGX_OK) { |
| 3175 | 492 return NGX_CONF_ERROR; |
| 493 } | |
| 639 | 494 } |
| 629 | 495 } |
| 496 | |
|
872
1c4a5b3f9110
Axe several perl interpreter instances: they may be useful in currently
Igor Sysoev <igor@sysoev.ru>
parents:
871
diff
changeset
|
497 #if !(NGX_HAVE_PERL_MULTIPLICITY) |
| 649 | 498 |
| 499 if (perl) { | |
| 1165 | 500 |
| 501 if (ngx_set_environment(cf->cycle, NULL) == NULL) { | |
| 502 return NGX_CONF_ERROR; | |
| 503 } | |
| 504 | |
|
3174
479fd46cd1c4
use ngx_conf_set_str_array_slot() for perl_require
Igor Sysoev <igor@sysoev.ru>
parents:
3173
diff
changeset
|
505 if (ngx_http_perl_run_requires(aTHX_ pmcf->requires, cf->log) |
| 653 | 506 != NGX_OK) |
| 507 { | |
| 649 | 508 return NGX_CONF_ERROR; |
| 509 } | |
| 510 | |
| 511 pmcf->perl = perl; | |
|
1163
8288459f15c9
set nginx_stash after reconfiguration in single interpreter perl
Igor Sysoev <igor@sysoev.ru>
parents:
1069
diff
changeset
|
512 pmcf->nginx = nginx_stash; |
| 649 | 513 |
| 514 return NGX_CONF_OK; | |
| 515 } | |
| 516 | |
| 517 #endif | |
| 518 | |
|
1221
aa653367028e
PERL_SYS_TERM() should be called once on exit only, this fixes the message
Igor Sysoev <igor@sysoev.ru>
parents:
1165
diff
changeset
|
519 if (nginx_stash == NULL) { |
|
aa653367028e
PERL_SYS_TERM() should be called once on exit only, this fixes the message
Igor Sysoev <igor@sysoev.ru>
parents:
1165
diff
changeset
|
520 PERL_SYS_INIT(&ngx_argc, &ngx_argv); |
|
aa653367028e
PERL_SYS_TERM() should be called once on exit only, this fixes the message
Igor Sysoev <igor@sysoev.ru>
parents:
1165
diff
changeset
|
521 } |
| 599 | 522 |
| 1069 | 523 pmcf->perl = ngx_http_perl_create_interpreter(cf, pmcf); |
| 599 | 524 |
| 525 if (pmcf->perl == NULL) { | |
| 526 return NGX_CONF_ERROR; | |
| 527 } | |
| 528 | |
|
873
f92ad15c2db1
fix segfaults in future workers those will use old configuration,
Igor Sysoev <igor@sysoev.ru>
parents:
872
diff
changeset
|
529 pmcf->nginx = nginx_stash; |
|
f92ad15c2db1
fix segfaults in future workers those will use old configuration,
Igor Sysoev <igor@sysoev.ru>
parents:
872
diff
changeset
|
530 |
|
872
1c4a5b3f9110
Axe several perl interpreter instances: they may be useful in currently
Igor Sysoev <igor@sysoev.ru>
parents:
871
diff
changeset
|
531 #if (NGX_HAVE_PERL_MULTIPLICITY) |
| 649 | 532 |
| 599 | 533 cln->handler = ngx_http_perl_cleanup_perl; |
| 534 cln->data = pmcf->perl; | |
| 535 | |
| 649 | 536 #else |
| 537 | |
| 538 perl = pmcf->perl; | |
| 539 | |
| 540 #endif | |
| 541 | |
| 599 | 542 return NGX_CONF_OK; |
| 543 } | |
| 544 | |
| 545 | |
| 546 static PerlInterpreter * | |
| 1069 | 547 ngx_http_perl_create_interpreter(ngx_conf_t *cf, |
| 548 ngx_http_perl_main_conf_t *pmcf) | |
| 599 | 549 { |
| 550 int n; | |
| 775 | 551 STRLEN len; |
| 552 SV *sv; | |
| 3175 | 553 char *ver, **embedding; |
| 554 ngx_str_t *m; | |
| 555 ngx_uint_t i; | |
| 599 | 556 PerlInterpreter *perl; |
| 557 | |
| 1069 | 558 ngx_log_debug0(NGX_LOG_DEBUG_HTTP, cf->log, 0, "create perl interpreter"); |
| 559 | |
| 560 if (ngx_set_environment(cf->cycle, NULL) == NULL) { | |
| 561 return NULL; | |
| 562 } | |
| 599 | 563 |
| 564 perl = perl_alloc(); | |
| 565 if (perl == NULL) { | |
| 1069 | 566 ngx_log_error(NGX_LOG_ALERT, cf->log, 0, "perl_alloc() failed"); |
| 599 | 567 return NULL; |
| 568 } | |
| 569 | |
| 570 { | |
| 571 | |
| 572 dTHXa(perl); | |
|
873
f92ad15c2db1
fix segfaults in future workers those will use old configuration,
Igor Sysoev <igor@sysoev.ru>
parents:
872
diff
changeset
|
573 PERL_SET_CONTEXT(perl); |
|
6820
eada22643e8b
Perl: added PERL_SET_INTERP().
Maxim Dounin <mdounin@mdounin.ru>
parents:
6791
diff
changeset
|
574 PERL_SET_INTERP(perl); |
|
873
f92ad15c2db1
fix segfaults in future workers those will use old configuration,
Igor Sysoev <igor@sysoev.ru>
parents:
872
diff
changeset
|
575 |
|
f92ad15c2db1
fix segfaults in future workers those will use old configuration,
Igor Sysoev <igor@sysoev.ru>
parents:
872
diff
changeset
|
576 perl_construct(perl); |
| 599 | 577 |
| 578 #ifdef PERL_EXIT_DESTRUCT_END | |
| 579 PL_exit_flags |= PERL_EXIT_DESTRUCT_END; | |
| 580 #endif | |
| 581 | |
| 3175 | 582 n = (pmcf->modules != NGX_CONF_UNSET_PTR) ? pmcf->modules->nelts * 2 : 0; |
| 583 | |
|
5800
0570e42ffeed
Perl: NULL-terminate argument list.
Piotr Sikora <piotr@cloudflare.com>
parents:
5362
diff
changeset
|
584 embedding = ngx_palloc(cf->pool, (5 + n) * sizeof(char *)); |
| 3175 | 585 if (embedding == NULL) { |
| 586 goto fail; | |
| 587 } | |
| 588 | |
| 599 | 589 embedding[0] = ""; |
| 590 | |
| 3175 | 591 if (n++) { |
| 592 m = pmcf->modules->elts; | |
| 593 for (i = 0; i < pmcf->modules->nelts; i++) { | |
| 594 embedding[2 * i + 1] = "-I"; | |
| 595 embedding[2 * i + 2] = (char *) m[i].data; | |
| 596 } | |
| 599 | 597 } |
| 598 | |
| 599 embedding[n++] = "-Mnginx"; | |
| 600 embedding[n++] = "-e"; | |
| 601 embedding[n++] = "0"; | |
|
5800
0570e42ffeed
Perl: NULL-terminate argument list.
Piotr Sikora <piotr@cloudflare.com>
parents:
5362
diff
changeset
|
602 embedding[n] = NULL; |
| 599 | 603 |
| 604 n = perl_parse(perl, ngx_http_perl_xs_init, n, embedding, NULL); | |
| 605 | |
| 606 if (n != 0) { | |
| 1069 | 607 ngx_log_error(NGX_LOG_ALERT, cf->log, 0, "perl_parse() failed: %d", n); |
| 599 | 608 goto fail; |
| 609 } | |
| 610 | |
| 775 | 611 sv = get_sv("nginx::VERSION", FALSE); |
| 612 ver = SvPV(sv, len); | |
| 613 | |
| 614 if (ngx_strcmp(ver, NGINX_VERSION) != 0) { | |
| 1069 | 615 ngx_log_error(NGX_LOG_ALERT, cf->log, 0, |
| 775 | 616 "version " NGINX_VERSION " of nginx.pm is required, " |
| 617 "but %s was found", ver); | |
| 618 goto fail; | |
| 619 } | |
| 620 | |
|
3174
479fd46cd1c4
use ngx_conf_set_str_array_slot() for perl_require
Igor Sysoev <igor@sysoev.ru>
parents:
3173
diff
changeset
|
621 if (ngx_http_perl_run_requires(aTHX_ pmcf->requires, cf->log) != NGX_OK) { |
| 649 | 622 goto fail; |
| 599 | 623 } |
| 624 | |
| 625 } | |
| 626 | |
| 627 return perl; | |
| 628 | |
| 629 fail: | |
| 630 | |
| 631 (void) perl_destruct(perl); | |
| 632 | |
| 633 perl_free(perl); | |
| 634 | |
| 635 return NULL; | |
| 636 } | |
| 637 | |
| 638 | |
| 649 | 639 static ngx_int_t |
| 653 | 640 ngx_http_perl_run_requires(pTHX_ ngx_array_t *requires, ngx_log_t *log) |
| 649 | 641 { |
| 3173 | 642 u_char *err; |
| 649 | 643 STRLEN len; |
|
3174
479fd46cd1c4
use ngx_conf_set_str_array_slot() for perl_require
Igor Sysoev <igor@sysoev.ru>
parents:
3173
diff
changeset
|
644 ngx_str_t *script; |
| 649 | 645 ngx_uint_t i; |
| 646 | |
|
3174
479fd46cd1c4
use ngx_conf_set_str_array_slot() for perl_require
Igor Sysoev <igor@sysoev.ru>
parents:
3173
diff
changeset
|
647 if (requires == NGX_CONF_UNSET_PTR) { |
|
479fd46cd1c4
use ngx_conf_set_str_array_slot() for perl_require
Igor Sysoev <igor@sysoev.ru>
parents:
3173
diff
changeset
|
648 return NGX_OK; |
|
479fd46cd1c4
use ngx_conf_set_str_array_slot() for perl_require
Igor Sysoev <igor@sysoev.ru>
parents:
3173
diff
changeset
|
649 } |
|
479fd46cd1c4
use ngx_conf_set_str_array_slot() for perl_require
Igor Sysoev <igor@sysoev.ru>
parents:
3173
diff
changeset
|
650 |
| 649 | 651 script = requires->elts; |
| 652 for (i = 0; i < requires->nelts; i++) { | |
| 653 | |
|
3174
479fd46cd1c4
use ngx_conf_set_str_array_slot() for perl_require
Igor Sysoev <igor@sysoev.ru>
parents:
3173
diff
changeset
|
654 require_pv((char *) script[i].data); |
| 649 | 655 |
| 656 if (SvTRUE(ERRSV)) { | |
| 657 | |
| 3173 | 658 err = (u_char *) SvPV(ERRSV, len); |
| 659 while (--len && (err[len] == CR || err[len] == LF)) { /* void */ } | |
| 649 | 660 |
| 661 ngx_log_error(NGX_LOG_EMERG, log, 0, | |
| 3173 | 662 "require_pv(\"%s\") failed: \"%*s\"", |
|
3174
479fd46cd1c4
use ngx_conf_set_str_array_slot() for perl_require
Igor Sysoev <igor@sysoev.ru>
parents:
3173
diff
changeset
|
663 script[i].data, len + 1, err); |
| 649 | 664 |
| 665 return NGX_ERROR; | |
| 666 } | |
| 667 } | |
| 668 | |
| 669 return NGX_OK; | |
| 670 } | |
| 671 | |
| 672 | |
| 599 | 673 static ngx_int_t |
|
873
f92ad15c2db1
fix segfaults in future workers those will use old configuration,
Igor Sysoev <igor@sysoev.ru>
parents:
872
diff
changeset
|
674 ngx_http_perl_call_handler(pTHX_ ngx_http_request_t *r, HV *nginx, SV *sub, |
|
1899
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
675 SV **args, ngx_str_t *handler, ngx_str_t *rv) |
| 599 | 676 { |
|
1702
86bb52e28ce0
fix segfault when $r->has_request_body() is called with ready body
Igor Sysoev <igor@sysoev.ru>
parents:
1565
diff
changeset
|
677 SV *sv; |
|
86bb52e28ce0
fix segfault when $r->has_request_body() is called with ready body
Igor Sysoev <igor@sysoev.ru>
parents:
1565
diff
changeset
|
678 int n, status; |
|
86bb52e28ce0
fix segfault when $r->has_request_body() is called with ready body
Igor Sysoev <igor@sysoev.ru>
parents:
1565
diff
changeset
|
679 char *line; |
| 3173 | 680 u_char *err; |
|
1702
86bb52e28ce0
fix segfault when $r->has_request_body() is called with ready body
Igor Sysoev <igor@sysoev.ru>
parents:
1565
diff
changeset
|
681 STRLEN len, n_a; |
|
86bb52e28ce0
fix segfault when $r->has_request_body() is called with ready body
Igor Sysoev <igor@sysoev.ru>
parents:
1565
diff
changeset
|
682 ngx_uint_t i; |
|
86bb52e28ce0
fix segfault when $r->has_request_body() is called with ready body
Igor Sysoev <igor@sysoev.ru>
parents:
1565
diff
changeset
|
683 ngx_connection_t *c; |
| 599 | 684 |
| 685 dSP; | |
| 686 | |
| 687 status = 0; | |
| 688 | |
| 689 ENTER; | |
| 690 SAVETMPS; | |
| 691 | |
| 692 PUSHMARK(sp); | |
| 693 | |
|
873
f92ad15c2db1
fix segfaults in future workers those will use old configuration,
Igor Sysoev <igor@sysoev.ru>
parents:
872
diff
changeset
|
694 sv = sv_2mortal(sv_bless(newRV_noinc(newSViv(PTR2IV(r))), nginx)); |
| 599 | 695 XPUSHs(sv); |
| 696 | |
| 697 if (args) { | |
|
1940
6a4c74bea81c
fix building on 64-bit platforms broken in r1900
Igor Sysoev <igor@sysoev.ru>
parents:
1899
diff
changeset
|
698 EXTEND(sp, (intptr_t) args[0]); |
| 599 | 699 |
|
5362
79b9101cecf4
Handling of ngx_int_t != intptr_t case.
Maxim Dounin <mdounin@mdounin.ru>
parents:
5330
diff
changeset
|
700 for (i = 1; i <= (uintptr_t) args[0]; i++) { |
|
1899
d24ef26f1205
pass additional arguments in ngx_http_perl_call_handler() as SV
Igor Sysoev <igor@sysoev.ru>
parents:
1897
diff
changeset
|
701 PUSHs(sv_2mortal(args[i])); |
| 599 | 702 } |
| 703 } | |
| 704 | |
| 705 PUTBACK; | |
| 706 | |
|
1702
86bb52e28ce0
fix segfault when $r->has_request_body() is called with ready body
Igor Sysoev <igor@sysoev.ru>
parents:
1565
diff
changeset
|
707 c = r->connection; |
|
86bb52e28ce0
fix segfault when $r->has_request_body() is called with ready body
Igor Sysoev <igor@sysoev.ru>
parents:
1565
diff
changeset
|
708 |
| 599 | 709 n = call_sv(sub, G_EVAL); |
| 710 | |
| 711 SPAGAIN; | |
| 712 | |
| 713 if (n) { | |
| 714 if (rv == NULL) { | |
| 715 status = POPi; | |
| 716 | |
|
1702
86bb52e28ce0
fix segfault when $r->has_request_body() is called with ready body
Igor Sysoev <igor@sysoev.ru>
parents:
1565
diff
changeset
|
717 ngx_log_debug1(NGX_LOG_DEBUG_HTTP, c->log, 0, |
| 599 | 718 "call_sv: %d", status); |
| 719 | |
| 720 } else { | |
| 619 | 721 line = SvPVx(POPs, n_a); |
| 599 | 722 rv->len = n_a; |
| 723 | |
| 2049 | 724 rv->data = ngx_pnalloc(r->pool, n_a); |
| 599 | 725 if (rv->data == NULL) { |
| 726 return NGX_ERROR; | |
| 727 } | |
| 728 | |
| 729 ngx_memcpy(rv->data, line, n_a); | |
| 730 } | |
| 731 } | |
| 732 | |
| 733 PUTBACK; | |
| 734 | |
| 735 FREETMPS; | |
| 736 LEAVE; | |
| 737 | |
| 738 /* check $@ */ | |
| 739 | |
| 740 if (SvTRUE(ERRSV)) { | |
| 741 | |
| 3173 | 742 err = (u_char *) SvPV(ERRSV, len); |
| 743 while (--len && (err[len] == CR || err[len] == LF)) { /* void */ } | |
| 599 | 744 |
|
1702
86bb52e28ce0
fix segfault when $r->has_request_body() is called with ready body
Igor Sysoev <igor@sysoev.ru>
parents:
1565
diff
changeset
|
745 ngx_log_error(NGX_LOG_ERR, c->log, 0, |
| 3173 | 746 "call_sv(\"%V\") failed: \"%*s\"", handler, len + 1, err); |
| 599 | 747 |
| 748 if (rv) { | |
| 749 return NGX_ERROR; | |
| 750 } | |
| 751 | |
| 752 return NGX_HTTP_INTERNAL_SERVER_ERROR; | |
| 753 } | |
| 754 | |
| 755 if (n != 1) { | |
|
1702
86bb52e28ce0
fix segfault when $r->has_request_body() is called with ready body
Igor Sysoev <igor@sysoev.ru>
parents:
1565
diff
changeset
|
756 ngx_log_error(NGX_LOG_ALERT, c->log, 0, |
| 599 | 757 "call_sv(\"%V\") returned %d results", handler, n); |
| 758 status = NGX_OK; | |
| 759 } | |
| 760 | |
| 761 if (rv) { | |
| 762 return NGX_OK; | |
| 763 } | |
| 764 | |
| 765 return (ngx_int_t) status; | |
| 766 } | |
| 767 | |
| 768 | |
| 769 static void | |
| 770 ngx_http_perl_eval_anon_sub(pTHX_ ngx_str_t *handler, SV **sv) | |
| 771 { | |
| 623 | 772 u_char *p; |
| 773 | |
| 774 for (p = handler->data; *p; p++) { | |
| 775 if (*p != ' ' && *p != '\t' && *p != CR && *p != LF) { | |
| 776 break; | |
| 777 } | |
| 778 } | |
| 779 | |
| 3132 | 780 if (ngx_strncmp(p, "sub ", 4) == 0 |
| 781 || ngx_strncmp(p, "sub{", 4) == 0 | |
| 782 || ngx_strncmp(p, "use ", 4) == 0) | |
| 783 { | |
| 623 | 784 *sv = eval_pv((char *) p, FALSE); |
| 874 | 785 |
| 786 /* eval_pv() does not set ERRSV on failure */ | |
| 787 | |
| 599 | 788 return; |
| 789 } | |
| 790 | |
| 791 *sv = NULL; | |
| 792 } | |
| 793 | |
| 794 | |
| 795 static void * | |
| 796 ngx_http_perl_create_main_conf(ngx_conf_t *cf) | |
| 797 { | |
| 798 ngx_http_perl_main_conf_t *pmcf; | |
| 799 | |
| 800 pmcf = ngx_pcalloc(cf->pool, sizeof(ngx_http_perl_main_conf_t)); | |
| 801 if (pmcf == NULL) { | |
|
2912
c7d57b539248
return NULL instead of NGX_CONF_ERROR on a create conf failure
Igor Sysoev <igor@sysoev.ru>
parents:
2721
diff
changeset
|
802 return NULL; |
| 599 | 803 } |
| 804 | |
| 3175 | 805 pmcf->modules = NGX_CONF_UNSET_PTR; |
|
3174
479fd46cd1c4
use ngx_conf_set_str_array_slot() for perl_require
Igor Sysoev <igor@sysoev.ru>
parents:
3173
diff
changeset
|
806 pmcf->requires = NGX_CONF_UNSET_PTR; |
| 599 | 807 |
| 808 return pmcf; | |
| 809 } | |
| 810 | |
| 811 | |
| 812 static char * | |
| 813 ngx_http_perl_init_main_conf(ngx_conf_t *cf, void *conf) | |
| 814 { | |
| 815 ngx_http_perl_main_conf_t *pmcf = conf; | |
| 816 | |
| 817 if (pmcf->perl == NULL) { | |
| 818 if (ngx_http_perl_init_interpreter(cf, pmcf) != NGX_CONF_OK) { | |
| 819 return NGX_CONF_ERROR; | |
| 820 } | |
| 821 } | |
| 822 | |
| 823 return NGX_CONF_OK; | |
| 824 } | |
| 825 | |
| 826 | |
|
872
1c4a5b3f9110
Axe several perl interpreter instances: they may be useful in currently
Igor Sysoev <igor@sysoev.ru>
parents:
871
diff
changeset
|
827 #if (NGX_HAVE_PERL_MULTIPLICITY) |
| 649 | 828 |
| 599 | 829 static void |
| 830 ngx_http_perl_cleanup_perl(void *data) | |
| 831 { | |
| 649 | 832 PerlInterpreter *perl = data; |
| 599 | 833 |
|
873
f92ad15c2db1
fix segfaults in future workers those will use old configuration,
Igor Sysoev <igor@sysoev.ru>
parents:
872
diff
changeset
|
834 PERL_SET_CONTEXT(perl); |
|
6820
eada22643e8b
Perl: added PERL_SET_INTERP().
Maxim Dounin <mdounin@mdounin.ru>
parents:
6791
diff
changeset
|
835 PERL_SET_INTERP(perl); |
|
873
f92ad15c2db1
fix segfaults in future workers those will use old configuration,
Igor Sysoev <igor@sysoev.ru>
parents:
872
diff
changeset
|
836 |
| 599 | 837 (void) perl_destruct(perl); |
| 838 | |
| 839 perl_free(perl); | |
| 2715 | 840 |
| 841 if (ngx_perl_term) { | |
| 842 ngx_log_debug0(NGX_LOG_DEBUG_HTTP, ngx_cycle->log, 0, "perl term"); | |
| 843 | |
| 844 PERL_SYS_TERM(); | |
| 845 } | |
| 599 | 846 } |
| 847 | |
| 649 | 848 #endif |
| 849 | |
| 850 | |
| 599 | 851 static ngx_int_t |
| 852 ngx_http_perl_preconfiguration(ngx_conf_t *cf) | |
| 853 { | |
| 617 | 854 #if (NGX_HTTP_SSI) |
| 599 | 855 ngx_int_t rc; |
| 856 ngx_http_ssi_main_conf_t *smcf; | |
| 857 | |
| 858 smcf = ngx_http_conf_get_module_main_conf(cf, ngx_http_ssi_filter_module); | |
| 859 | |
| 860 rc = ngx_hash_add_key(&smcf->commands, &ngx_http_perl_ssi_command.name, | |
| 861 &ngx_http_perl_ssi_command, NGX_HASH_READONLY_KEY); | |
| 862 | |
| 863 if (rc != NGX_OK) { | |
| 864 if (rc == NGX_BUSY) { | |
| 865 ngx_conf_log_error(NGX_LOG_EMERG, cf, 0, | |
| 866 "conflicting SSI command \"%V\"", | |
| 867 &ngx_http_perl_ssi_command.name); | |
| 868 } | |
| 869 | |
| 870 return NGX_ERROR; | |
| 871 } | |
| 617 | 872 #endif |
| 599 | 873 |
| 874 return NGX_OK; | |
| 875 } | |
| 876 | |
| 877 | |
| 878 static void * | |
| 879 ngx_http_perl_create_loc_conf(ngx_conf_t *cf) | |
| 880 { | |
| 881 ngx_http_perl_loc_conf_t *plcf; | |
| 882 | |
| 883 plcf = ngx_pcalloc(cf->pool, sizeof(ngx_http_perl_loc_conf_t)); | |
| 884 if (plcf == NULL) { | |
|
2912
c7d57b539248
return NULL instead of NGX_CONF_ERROR on a create conf failure
Igor Sysoev <igor@sysoev.ru>
parents:
2721
diff
changeset
|
885 return NULL; |
| 599 | 886 } |
| 887 | |
| 888 /* | |
| 889 * set by ngx_pcalloc(): | |
| 890 * | |
| 891 * plcf->handler = { 0, NULL }; | |
| 892 */ | |
| 893 | |
| 894 return plcf; | |
| 895 } | |
| 896 | |
| 897 | |
| 898 static char * | |
| 899 ngx_http_perl_merge_loc_conf(ngx_conf_t *cf, void *parent, void *child) | |
| 900 { | |
| 901 ngx_http_perl_loc_conf_t *prev = parent; | |
| 902 ngx_http_perl_loc_conf_t *conf = child; | |
| 903 | |
| 904 if (conf->sub == NULL) { | |
| 905 conf->sub = prev->sub; | |
| 906 conf->handler = prev->handler; | |
| 907 } | |
| 908 | |
| 909 return NGX_CONF_OK; | |
| 910 } | |
| 911 | |
| 912 | |
| 913 static char * | |
| 914 ngx_http_perl(ngx_conf_t *cf, ngx_command_t *cmd, void *conf) | |
| 915 { | |
| 916 ngx_http_perl_loc_conf_t *plcf = conf; | |
| 917 | |
| 918 ngx_str_t *value; | |
| 919 ngx_http_core_loc_conf_t *clcf; | |
| 920 ngx_http_perl_main_conf_t *pmcf; | |
| 921 | |
| 922 value = cf->args->elts; | |
| 923 | |
| 924 if (plcf->handler.data) { | |
| 925 ngx_conf_log_error(NGX_LOG_EMERG, cf, 0, | |
| 926 "duplicate perl handler \"%V\"", &value[1]); | |
| 927 return NGX_CONF_ERROR; | |
| 928 } | |
| 929 | |
| 930 pmcf = ngx_http_conf_get_module_main_conf(cf, ngx_http_perl_module); | |
| 931 | |
| 932 if (pmcf->perl == NULL) { | |
| 933 if (ngx_http_perl_init_interpreter(cf, pmcf) != NGX_CONF_OK) { | |
| 934 return NGX_CONF_ERROR; | |
| 935 } | |
| 936 } | |
| 937 | |
| 938 plcf->handler = value[1]; | |
| 939 | |
| 940 { | |
| 941 | |
| 942 dTHXa(pmcf->perl); | |
|
873
f92ad15c2db1
fix segfaults in future workers those will use old configuration,
Igor Sysoev <igor@sysoev.ru>
parents:
872
diff
changeset
|
943 PERL_SET_CONTEXT(pmcf->perl); |
|
6820
eada22643e8b
Perl: added PERL_SET_INTERP().
Maxim Dounin <mdounin@mdounin.ru>
parents:
6791
diff
changeset
|
944 PERL_SET_INTERP(pmcf->perl); |
| 599 | 945 |
| 946 ngx_http_perl_eval_anon_sub(aTHX_ &value[1], &plcf->sub); | |
| 947 | |
| 948 if (plcf->sub == &PL_sv_undef) { | |
| 949 ngx_conf_log_error(NGX_LOG_ERR, cf, 0, | |
| 950 "eval_pv(\"%V\") failed", &value[1]); | |
| 951 return NGX_CONF_ERROR; | |
| 952 } | |
| 953 | |
| 954 if (plcf->sub == NULL) { | |
| 955 plcf->sub = newSVpvn((char *) value[1].data, value[1].len); | |
| 956 } | |
| 957 | |
| 958 } | |
| 959 | |
| 960 clcf = ngx_http_conf_get_module_loc_conf(cf, ngx_http_core_module); | |
| 961 clcf->handler = ngx_http_perl_handler; | |
| 962 | |
| 963 return NGX_CONF_OK; | |
| 964 } | |
| 965 | |
| 966 | |
| 967 static char * | |
| 968 ngx_http_perl_set(ngx_conf_t *cf, ngx_command_t *cmd, void *conf) | |
| 969 { | |
| 970 ngx_int_t index; | |
| 971 ngx_str_t *value; | |
| 972 ngx_http_variable_t *v; | |
| 973 ngx_http_perl_variable_t *pv; | |
| 974 ngx_http_perl_main_conf_t *pmcf; | |
| 975 | |
| 976 value = cf->args->elts; | |
| 977 | |
|
4972
8b635cf36ccc
Added checks that disallow adding a variable with an empty name.
Ruslan Ermilov <ru@nginx.com>
parents:
4963
diff
changeset
|
978 if (value[1].data[0] != '$') { |
| 599 | 979 ngx_conf_log_error(NGX_LOG_EMERG, cf, 0, |
| 980 "invalid variable name \"%V\"", &value[1]); | |
| 981 return NGX_CONF_ERROR; | |
| 982 } | |
| 983 | |
| 984 value[1].len--; | |
| 985 value[1].data++; | |
| 986 | |
| 1565 | 987 v = ngx_http_add_variable(cf, &value[1], NGX_HTTP_VAR_CHANGEABLE); |
| 599 | 988 if (v == NULL) { |
| 989 return NGX_CONF_ERROR; | |
| 990 } | |
| 991 | |
| 992 pv = ngx_palloc(cf->pool, sizeof(ngx_http_perl_variable_t)); | |
| 993 if (pv == NULL) { | |
| 994 return NGX_CONF_ERROR; | |
| 995 } | |
| 996 | |
| 997 index = ngx_http_get_variable_index(cf, &value[1]); | |
| 998 if (index == NGX_ERROR) { | |
| 999 return NGX_CONF_ERROR; | |
| 1000 } | |
| 1001 | |
| 1002 pmcf = ngx_http_conf_get_module_main_conf(cf, ngx_http_perl_module); | |
| 1003 | |
| 1004 if (pmcf->perl == NULL) { | |
| 1005 if (ngx_http_perl_init_interpreter(cf, pmcf) != NGX_CONF_OK) { | |
| 1006 return NGX_CONF_ERROR; | |
| 1007 } | |
| 1008 } | |
| 1009 | |
| 1010 pv->handler = value[2]; | |
| 1011 | |
| 1012 { | |
| 1013 | |
| 1014 dTHXa(pmcf->perl); | |
|
873
f92ad15c2db1
fix segfaults in future workers those will use old configuration,
Igor Sysoev <igor@sysoev.ru>
parents:
872
diff
changeset
|
1015 PERL_SET_CONTEXT(pmcf->perl); |
|
6820
eada22643e8b
Perl: added PERL_SET_INTERP().
Maxim Dounin <mdounin@mdounin.ru>
parents:
6791
diff
changeset
|
1016 PERL_SET_INTERP(pmcf->perl); |
| 599 | 1017 |
| 1018 ngx_http_perl_eval_anon_sub(aTHX_ &value[2], &pv->sub); | |
| 1019 | |
| 1020 if (pv->sub == &PL_sv_undef) { | |
| 1021 ngx_conf_log_error(NGX_LOG_ERR, cf, 0, | |
| 1022 "eval_pv(\"%V\") failed", &value[2]); | |
| 1023 return NGX_CONF_ERROR; | |
| 1024 } | |
| 1025 | |
| 1026 if (pv->sub == NULL) { | |
| 1027 pv->sub = newSVpvn((char *) value[2].data, value[2].len); | |
| 1028 } | |
| 1029 | |
| 1030 } | |
| 1031 | |
| 637 | 1032 v->get_handler = ngx_http_perl_variable; |
| 599 | 1033 v->data = (uintptr_t) pv; |
| 1034 | |
| 1035 return NGX_CONF_OK; | |
| 1036 } | |
|
1221
aa653367028e
PERL_SYS_TERM() should be called once on exit only, this fixes the message
Igor Sysoev <igor@sysoev.ru>
parents:
1165
diff
changeset
|
1037 |
|
aa653367028e
PERL_SYS_TERM() should be called once on exit only, this fixes the message
Igor Sysoev <igor@sysoev.ru>
parents:
1165
diff
changeset
|
1038 |
| 1257 | 1039 static ngx_int_t |
| 1040 ngx_http_perl_init_worker(ngx_cycle_t *cycle) | |
| 1041 { | |
|
1258
5076d97b9cf0
fix building on threaded or multiplicity interpreter perl,
Igor Sysoev <igor@sysoev.ru>
parents:
1257
diff
changeset
|
1042 ngx_http_perl_main_conf_t *pmcf; |
|
5076d97b9cf0
fix building on threaded or multiplicity interpreter perl,
Igor Sysoev <igor@sysoev.ru>
parents:
1257
diff
changeset
|
1043 |
|
5076d97b9cf0
fix building on threaded or multiplicity interpreter perl,
Igor Sysoev <igor@sysoev.ru>
parents:
1257
diff
changeset
|
1044 pmcf = ngx_http_cycle_get_module_main_conf(cycle, ngx_http_perl_module); |
|
5076d97b9cf0
fix building on threaded or multiplicity interpreter perl,
Igor Sysoev <igor@sysoev.ru>
parents:
1257
diff
changeset
|
1045 |
|
2713
b4d8aef4a1ad
fix segfault if no http section is defined in confguraiton,
Igor Sysoev <igor@sysoev.ru>
parents:
2388
diff
changeset
|
1046 if (pmcf) { |
|
b4d8aef4a1ad
fix segfault if no http section is defined in confguraiton,
Igor Sysoev <igor@sysoev.ru>
parents:
2388
diff
changeset
|
1047 dTHXa(pmcf->perl); |
|
b4d8aef4a1ad
fix segfault if no http section is defined in confguraiton,
Igor Sysoev <igor@sysoev.ru>
parents:
2388
diff
changeset
|
1048 PERL_SET_CONTEXT(pmcf->perl); |
|
6820
eada22643e8b
Perl: added PERL_SET_INTERP().
Maxim Dounin <mdounin@mdounin.ru>
parents:
6791
diff
changeset
|
1049 PERL_SET_INTERP(pmcf->perl); |
|
1258
5076d97b9cf0
fix building on threaded or multiplicity interpreter perl,
Igor Sysoev <igor@sysoev.ru>
parents:
1257
diff
changeset
|
1050 |
|
2713
b4d8aef4a1ad
fix segfault if no http section is defined in confguraiton,
Igor Sysoev <igor@sysoev.ru>
parents:
2388
diff
changeset
|
1051 /* set worker's $$ */ |
| 1257 | 1052 |
|
2713
b4d8aef4a1ad
fix segfault if no http section is defined in confguraiton,
Igor Sysoev <igor@sysoev.ru>
parents:
2388
diff
changeset
|
1053 sv_setiv(GvSV(gv_fetchpv("$", TRUE, SVt_PV)), (I32) ngx_pid); |
|
1258
5076d97b9cf0
fix building on threaded or multiplicity interpreter perl,
Igor Sysoev <igor@sysoev.ru>
parents:
1257
diff
changeset
|
1054 } |
|
5076d97b9cf0
fix building on threaded or multiplicity interpreter perl,
Igor Sysoev <igor@sysoev.ru>
parents:
1257
diff
changeset
|
1055 |
| 1257 | 1056 return NGX_OK; |
| 1057 } | |
| 1058 | |
| 1941 | 1059 |
|
1221
aa653367028e
PERL_SYS_TERM() should be called once on exit only, this fixes the message
Igor Sysoev <igor@sysoev.ru>
parents:
1165
diff
changeset
|
1060 static void |
|
aa653367028e
PERL_SYS_TERM() should be called once on exit only, this fixes the message
Igor Sysoev <igor@sysoev.ru>
parents:
1165
diff
changeset
|
1061 ngx_http_perl_exit(ngx_cycle_t *cycle) |
|
aa653367028e
PERL_SYS_TERM() should be called once on exit only, this fixes the message
Igor Sysoev <igor@sysoev.ru>
parents:
1165
diff
changeset
|
1062 { |
| 2715 | 1063 #if (NGX_HAVE_PERL_MULTIPLICITY) |
| 1064 | |
|
3351
b759cf8f6000
add comment from r2716 commit message
Igor Sysoev <igor@sysoev.ru>
parents:
3350
diff
changeset
|
1065 /* |
|
b759cf8f6000
add comment from r2716 commit message
Igor Sysoev <igor@sysoev.ru>
parents:
3350
diff
changeset
|
1066 * the master exit hook is run before global pool cleanup, |
|
b759cf8f6000
add comment from r2716 commit message
Igor Sysoev <igor@sysoev.ru>
parents:
3350
diff
changeset
|
1067 * therefore just set flag here |
|
b759cf8f6000
add comment from r2716 commit message
Igor Sysoev <igor@sysoev.ru>
parents:
3350
diff
changeset
|
1068 */ |
|
b759cf8f6000
add comment from r2716 commit message
Igor Sysoev <igor@sysoev.ru>
parents:
3350
diff
changeset
|
1069 |
| 2715 | 1070 ngx_perl_term = 1; |
| 1071 | |
| 1072 #else | |
|
3351
b759cf8f6000
add comment from r2716 commit message
Igor Sysoev <igor@sysoev.ru>
parents:
3350
diff
changeset
|
1073 |
|
3350
67da53a19e02
use global perl variable in perl_destruct()/perl_free()
Igor Sysoev <igor@sysoev.ru>
parents:
3175
diff
changeset
|
1074 if (nginx_stash) { |
| 2715 | 1075 ngx_log_debug0(NGX_LOG_DEBUG_HTTP, cycle->log, 0, "perl term"); |
| 1076 | |
|
3350
67da53a19e02
use global perl variable in perl_destruct()/perl_free()
Igor Sysoev <igor@sysoev.ru>
parents:
3175
diff
changeset
|
1077 (void) perl_destruct(perl); |
| 2715 | 1078 |
|
3350
67da53a19e02
use global perl variable in perl_destruct()/perl_free()
Igor Sysoev <igor@sysoev.ru>
parents:
3175
diff
changeset
|
1079 perl_free(perl); |
|
1946
171a283af56b
some perl builds require my_perl for PERL_SYS_TERM()
Igor Sysoev <igor@sysoev.ru>
parents:
1941
diff
changeset
|
1080 |
|
2714
4dd1773990db
fix segfault on exit if no http section is defined in confguraiton,
Igor Sysoev <igor@sysoev.ru>
parents:
2713
diff
changeset
|
1081 PERL_SYS_TERM(); |
|
1946
171a283af56b
some perl builds require my_perl for PERL_SYS_TERM()
Igor Sysoev <igor@sysoev.ru>
parents:
1941
diff
changeset
|
1082 } |
| 2715 | 1083 |
| 1084 #endif | |
|
1221
aa653367028e
PERL_SYS_TERM() should be called once on exit only, this fixes the message
Igor Sysoev <igor@sysoev.ru>
parents:
1165
diff
changeset
|
1085 } |