archive/php-src
1
0
Fork 0
mirror of https://github.com/php/php-src.git synced 2025-08-15 21:48:51 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

Don't evaluate GMP comparison multiple times (#18321)

Browse source 
ZEND_THREEWAY_COMPARE evaluates its operands multiple times.
This commit is contained in:
Niels Dossche 2025-04-14 11:19:14 +02:00 • committed by GitHub
parent 4eb8839037
commit a142f10aa1
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 4 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

6
ext/gmp/gmp.c
View file

@ -480,7 +480,8 @@ static int gmp_compare(zval *op1, zval *op2) /* {{{ */
return ZEND_UNCOMPARABLE; return ZEND_UNCOMPARABLE;
} }
return ZEND_THREEWAY_COMPARE(mpz_cmp(gmp_op1, gmp_op2), 0); int ret = mpz_cmp(gmp_op1, gmp_op2); /* avoid multiple evaluations */
return ZEND_THREEWAY_COMPARE(ret, 0);
} }
/* }}} */ /* }}} */
@ -1422,7 +1423,8 @@ ZEND_FUNCTION(gmp_cmp)
GMP_Z_PARAM_INTO_MPZ_PTR(gmpnum_b) GMP_Z_PARAM_INTO_MPZ_PTR(gmpnum_b)
ZEND_PARSE_PARAMETERS_END(); ZEND_PARSE_PARAMETERS_END();
RETURN_LONG(ZEND_THREEWAY_COMPARE(mpz_cmp(gmpnum_a, gmpnum_b), 0)); int ret = mpz_cmp(gmpnum_a, gmpnum_b); /* avoid multiple evaluations */
RETURN_LONG(ZEND_THREEWAY_COMPARE(ret, 0));
} }
/* }}} */ /* }}} */