From 4d07d6bbb74e7ebe1f95cf2b89ae99aa35a9eeb2 Mon Sep 17 00:00:00 2001 From: Remi Collet Date: Fri, 13 Sep 2024 11:47:54 +0200 Subject: rename constant from XPASS_CRYPT_ to CRYPT_PREFIX_ --- package.xml | 2 +- tests/crypt_checksalt.phpt | 2 +- tests/crypt_gensalt.phpt | 18 +++++++++--------- tests/password_compat.phpt | 2 +- xpass.c | 8 ++++---- xpass.stub.php | 22 ++++++++++------------ xpass_arginfo.h | 22 +++++++++++----------- 7 files changed, 37 insertions(+), 39 deletions(-) diff --git a/package.xml b/package.xml index b47f016..f59c2bd 100644 --- a/package.xml +++ b/package.xml @@ -27,7 +27,7 @@ distributions, using extended crypt library (libxcrypt): PHP-3.01 -- add crypt_gensalt(?string $salt = null, int $count = 0): ?string {} +- add crypt_gensalt(?string $prefix = null, int $count = 0): ?string {} - add crypt_preferred_method(): ?string {} - add crypt_checksalt(string $salt): int {} diff --git a/tests/crypt_checksalt.phpt b/tests/crypt_checksalt.phpt index f34f585..0a0f6ee 100644 --- a/tests/crypt_checksalt.phpt +++ b/tests/crypt_checksalt.phpt @@ -3,7 +3,7 @@ Test crypt_checksalt --FILE-- --EXPECTF-- diff --git a/tests/password_compat.phpt b/tests/password_compat.phpt index 3dd6ad2..2571b02 100644 --- a/tests/password_compat.phpt +++ b/tests/password_compat.phpt @@ -9,7 +9,7 @@ $h = password_hash($secret, PASSWORD_BCRYPT); var_dump($h, password_verify($secret, $h), $h===crypt($secret, $h)); /* generate with crypt, check with both */ -$h = crypt($secret, crypt_gensalt(XPASS_CRYPT_BLOWFISH)); +$h = crypt($secret, crypt_gensalt(CRYPT_PREFIX_BLOWFISH)); var_dump($h, password_verify($secret, $h), $h===crypt($secret, $h)); ?> --EXPECTF-- diff --git a/xpass.c b/xpass.c index cb16131..b37863e 100644 --- a/xpass.c +++ b/xpass.c @@ -190,14 +190,14 @@ PHP_FUNCTION(crypt_preferred_method) /* {{{ Determine whether the user's passphrase should be re-hashed using the currently preferred hashing method */ PHP_FUNCTION(crypt_checksalt) { - char *prefix; - size_t prefix_len; + char *salt; + size_t salt_len; ZEND_PARSE_PARAMETERS_START(1, 1) - Z_PARAM_STRING(prefix, prefix_len) + Z_PARAM_STRING(salt, salt_len) ZEND_PARSE_PARAMETERS_END(); - RETURN_LONG(crypt_checksalt(prefix)); + RETURN_LONG(crypt_checksalt(salt)); } /* }}} */ diff --git a/xpass.stub.php b/xpass.stub.php index 4cb0d67..68718da 100644 --- a/xpass.stub.php +++ b/xpass.stub.php @@ -6,26 +6,24 @@ */ -/* use XPASS prefix to avoid conflicts with standard constants */ - /** @var string */ -const XPASS_CRYPT_STD_DES = ''; +const CRYPT_PREFIX_STD_DES = ''; /** @var string */ -const XPASS_CRYPT_EXT_DES = '_'; +const CRYPT_PREFIX_EXT_DES = '_'; /** @var string */ -const XPASS_CRYPT_MD5 = '$1$'; +const CRYPT_PREFIX_MD5 = '$1$'; /** @var string */ -const XPASS_CRYPT_BLOWFISH = '$2y$'; +const CRYPT_PREFIX_BLOWFISH = '$2y$'; /** @var string */ -const XPASS_CRYPT_SHA256 = '$5$'; +const CRYPT_PREFIX_SHA256 = '$5$'; /** @var string */ -const XPASS_CRYPT_SHA512 = '$6$'; +const CRYPT_PREFIX_SHA512 = '$6$'; /** @var string */ -const XPASS_CRYPT_SCRYPT = '$7$'; +const CRYPT_PREFIX_SCRYPT = '$7$'; /** @var string */ -const XPASS_CRYPT_GOST_YESCRYPT = '$gy$'; +const CRYPT_PREFIX_GOST_YESCRYPT = '$gy$'; /** @var string */ -const XPASS_CRYPT_YESCRYPT = '$y$'; +const CRYPT_PREFIX_YESCRYPT = '$y$'; /** * @var int @@ -54,7 +52,7 @@ const CRYPT_SALT_METHOD_LEGACY = UNKNOWN; const CRYPT_SALT_TOO_CHEAP = UNKNOWN; -function crypt_gensalt(?string $salt = null, int $count = 0): ?string {} +function crypt_gensalt(?string $prefix = null, int $count = 0): ?string {} function crypt_preferred_method(): ?string {} diff --git a/xpass_arginfo.h b/xpass_arginfo.h index 87ad78f..f530d0f 100644 --- a/xpass_arginfo.h +++ b/xpass_arginfo.h @@ -1,8 +1,8 @@ /* This is a generated file, edit the .stub.php file instead. - * Stub hash: 09c1d047a579afe870fadd302bd2e34a5d06c370 */ + * Stub hash: 9f75db3279543b07de6b59e720e8521694200a7c */ ZEND_BEGIN_ARG_WITH_RETURN_TYPE_INFO_EX(arginfo_crypt_gensalt, 0, 0, IS_STRING, 1) - ZEND_ARG_TYPE_INFO_WITH_DEFAULT_VALUE(0, salt, IS_STRING, 1, "null") + ZEND_ARG_TYPE_INFO_WITH_DEFAULT_VALUE(0, prefix, IS_STRING, 1, "null") ZEND_ARG_TYPE_INFO_WITH_DEFAULT_VALUE(0, count, IS_LONG, 0, "0") ZEND_END_ARG_INFO() @@ -26,15 +26,15 @@ static const zend_function_entry ext_functions[] = { static void register_xpass_symbols(int module_number) { - REGISTER_STRING_CONSTANT("XPASS_CRYPT_STD_DES", "", CONST_PERSISTENT); - REGISTER_STRING_CONSTANT("XPASS_CRYPT_EXT_DES", "_", CONST_PERSISTENT); - REGISTER_STRING_CONSTANT("XPASS_CRYPT_MD5", "$1$", CONST_PERSISTENT); - REGISTER_STRING_CONSTANT("XPASS_CRYPT_BLOWFISH", "$2y$", CONST_PERSISTENT); - REGISTER_STRING_CONSTANT("XPASS_CRYPT_SHA256", "$5$", CONST_PERSISTENT); - REGISTER_STRING_CONSTANT("XPASS_CRYPT_SHA512", "$6$", CONST_PERSISTENT); - REGISTER_STRING_CONSTANT("XPASS_CRYPT_SCRYPT", "$7$", CONST_PERSISTENT); - REGISTER_STRING_CONSTANT("XPASS_CRYPT_GOST_YESCRYPT", "$gy$", CONST_PERSISTENT); - REGISTER_STRING_CONSTANT("XPASS_CRYPT_YESCRYPT", "$y$", CONST_PERSISTENT); + REGISTER_STRING_CONSTANT("CRYPT_PREFIX_STD_DES", "", CONST_PERSISTENT); + REGISTER_STRING_CONSTANT("CRYPT_PREFIX_EXT_DES", "_", CONST_PERSISTENT); + REGISTER_STRING_CONSTANT("CRYPT_PREFIX_MD5", "$1$", CONST_PERSISTENT); + REGISTER_STRING_CONSTANT("CRYPT_PREFIX_BLOWFISH", "$2y$", CONST_PERSISTENT); + REGISTER_STRING_CONSTANT("CRYPT_PREFIX_SHA256", "$5$", CONST_PERSISTENT); + REGISTER_STRING_CONSTANT("CRYPT_PREFIX_SHA512", "$6$", CONST_PERSISTENT); + REGISTER_STRING_CONSTANT("CRYPT_PREFIX_SCRYPT", "$7$", CONST_PERSISTENT); + REGISTER_STRING_CONSTANT("CRYPT_PREFIX_GOST_YESCRYPT", "$gy$", CONST_PERSISTENT); + REGISTER_STRING_CONSTANT("CRYPT_PREFIX_YESCRYPT", "$y$", CONST_PERSISTENT); REGISTER_LONG_CONSTANT("CRYPT_SALT_OK", CRYPT_SALT_OK, CONST_PERSISTENT); REGISTER_LONG_CONSTANT("CRYPT_SALT_INVALID", CRYPT_SALT_INVALID, CONST_PERSISTENT); REGISTER_LONG_CONSTANT("CRYPT_SALT_METHOD_DISABLED", CRYPT_SALT_METHOD_DISABLED, CONST_PERSISTENT); -- cgit