Commit ce534c612b4 for php.net

commit ce534c612b4cd6bd8445c70d8d8ee794d1076b5f
Author: Niels Dossche <7771979+ndossche@users.noreply.github.com>
Date:   Sun Dec 28 04:04:16 2025 -0800

    Fix GH-19961: Static analysis arrayIndexThenCheck warning in firebird (#20790)

    Static analysis reports that the bounds check comes after reading the
    byte from the buffer.
    In practice, this is tagged data that loops until the end tag is found
    and therefore there isn't really a bug. The extra length check is only
    there for extra hardening.
    So we simply silence the static analysers and improve the hardening.

    See also https://docwiki.embarcadero.com/InterBase/15/en/Isc_dsql_sql_info()

diff --git a/ext/pdo_firebird/firebird_driver.c b/ext/pdo_firebird/firebird_driver.c
index 504e739a974..ef94d6fcb00 100644
--- a/ext/pdo_firebird/firebird_driver.c
+++ b/ext/pdo_firebird/firebird_driver.c
@@ -632,7 +632,7 @@ static zend_long firebird_handle_doer(pdo_dbh_t *dbh, const zend_string *sql) /*
 			ret = -1;
 			goto free_statement;
 		}
-		while (result[i] != isc_info_end && i < result_size) {
+		while (i < result_size && result[i] != isc_info_end) {
 			short len = (short)isc_vax_integer(&result[i+1],2);
 			/* bail out on bad len */
 			if (len != 1 && len != 2 && len != 4) {