From: Joe Watkins <krakjoe@php.net>
Date: Mon, 28 Mar 2016 10:54:25 +0000 (+0100)
Subject: work on #71428
X-Git-Tag: php-7.0.6RC1~59
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=ee9a78a033696ff9546fb1dbfecd28f20477b511;p=php

work on #71428
---

diff --git a/Zend/tests/bug71428.1.phpt b/Zend/tests/bug71428.1.phpt
new file mode 100644
index 0000000000..b754687560
--- /dev/null
+++ b/Zend/tests/bug71428.1.phpt
@@ -0,0 +1,13 @@
+--TEST--
+bug #71428.1: inheritance with null default values
+--FILE--
+<?php
+class A {
+    public function m(array $a = null) {}
+}
+class B extends A {
+    public function m(array $a = []) {}
+}
+--EXPECTF--
+Warning: Declaration of B::m(array $a = Array) should be compatible with A::m(array $a = NULL) in %sbug71428.1.php on line 7
+
diff --git a/Zend/zend_inheritance.c b/Zend/zend_inheritance.c
index 9eb7a0b9c9..d904a587a1 100644
--- a/Zend/zend_inheritance.c
+++ b/Zend/zend_inheritance.c
@@ -238,6 +238,11 @@ static int zend_do_perform_type_hint_check(const zend_function *fe, zend_arg_inf
 		zend_string_release(fe_class_name);
 	}
 
+	if (proto_arg_info->allow_null && !fe_arg_info->allow_null) {
+		/* incompatible nullability */
+		return 0;
+	}
+
 	if (fe_arg_info->type_hint != proto_arg_info->type_hint) {
 		/* Incompatible type */
 		return 0;