fix: Make sure that if TSInteropMarshaller.Allocate fails, we get the…

… right exception
unoplatform · Sep 16, 2021 · ba663c0 · ba663c0
1 parent 0dfd158
commit ba663c0
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/src/Uno.Foundation.Runtime.WebAssembly/Interop/TSInteropMarshaller.wasm.cs b/src/Uno.Foundation.Runtime.WebAssembly/Interop/TSInteropMarshaller.wasm.cs
@@ -132,7 +132,13 @@ public static HandleRef<T> Allocate<T>(string propertySetterName, string? proper
 			}
 			catch (Exception e)
 			{
-				value.Dispose();
+				try
+				{
+					value.Dispose();
+				}
+				// If the allocation failed, the dispose will most likely also fail,
+				// but we want to propagate the real exception of the allocation!
+				catch (Exception) { }
 
 				if (_logger.Value.IsEnabled(LogLevel.Error))
 				{