Skip to content

use zend_string_equals* APIs for string comparisons #21707

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
LamentXU123 wants to merge 5 commits into
base: master
Choose a base branch
from
+5 −4
Open

use zend_string_equals* APIs for string comparisons #21707

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
a4db4b2
refactor
LamentXU123 Apr 10, 2026
c49b8a5
Update php_encoding.c
LamentXU123 Apr 10, 2026
5e00917
Update php_encoding.c
LamentXU123 Apr 10, 2026
273d921
Update php_encoding.c
LamentXU123 Apr 10, 2026
21124e0
apply review changes
LamentXU123 Apr 14, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
apply review changes
  • Loading branch information
@LamentXU123
LamentXU123 authored Apr 14, 2026
commit 21124e0988798b9a8083feeb54012bdc2946db65
2 changes: 1 addition & 1 deletion Zend/zend_object_handlers.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -563,7 +563,7 @@ ZEND_API zend_result zend_check_property_access(const zend_object *zobj, zend_st
if (!(property_info->flags & ZEND_ACC_PRIVATE)) {
/* we we're looking for a private prop but found a non private one of the same name */
return FAILURE;
} else if (!zend_string_equals(prop_info_name, property_info->name)) {
} else if (strcmp(ZSTR_VAL(prop_info_name)+1, ZSTR_VAL(property_info->name)+1)) {
/* we we're looking for a private prop but found a private one of the same name but another class */
return FAILURE;
}
Expand Down
Loading