- Add missing check

felipensp · felipensp · commit 46983d532e7e · 2010-11-14T01:38:52.000Z
diff --git a/Zend/zend_constants.c b/Zend/zend_constants.c
@@ -440,7 +440,8 @@ ZEND_API int zend_register_constant(zend_constant *c TSRMLS_DC)
 		|| zend_hash_add(EG(zend_constants), name, c->name_len, (void *) c, sizeof(zend_constant), NULL)==FAILURE) {
 		
 		/* The internal __COMPILER_HALT_OFFSET__ is prefixed by NULL byte */
-		if (memcmp(name, "\0__COMPILER_HALT_OFFSET__", sizeof("\0__COMPILER_HALT_OFFSET__")) == 0) {
+		if (c->name[0] == '\0' && c->name_len > sizeof("\0__COMPILER_HALT_OFFSET__")
+			&& memcmp(name, "\0__COMPILER_HALT_OFFSET__", sizeof("\0__COMPILER_HALT_OFFSET__")) == 0) {
 			name++;
 		}
 		zend_error(E_NOTICE,"Constant %s already defined", name);

Original file line number	Diff line number	Diff line change
`@@ -440,7 +440,8 @@ ZEND_API int zend_register_constant(zend_constant *c TSRMLS_DC)`
`440`	`440`	`\|\| zend_hash_add(EG(zend_constants), name, c->name_len, (void *) c, sizeof(zend_constant), NULL)==FAILURE) {`
`441`	`441`
`442`	`442`	`/* The internal __COMPILER_HALT_OFFSET__ is prefixed by NULL byte */`
`443`		`- if (memcmp(name, "\0__COMPILER_HALT_OFFSET__", sizeof("\0__COMPILER_HALT_OFFSET__")) == 0) {`
	`443`	`+ if (c->name[0] == '\0' && c->name_len > sizeof("\0__COMPILER_HALT_OFFSET__")`
	`444`	`+ && memcmp(name, "\0__COMPILER_HALT_OFFSET__", sizeof("\0__COMPILER_HALT_OFFSET__")) == 0) {`
`444`	`445`	`name++;`
`445`	`446`	`}`
`446`	`447`	`zend_error(E_NOTICE,"Constant %s already defined", name);`