From f267a00525ba63bab3d2ce24a6e1cd5846b330cb Mon Sep 17 00:00:00 2001 From: Kirill Osenkov Date: Thu, 17 Feb 2022 11:13:19 -0800 Subject: [PATCH] Don't return null Jdk locations VSMac has a first-chance exception because we're returning null here for some attributes --- .../Jdks/JdkLocations.MacOS.cs | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/Xamarin.Android.Tools.AndroidSdk/Jdks/JdkLocations.MacOS.cs b/src/Xamarin.Android.Tools.AndroidSdk/Jdks/JdkLocations.MacOS.cs index dc2a8f5..cd62323 100644 --- a/src/Xamarin.Android.Tools.AndroidSdk/Jdks/JdkLocations.MacOS.cs +++ b/src/Xamarin.Android.Tools.AndroidSdk/Jdks/JdkLocations.MacOS.cs @@ -21,8 +21,10 @@ static IEnumerable GetUnixConfiguredJdkPaths (Action { var config = AndroidSdkUnix.GetUnixConfigFile (logger); foreach (var java_sdk in config.Root.Elements ("java-sdk")) { - var path = (string) java_sdk.Attribute ("path"); - yield return path; + var path = (string) java_sdk.Attribute ("path"); + if (path != null) { + yield return path; + } } }