chore: wrap GetFromRuntimeInformation() in try-catch (#1554)

getsentry · Mar 26, 2022 · 1cdde37 · 1cdde37
1 parent e3ca414
commit 1cdde37
Showing 1 changed file with 14 additions and 5 deletions.
diff --git a/src/Sentry/PlatformAbstractions/RuntimeInfo.cs b/src/Sentry/PlatformAbstractions/RuntimeInfo.cs
@@ -102,11 +102,20 @@ internal static void SetNetCoreVersion(Runtime runtime)
 
         internal static Runtime? GetFromRuntimeInformation()
         {
-            // Preferred API: netstandard2.0
-            // https://github.com/dotnet/corefx/blob/master/src/System.Runtime.InteropServices.RuntimeInformation/src/System/Runtime/InteropServices/RuntimeInformation/RuntimeInformation.cs
-            // https://github.com/mono/mono/blob/90b49aa3aebb594e0409341f9dca63b74f9df52e/mcs/class/corlib/System.Runtime.InteropServices.RuntimeInformation/RuntimeInformation.cs
-            // e.g: .NET Framework 4.7.2633.0, .NET Native, WebAssembly
-            var frameworkDescription = RuntimeInformation.FrameworkDescription;
+            string frameworkDescription;
+            try
+            {
+                // Preferred API: netstandard2.0
+                // https://github.com/dotnet/corefx/blob/master/src/System.Runtime.InteropServices.RuntimeInformation/src/System/Runtime/InteropServices/RuntimeInformation/RuntimeInformation.cs
+                // https://github.com/mono/mono/blob/90b49aa3aebb594e0409341f9dca63b74f9df52e/mcs/class/corlib/System.Runtime.InteropServices.RuntimeInformation/RuntimeInformation.cs
+                // e.g: .NET Framework 4.7.2633.0, .NET Native, WebAssembly
+                // Note: this throws on some Unity IL2CPP versions
+                frameworkDescription = RuntimeInformation.FrameworkDescription;
+            }
+            catch
+            {
+                return null;
+            }
 
             return Parse(frameworkDescription);
         }