diff options
| author | Gabriel Caruso <carusogabriel34@gmail.com> | 2020-04-23 20:17:55 +0200 |
|---|---|---|
| committer | Gabriel Caruso <carusogabriel34@gmail.com> | 2020-04-26 02:16:39 +0200 |
| commit | 6f908a0bf4b51cfe3327e3eb245af8831c5cac4d (patch) | |
| tree | bf39dd6af7dc832575b3339ed437a6a5a069a436 /Zend/zend_API.c | |
| parent | 5bf01fca60ed802db9182b495b229846d8c95af4 (diff) | |
| download | php-git-6f908a0bf4b51cfe3327e3eb245af8831c5cac4d.tar.gz | |
Check Serialization magic methods structure
Closes GH-5441
Diffstat (limited to 'Zend/zend_API.c')
| -rw-r--r-- | Zend/zend_API.c | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/Zend/zend_API.c b/Zend/zend_API.c index 7ec8806d18..c144bf82fe 100644 --- a/Zend/zend_API.c +++ b/Zend/zend_API.c @@ -2023,6 +2023,18 @@ ZEND_API void zend_check_magic_method_implementation(const zend_class_entry *ce, } else if (name_len == sizeof(ZEND_DEBUGINFO_FUNC_NAME) - 1 && !memcmp(lcname, ZEND_DEBUGINFO_FUNC_NAME, sizeof(ZEND_DEBUGINFO_FUNC_NAME)-1) && fptr->common.num_args != 0) { zend_error(error_type, "Method %s::__debugInfo() cannot take arguments", ZSTR_VAL(ce->name)); + } else if ( + name_len == sizeof("__serialize") - 1 + && !memcmp(lcname, "__serialize", sizeof("__serialize") - 1) + && fptr->common.num_args != 0 + ) { + zend_error(error_type, "Method %s::__serialize() cannot take arguments", ZSTR_VAL(ce->name)); + } else if ( + name_len == sizeof("__unserialize") - 1 + && !memcmp(lcname, "__unserialize", sizeof("__unserialize") - 1) + && fptr->common.num_args != 1 + ) { + zend_error(error_type, "Method %s::__unserialize() must take exactly 1 argument", ZSTR_VAL(ce->name)); } } /* }}} */ |