Skip to content

Commit ceadf78

Browse files
committed
Merge branch 'PHP-5.5' into PHP-5.6
* PHP-5.5: BFN Fixed bug #67715 (php-milter does not build and crashes randomly).
2 parents 7fe21ce + 48cadf2 commit ceadf78

File tree

3 files changed

+10
-6
lines changed

3 files changed

+10
-6
lines changed

sapi/milter/Makefile.frag

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
milter: $(SAPI_MILTER_PATH)
22

3-
$(SAPI_MILTER_PATH): $(PHP_GLOBAL_OBJS) $(PHP_BINARY_OBJS) $(PHP_SAPI_OBJS)
3+
$(SAPI_MILTER_PATH): $(PHP_GLOBAL_OBJS) $(PHP_BINARY_OBJS) $(PHP_MILTER_OBJS)
44
$(BUILD_MILTER)
55

66
install-milter: $(SAPI_MILTER_PATH)
7-
@$(INSTALL) -m 0755 $(SAPI_MILTER_PATH) $(bindir)/php-milter
7+
@$(INSTALL) -m 0755 $(SAPI_MILTER_PATH) $(INSTALL_ROOT)$(bindir)/php-milter
88

sapi/milter/config.m4

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,10 +22,10 @@ if test "$PHP_MILTER" != "no"; then
2222

2323
SAPI_MILTER_PATH=sapi/milter/php-milter
2424
PHP_BUILD_THREAD_SAFE
25-
PHP_ADD_MAKEFILE_FRAGMENT($abs_srcdir/sapi/milter/Makefile.frag)
25+
PHP_ADD_MAKEFILE_FRAGMENT($abs_srcdir/sapi/milter/Makefile.frag,$abs_srcdir/sapi/milter,sapi/milter)
2626
PHP_SELECT_SAPI(milter, program, php_milter.c getopt.c,,'$(SAPI_MILTER_PATH)')
2727
PHP_ADD_LIBRARY_WITH_PATH(milter, $MILTERPATH,)
28-
BUILD_MILTER="\$(LIBTOOL) --mode=link \$(CC) -export-dynamic \$(CFLAGS_CLEAN) \$(EXTRA_CFLAGS) \$(EXTRA_LDFLAGS) \$(LDFLAGS) \$(PHP_RPATHS) \$(PHP_GLOBAL_OBJS) \$(PHP_BINARY_OBJS) \$(PHP_MILTER_OBJS) \$(EXTRA_LIBS) \$(ZEND_EXTRA_LIBS) -o \$(SAPI_MILTER_PATH)"
28+
BUILD_MILTER="\$(LIBTOOL) --mode=link \$(CC) -export-dynamic \$(CFLAGS_CLEAN) \$(EXTRA_CFLAGS) \$(EXTRA_LDFLAGS_PROGRAM) \$(LDFLAGS) \$(PHP_RPATHS) \$(PHP_GLOBAL_OBJS) \$(PHP_BINARY_OBJS) \$(PHP_MILTER_OBJS) \$(EXTRA_LIBS) \$(ZEND_EXTRA_LIBS) -o \$(SAPI_MILTER_PATH)"
2929
PHP_SUBST(SAPI_MILTER_PATH)
3030
PHP_SUBST(BUILD_MILTER)
3131
fi

sapi/milter/php_milter.c

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -525,6 +525,10 @@ static sfsistat mlfi_close(SMFICTX *ctx)
525525
int status;
526526
TSRMLS_FETCH();
527527

528+
if (!SG(sapi_started) && SUCCESS != php_request_startup(TSRMLS_C)) {
529+
return ret;
530+
}
531+
528532
/* call userland */
529533
INIT_ZVAL(function_name);
530534
ZVAL_STRING(&function_name, "milter_close", 0);
@@ -550,7 +554,7 @@ static sfsistat mlfi_close(SMFICTX *ctx)
550554

551555
/* {{{ Milter entry struct
552556
*/
553-
struct smfiDesc smfilter = {
557+
static struct smfiDesc smfilter = {
554558
"php-milter", /* filter name */
555559
SMFI_VERSION, /* version code -- leave untouched */
556560
0, /* flags */
@@ -1015,6 +1019,7 @@ int main(int argc, char *argv[])
10151019

10161020

10171021
tsrm_startup(1, 1, 0, NULL);
1022+
tsrm_ls = ts_resource(0);
10181023
sapi_startup(&milter_sapi_module);
10191024

10201025
while ((c=ap_php_getopt(argc, argv, OPTSTRING))!=-1) {
@@ -1032,7 +1037,6 @@ int main(int argc, char *argv[])
10321037

10331038
milter_sapi_module.executable_location = argv[0];
10341039

1035-
tsrm_ls = ts_resource(0);
10361040

10371041
sapi_module.startup(&milter_sapi_module);
10381042

0 commit comments

Comments
 (0)