changes from feedback

Author: devnexen

Zend/zend_hash.h

@@ -853,13 +853,18 @@ static zend_always_inline void *zend_hash_next_index_insert_mem(HashTable *ht, v
 {
 	zval tmp, *zv;
 
-	ZVAL_PTR(&tmp, NULL);
-	if ((zv = zend_hash_next_index_insert(ht, &tmp))) {
-		Z_PTR_P(zv) = pemalloc(size, GC_FLAGS(ht) & IS_ARRAY_PERSISTENT);
-		memcpy(Z_PTR_P(zv), pData, size);
-		return Z_PTR_P(zv);
+	void *p = pemalloc(size, GC_FLAGS(ht) & IS_ARRAY_PERSISTENT);
+	if (!p) {
+		return NULL;
 	}
-	return NULL;
+
+	memcpy(p, pData, size);
+	ZVAL_PTR(&tmp, p);
+	if (!(zv = zend_hash_next_index_insert(ht, &tmp))) {
+		pefree(p, GC_FLAGS(ht) & IS_ARRAY_PERSISTENT);
+		return NULL;
+	}
+	return Z_PTR_P(zv);
 }
 
 static zend_always_inline void *zend_hash_find_ptr(const HashTable *ht, zend_string *key)

ext/standard/basic_functions.c

@@ -1582,12 +1582,9 @@ static void fci_release(zend_fcall_info *fci, zend_fcall_info_cache *fci_cache)
 void user_shutdown_function_dtor(zval *zv) /* {{{ */
 {
 	php_shutdown_function_entry *shutdown_function_entry = Z_PTR_P(zv);
-	zend_fcall_info *fci = &shutdown_function_entry->fci;
 
-	if (fci) {
-		zend_fcall_info_args_clear(fci, true);
-		fci_release(fci, &shutdown_function_entry->fci_cache);
-	}
+	zend_fcall_info_args_clear(&shutdown_function_entry->fci, true);
+	fci_release(&shutdown_function_entry->fci, &shutdown_function_entry->fci_cache);
 	efree(shutdown_function_entry);
 }
 /* }}} */