From 4273aac93139ac1ca8cb40e6ffa8857752904c7a Mon Sep 17 00:00:00 2001
From: Timotheus Pokorra <timotheus.pokorra@solidcharity.com>
Date: Feb 22 2021 21:15:46 +0000
Subject: backport patch: fix early return in Process.Start (#1839410)


---

diff --git a/mono-6.12.0-fix-process-start.patch b/mono-6.12.0-fix-process-start.patch
new file mode 100644
index 0000000..4b9e02c
--- /dev/null
+++ b/mono-6.12.0-fix-process-start.patch
@@ -0,0 +1,42 @@
+From 325577feffad1f87b07d92710c0d37c79c5f50a1 Mon Sep 17 00:00:00 2001
+From: Ryan Lucia <rylucia@microsoft.com>
+Date: Thu, 4 Feb 2021 08:55:48 -0500
+Subject: [PATCH] Fix early return in Process.Start
+
+---
+ mono/metadata/w32process-unix.c | 7 +++++--
+ 1 file changed, 5 insertions(+), 2 deletions(-)
+
+diff --git a/mono/metadata/w32process-unix.c b/mono/metadata/w32process-unix.c
+index 663cc6ff5111..d030101aede6 100644
+--- a/mono/metadata/w32process-unix.c
++++ b/mono/metadata/w32process-unix.c
+@@ -2032,6 +2032,9 @@ ves_icall_System_Diagnostics_Process_ShellExecuteEx_internal (MonoW32ProcessStar
+ 
+ 		static char *handler;
+ 		static gunichar2 *handler_utf16;
++#ifndef HOST_DARWIN
++		gboolean finished = FALSE;
++#endif
+ 
+ 		if (handler_utf16 == (gunichar2 *)-1) {
+ 			ret = FALSE;
+@@ -2057,6 +2060,7 @@ ves_icall_System_Diagnostics_Process_ShellExecuteEx_internal (MonoW32ProcessStar
+ 				if (handler == NULL){
+ 					handler_utf16 = (gunichar2 *) -1;
+ 					ret = FALSE;
++					finished = TRUE;
+ 				} else {
+ 					/* kfmclient needs exec argument */
+ 					char *old = handler;
+@@ -2067,9 +2071,8 @@ ves_icall_System_Diagnostics_Process_ShellExecuteEx_internal (MonoW32ProcessStar
+ 			}
+ 		}
+ 		MONO_EXIT_GC_SAFE;
+-		if (ret == FALSE){
++		if (finished)
+ 			goto done;
+-		}
+ #endif
+ 		handler_utf16 = g_utf8_to_utf16 (handler, -1, NULL, NULL, NULL);
+ 		g_free (handler);
diff --git a/mono.spec b/mono.spec
index 7d787ec..340ae47 100644
--- a/mono.spec
+++ b/mono.spec
@@ -24,7 +24,7 @@
 %global xamarinrelease 107
 Name:           mono
 Version:        6.12.0
-Release:        2%{?dist}
+Release:        3%{?dist}
 Summary:        Cross-platform, Open Source, .NET development framework
 
 License:        MIT
@@ -58,6 +58,8 @@ Patch11:        0001-Replace-new-Csharp-features-with-old-ones.patch
 Patch12:        0001-Reenable-mdoc.exe-build.patch
 # fix issue with conflicts between i686 and x86_64 package (#1853724)
 Patch13:	mono-6.6.0-fix-multi-arch-issue.patch
+# fix issue with process start (#1839410)
+Patch14:	mono-6.12.0-fix-process-start.patch
 
 BuildRequires: make
 BuildRequires:  bison
@@ -349,6 +351,7 @@ pushd external/api-doc-tools
 popd
 %patch12 -p1
 %patch13 -p1
+%patch14 -p1
 
 # don't build mono-helix-client which requires the helix-binaries to build
 sed -i 's|mono-helix-client||g' mcs/tools/Makefile
@@ -941,6 +944,9 @@ cert-sync /etc/pki/tls/certs/ca-bundle.crt
 %files complete
 
 %changelog
+* Mon Feb 22 2021 Timotheus Pokorra <timotheus.pokorra@solidcharity.com> - 6.12.0-3
+- backport patch: fix early return in Process.Start (#1839410)
+
 * Tue Jan 26 2021 Fedora Release Engineering <releng@fedoraproject.org> - 6.12.0-2
 - Rebuilt for https://fedoraproject.org/wiki/Fedora_34_Mass_Rebuild