changes from feedback

we do not set curl (function) user handler if there is no actual
callback.

Author: devnexen

Zend/zend_API.h

@@ -843,12 +843,6 @@ static zend_always_inline void zend_call_known_fcc(
 	const zend_fcall_info_cache *fcc, zval *retval_ptr, uint32_t param_count, zval *params, HashTable *named_params)
 {
 	zend_function *func = fcc->function_handler;
-	if (UNEXPECTED(!func)) {
-		if (retval_ptr) {
-			ZVAL_UNDEF(retval_ptr);
-		}
-		return;
-	}
 	/* Need to copy trampolines as they get released after they are called */
 	if (UNEXPECTED(func->common.fn_flags & ZEND_ACC_CALL_VIA_TRAMPOLINE)) {
 		func = (zend_function*) emalloc(sizeof(zend_function));

ext/curl/interface.c

@@ -1634,7 +1634,16 @@ static bool php_curl_set_callable_handler(zend_fcall_info_cache *const handler_f
 #define HANDLE_CURL_OPTION_CALLABLE_PHP_CURL_USER(curl_ptr, constant_no_function, handler_type) \
 	case constant_no_function##FUNCTION: { \
 		bool result = php_curl_set_callable_handler(&curl_ptr->handlers.handler_type->fcc, zvalue, is_array_config, #constant_no_function "FUNCTION"); \
-		if (!result) { \
+		if (!result || (curl_ptr->handlers.handler_type && !curl_ptr->handlers.handler_type->fcc.function_handler)) { \
+			if (curl_ptr->handlers.read) { \
+                        	curl_ptr->handlers.read->method = PHP_CURL_STDOUT; \
+			} \
+			if (curl_ptr->handlers.write) { \
+                        	curl_ptr->handlers.write->method = PHP_CURL_DIRECT; \
+			} \
+			if (curl_ptr->handlers.write_header) { \
+                        	curl_ptr->handlers.write_header->method = PHP_CURL_IGNORE; \
+			} \
 			return FAILURE; \
 		} \
 		curl_ptr->handlers.handler_type->method = PHP_CURL_USER; \