Skip to content

Commit 51f6d43

Browse files
committed
Revert "drop use of NRSAFESTR when accessing ..."
This partially reverts commit 3a356ea. `NRSAFESTR` should still be used when accessing package_name and package_version for debug log message.
1 parent 5a73606 commit 51f6d43

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

agent/lib_composer.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,8 @@ static void nr_execute_handle_autoload_composer_get_packages_information(
146146
}
147147
if (nr_php_is_zval_non_empty_string(package_version)) {
148148
nrl_verbosedebug(NRL_INSTRUMENT, "package %s, version %s",
149-
ZSTR_VAL(package_name), Z_STRVAL_P(package_version));
149+
NRSAFESTR(ZSTR_VAL(package_name)),
150+
NRSAFESTR(Z_STRVAL_P(package_version)));
150151
nr_txn_add_php_package_from_source(NRPRG(txn), ZSTR_VAL(package_name),
151152
Z_STRVAL_P(package_version),
152153
NR_PHP_PACKAGE_SOURCE_COMPOSER);

0 commit comments

Comments
 (0)