#18905 [Csd]: apache 2.0.40 changed template for register_*_filter adding additional arg

From: Date: Wed, 14 Aug 2002 17:43:39 +0000
Subject: #18905 [Csd]: apache 2.0.40 changed template for register_*_filter adding additional arg
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-16805@lists.php.net to get a copy of this message
 ID:               18905
 User updated by:  jon.tankersley@eds.com
 Reported By:      jon.tankersley@eds.com
 Status:           Closed
 Bug Type:         Apache related
 Operating System: Solaris 2.8
 PHP Version:      4.2.2
 New Comment:

Adding a NULL as parameter 3 is okay..... but php_functions.c for some
reason is using a deprecated macro.....
MODULE_MAGIC_AT_LEAST
I don't know if it was in 4.2.1, I blew that away last night.

The unstable might be a better bet.


Previous Comments:
------------------------------------------------------------------------

[2002-08-14 12:56:46] duke@mastre.com

So the fix is..:  Either use the latest PHP non-STABLE snapshot (as of
now, php4-200208140900), or use php4-STABLE-200208140900 and modify
sapi/apache2filter/sapi_apache2.c and add NULL as parameter 3 for
ap_register_*_filter (lines 567 & 568).

------------------------------------------------------------------------

[2002-08-14 12:36:17] jon.tankersley@eds.com

It wasn't in the latest download from the php.net main site or easily
found in the bugs database.

There may also be a problem with the 4.2.1-4.2.2 unified diff file. 
That failed with a missing data at the end and I then went after the
4.2.2 full source tree.

Thanks for the quick turn around though.

------------------------------------------------------------------------

[2002-08-14 12:04:21] jon.tankersley@eds.com

Found reference in the Apache changes:

  *) Add a filter_init parameter to the filter registration
     functions so that a filter can execute arbitrary code
     before the handlers are invoked.  This resolves a
     problem where mod_include requests would incorrectly
     return a 304.  [Justin Erenkrantz]

------------------------------------------------------------------------

[2002-08-14 11:49:52] edink@php.net

Just to to clarify:

This bug has been fixed in CVS. You can grab a snapshot of the CVS
version at http://snaps.php.net/.

------------------------------------------------------------------------

[2002-08-14 11:42:43] sander@php.net

Someone already fixed that months ago ;)

------------------------------------------------------------------------

The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
    http://bugs.php.net/18905

-- 
Edit this bug report at http://bugs.php.net/?id=18905&edit=1



Thread (12 messages)

« previous php.bugs (#16805) next »