Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

compatibility with PHP 5.3.3 #6

Closed
christopherowen opened this issue Apr 2, 2013 · 4 comments
Closed

compatibility with PHP 5.3.3 #6

christopherowen opened this issue Apr 2, 2013 · 4 comments

Comments

@christopherowen
Copy link

PHP version 5.3.3 is the version that ships with Redhat Enterprise Linux 6 and CentOS 6.

make test will return the following on these systems:

Warning: PHP Startup: Unable to load dynamic library '/usr/local/src/taint-1.2.2/modules/taint.so' - /usr/local/src/taint-1.2.2/modules/taint.so: undefined symbol: INIT_PZVAL_COPY in Unknown on line 0

and compared to a php 5.4 installation, INIT_PZVAL_COPY is truly not defined in /usr/include/php/Zend/zend.h.

Is there any way to get taint working with the php version shipped by RHEL/CentOS?

@christopherowen
Copy link
Author

Adding the following in /usr/include/php/Zend/zend.h allows make test to work:

#define ZVAL_COPY_VALUE(z, v)
do {
(z)->value = (v)->value;
Z_TYPE_P(z) = Z_TYPE_P(v);
} while (0)

#define INIT_PZVAL_COPY(z, v)
do {
ZVAL_COPY_VALUE(z, v);
Z_SET_REFCOUNT_P(z, 1);
Z_UNSET_ISREF_P(z);
} while (0)

@laruence
Copy link
Owner

laruence commented Apr 3, 2013

thanks for reporting this, I will fix the build, thanks

@jimmyyem
Copy link

jimmyyem commented Jan 5, 2015

I installed taint and I download source code from: http://pecl.php.net/package/taint. Then I restart http but I found the following error message from /tmp/php_error.log

[05-Jan-2015 16:54:08] PHP Warning: PHP Startup: Unable to load dynamic library '/usr/lib64/php/modules/taint.so' - /usr/lib64/php/modules/taint.so: undefined symbol: INIT_PZVAL_COPY in Unknown on line 0

@christopherowen
Copy link
Author

the patch is still necessary

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants