Mercurial > hg > nginx
annotate src/http/modules/perl/ngx_http_perl_module.h @ 7524:deebe988cbd7
Perl: reworked perl module to pass ctx instead of request.
This ensures that correct ctx is always available, including after
filter finalization. In particular, this fixes a segmentation fault
with the following configuration:
location / {
image_filter test;
perl 'sub {
my $r = shift;
$r->send_http_header();
$r->print("foo\n");
$r->print("bar\n");
}';
}
This also seems to be the only way to correctly handle filter finalization
in various complex cases, for example, when embedded perl is used both
in the original handler and in an error page called after filter
finalization.
| author | Maxim Dounin <mdounin@mdounin.ru> |
|---|---|
| date | Fri, 12 Jul 2019 11:29:22 +0300 |
| parents | d620f497c50f |
| children | 575480d3fd01 |
| rev | line source |
|---|---|
| 599 | 1 |
| 2 /* | |
| 3 * Copyright (C) Igor Sysoev | |
| 4412 | 4 * Copyright (C) Nginx, Inc. |
| 599 | 5 */ |
| 6 | |
| 7 | |
| 8 #ifndef _NGX_HTTP_PERL_MODULE_H_INCLUDED_ | |
| 9 #define _NGX_HTTP_PERL_MODULE_H_INCLUDED_ | |
| 10 | |
| 11 | |
| 12 #include <ngx_config.h> | |
| 13 #include <ngx_core.h> | |
| 14 #include <ngx_http.h> | |
| 775 | 15 #include <nginx.h> |
| 599 | 16 |
| 17 #include <EXTERN.h> | |
| 18 #include <perl.h> | |
| 19 | |
| 20 | |
| 21 typedef ngx_http_request_t *nginx; | |
| 22 | |
| 23 typedef struct { | |
|
7524
deebe988cbd7
Perl: reworked perl module to pass ctx instead of request.
Maxim Dounin <mdounin@mdounin.ru>
parents:
4412
diff
changeset
|
24 ngx_http_request_t *request; |
|
deebe988cbd7
Perl: reworked perl module to pass ctx instead of request.
Maxim Dounin <mdounin@mdounin.ru>
parents:
4412
diff
changeset
|
25 |
| 633 | 26 ngx_str_t filename; |
| 599 | 27 ngx_str_t redirect_uri; |
| 28 ngx_str_t redirect_args; | |
| 911 | 29 |
| 681 | 30 SV *next; |
| 31 | |
| 1897 | 32 ngx_uint_t done; /* unsigned done:1; */ |
| 599 | 33 |
|
912
7fa926a7926d
$r->variable() supports perl only variables
Igor Sysoev <igor@sysoev.ru>
parents:
911
diff
changeset
|
34 ngx_array_t *variables; /* array of ngx_http_perl_var_t */ |
|
7fa926a7926d
$r->variable() supports perl only variables
Igor Sysoev <igor@sysoev.ru>
parents:
911
diff
changeset
|
35 |
| 617 | 36 #if (NGX_HTTP_SSI) |
| 599 | 37 ngx_http_ssi_ctx_t *ssi; |
| 617 | 38 #endif |
| 599 | 39 } ngx_http_perl_ctx_t; |
| 40 | |
| 41 | |
|
912
7fa926a7926d
$r->variable() supports perl only variables
Igor Sysoev <igor@sysoev.ru>
parents:
911
diff
changeset
|
42 typedef struct { |
|
7fa926a7926d
$r->variable() supports perl only variables
Igor Sysoev <igor@sysoev.ru>
parents:
911
diff
changeset
|
43 ngx_uint_t hash; |
|
7fa926a7926d
$r->variable() supports perl only variables
Igor Sysoev <igor@sysoev.ru>
parents:
911
diff
changeset
|
44 ngx_str_t name; |
|
7fa926a7926d
$r->variable() supports perl only variables
Igor Sysoev <igor@sysoev.ru>
parents:
911
diff
changeset
|
45 ngx_str_t value; |
|
7fa926a7926d
$r->variable() supports perl only variables
Igor Sysoev <igor@sysoev.ru>
parents:
911
diff
changeset
|
46 } ngx_http_perl_var_t; |
|
7fa926a7926d
$r->variable() supports perl only variables
Igor Sysoev <igor@sysoev.ru>
parents:
911
diff
changeset
|
47 |
|
7fa926a7926d
$r->variable() supports perl only variables
Igor Sysoev <igor@sysoev.ru>
parents:
911
diff
changeset
|
48 |
| 599 | 49 extern ngx_module_t ngx_http_perl_module; |
| 50 | |
| 51 | |
| 52 /* | |
| 53 * workaround for "unused variable `Perl___notused'" warning | |
| 54 * when building with perl 5.6.1 | |
| 55 */ | |
| 56 #ifndef PERL_IMPLICIT_CONTEXT | |
| 57 #undef dTHXa | |
| 58 #define dTHXa(a) | |
| 59 #endif | |
| 60 | |
| 61 | |
| 868 | 62 extern void boot_DynaLoader(pTHX_ CV* cv); |
| 599 | 63 |
| 64 | |
| 681 | 65 void ngx_http_perl_handle_request(ngx_http_request_t *r); |
| 1897 | 66 void ngx_http_perl_sleep_handler(ngx_http_request_t *r); |
| 681 | 67 |
| 68 | |
| 599 | 69 #endif /* _NGX_HTTP_PERL_MODULE_H_INCLUDED_ */ |