Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 UTC on 18 March 2024, to get $100 off your ticket.
Hi guys, there is a big problem with the code in the module, look:
function soapclient_init() {
global $_soapclient_MCRYPT;
global $_soapclient_AES128;
// check, which encryption library to be used.
$_soapclient_MCRYPT = NULL;
$_soapclient_AES128 = NULL;
if ( ! extension_loaded('mcrypt') ) {
// try loading the mcrypt
$ext = ( drupal_strtoupper( drupal_substr(PHP_OS, 0, 3) ) === 'WIN' ) ? 'php_mcrypt.dll' : 'mcrypt.so';
if ( ! @dl($ext) ) { // THIS CAUSE A FUCKED FAMOUS WSOD (White Screen Of Death)
// cannot load mcrypt, use AES128 for password encryption.
require_once("lib/AES128.php");
$_soapclient_AES128 = new AES128(FALSE, FALSE);
return;
}
}
$_soapclient_MCRYPT = mcrypt_module_open(MCRYPT_3DES, '', 'cbc', '');
}
As you can see the code not match de SO installed (Debian/CentOS/etc...) and not all systems have mcrypt library, look:
http://www.linuxquestions.org/questions/red-hat-31/rhel6-no-php-mcrypt-s...
It would be advisable not to try to load the library. In its place would be better to use the file provided with the module 'AES128.php' to avoid this problem.
Best Regards.
Comment | File | Size | Author |
---|---|---|---|
#1 | soapclient.module.patch | 974 bytes | anieves |
Comments
Comment #1
anieves CreditAttribution: anieves commentedSuggested patch.
Comment #2
gregglesRe-stating the title to match what I think is happening in the code.
Also, this is "needs review."
Comment #3
apadernoI am closing this issue, as Drupal 4.7, 5, and 6 are now not supported.