Revert "ext/phar: use zend_string* and concat API for constructor argument"

This reverts commit 131f98a.

Author: Girgias

ext/phar/phar_object.c

@@ -1162,29 +1162,20 @@ PHP_METHOD(Phar, __construct)
 	phar_obj->archive = phar_data;
 	phar_obj->spl.oth_handler = &phar_spl_foreign_handler;
 
-	zend_string *file_name_for_recursive_director_iterator_constructor;
 	if (entry) {
-		file_name_for_recursive_director_iterator_constructor = zend_string_concat3(
-			ZEND_STRL("phar://"),
-			phar_data->fname, phar_data->fname_len,
-			ZSTR_VAL(entry), ZSTR_LEN(entry)
-		);
+		fname_len = spprintf(&fname, 0, "phar://%s%s", phar_data->fname, ZSTR_VAL(entry));
 		zend_string_release_ex(entry, false);
 	} else {
-		file_name_for_recursive_director_iterator_constructor = zend_string_concat2(
-			ZEND_STRL("phar://"),
-			phar_data->fname, phar_data->fname_len
-		);
+		fname_len = spprintf(&fname, 0, "phar://%s", phar_data->fname);
 	}
 
-	ZVAL_STR(&arg1, file_name_for_recursive_director_iterator_constructor);
+	ZVAL_STRINGL(&arg1, fname, fname_len);
 	ZVAL_LONG(&arg2, flags);
 
 	zend_call_known_instance_method_with_2_params(spl_ce_RecursiveDirectoryIterator->constructor,
 		Z_OBJ_P(ZEND_THIS), NULL, &arg1, &arg2);
 
-	/* Freeing arg1 */
-	zend_string_release_ex(file_name_for_recursive_director_iterator_constructor, false);
+	zval_ptr_dtor(&arg1);
 
 	if (!phar_data->is_persistent) {
 		phar_obj->archive->is_data = is_data;
@@ -1194,6 +1185,7 @@ PHP_METHOD(Phar, __construct)
 	}
 
 	phar_obj->spl.info_class = phar_ce_entry;
+	efree(fname);
 }
 /* }}} */