Regression in 8.5: float values are treated as invalid when updating LDAP values
@Girgias arbeitet bereits daran.
Seit 21.5.2026.
- Vorherrschende Sprache
- C
- Sterne
- 40.4k
- Forks
- 8.1k
- Ø Merge
- 2 T. 13 Std.
- Gemergte PRs (30 T.)
- 96
Beschreibung
Description
The following code:
<?php
ldap_modify($ds, $dn, ['shadowlastchange' => round(time() / (24*3600))]);
Resulted in this output:
LDAP value must be of type string|int|bool, float given (0)
But I expected this output instead:
no output, the operation succeeded before
Problem is caused by php_ldap_try_get_ldap_value_from_zval and used by he following commit: https://github.com/php/php-src/commit/636b6a1a59efe655a17e9a4a2d67865b53babe0c
I can obviously now cast all these kind of timestamp calculations with round(...) to int: (int)round(...), but this is existing code which worked for 10+ years.
So the question is: was that BC breach intended, or does it make sense to handle that cast to int automatically in php_ldap_try_get_ldap_value_from_zval?
Ralf
PHP Version
PHP 8.5.5 (cli) (built: Apr 11 2026 06:53:07) (NTS)
Copyright (c) The PHP Group
Built by Debian
Zend Engine v4.5.5, Copyright (c) Zend Technologies
with Zend OPcache v8.5.5, Copyright (c), by Zend Technologies
Operating System
No response
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Bewertung
Dieses Issue wurde noch nicht bewertet.