Fix now manual stream checks

Author: Girgias

ext/libxml/libxml.c

@@ -771,11 +771,17 @@ static xmlParserInputPtr php_libxml_external_entity_loader(const char *URL,
 		} else if (Z_TYPE(retval) == IS_RESOURCE) {
 			php_stream *stream;
 			php_stream_from_zval_no_verify(stream, &retval);
-			if (stream == NULL) {
+			if (UNEXPECTED(stream == NULL)) {
+				zval callable;
+				zend_get_callable_zval_from_fcc(&LIBXML(entity_loader_callback), &callable);
+
+				zend_string *callable_name = zend_get_callable_name(&callable);
 				php_libxml_ctx_error(context,
 						"The user entity loader callback '%s' has returned a "
 						"resource, but it is not a stream",
-						ZSTR_VAL(LIBXML(entity_loader_callback).function_handler->common.function_name));
+						ZSTR_VAL(callable_name));
+				zend_string_release(callable_name);
+				zval_dtor(&callable);
 			} else {
 				/* TODO: allow storing the encoding in the stream context? */
 				xmlCharEncoding enc = XML_CHAR_ENCODING_NONE;

ext/libxml/tests/libxml_get_external_entity_loader_error_callback_name.phpt

@@ -36,5 +36,8 @@ try {
 }
 
 ?>
---EXPECT--
-string(73) "DOMDocument::validate(): supplied resource is not a valid stream resource"
+--EXPECTF--
+Warning: DOMDocument::validate(): The user entity loader callback 'Handler::handle' has returned a resource, but it is not a streamFailed to load external entity "-//FOO/BAR" in %s on line %d
+
+Warning: DOMDocument::validate(): Could not load the external subset "http://example.com/foobar" in %s on line %d
+bool(false)

ext/posix/posix.c

@@ -422,7 +422,8 @@ static zend_result php_posix_stream_get_fd(zval *zfp, zend_long *ret) /* {{{ */
 
 	php_stream_from_zval_no_verify(stream, zfp);
 
-	if (stream == NULL) {
+	if (UNEXPECTED(stream == NULL)) {
+		zend_argument_type_error(1, "must be an open stream resource");
 		return FAILURE;
 	}