- Fix Bug #35720 A final constructor can be overwritten

diff --git a/Zend/zend_compile.c b/Zend/zend_compile.c
index ff20c63625cfbf1eacf9a5dd2edacf4185057e75..8532fcb46f6397652e7a879e555c500cbe938dbd 100644 (file)
--- a/Zend/zend_compile.c
+++ b/Zend/zend_compile.c
@@ -1941,6 +1941,12 @@ static void do_inherit_parent_constructor(zend_class_entry *ce TSRMLS_DC)
                ce->destructor   = ce->parent->destructor;
        }
        if (ce->constructor) {
+               if (ce->parent->constructor && ce->parent->constructor->common.fn_flags & ZEND_ACC_FINAL) {
+                       zend_error(E_ERROR, "Cannot override final %v::%v() with %v::%v()",
+                               ce->parent->name, ce->parent->constructor->common.function_name,
+                               ce->name, ce->constructor->common.function_name
+                               );
+               }
                return;
        }
        

diff --git a/tests/classes/final_ctor1.phpt b/tests/classes/final_ctor1.phpt
new file mode 100755 (executable)
index 0000000..ebfa080
--- /dev/null
+++ b/tests/classes/final_ctor1.phpt
@@ -0,0 +1,29 @@
+--TEST--
+ZE2 cannot override final __construct
+--FILE--
+<?php
+
+class Base
+{
+       public final function __construct()
+       {
+       }
+}
+
+class Works extends Base
+{
+}
+
+class Extended extends Base
+{
+       public function Extended()
+       {
+       }
+}
+
+ReflectionClass::export('Extended');
+
+?>
+--EXPECTF--
+
+Fatal error: Cannot override final Base::__construct() with Extended::Extended() in %sfinal_ctor1.php on line %d

diff --git a/tests/classes/final_ctor2.phpt b/tests/classes/final_ctor2.phpt
new file mode 100755 (executable)
index 0000000..905337b
--- /dev/null
+++ b/tests/classes/final_ctor2.phpt
@@ -0,0 +1,29 @@
+--TEST--
+ZE2 cannot override final old style ctor
+--FILE--
+<?php
+
+class Base
+{
+       public final function Base()
+       {
+       }
+}
+
+class Works extends Base
+{
+}
+
+class Extended extends Base
+{
+       public function __construct()
+       {
+       }
+}
+
+ReflectionClass::export('Extended');
+
+?>
+--EXPECTF--
+
+Fatal error: Cannot override final Base::Base() with Extended::__construct() in %sfinal_ctor2.php on line %d