avoid allocation X509Chain if there is no certificate (#74616)

a3960a81 · Tomas Weinfurt · GitHub · b27b2988 · a3960a81
隐藏空白更改
内联并排

Showing with 5 addition and 3 deletion

src/libraries/System.Net.Security/src/System/Net/CertificateValidationPal.Unix.cs ....Security/src/System/Net/CertificateValidationPal.Unix.cs +5 -3

未找到文件。
--- a/src/libraries/System.Net.Security/src/System/Net/CertificateValidationPal.Unix.cs
+++ b/src/libraries/System.Net.Security/src/System/Net/CertificateValidationPal.Unix.cs
@@ -43,12 +43,14 @@ internal static partial class CertificateValidationPal
            {
                QueryContextRemoteCertificate(securityContext, out remoteContext);

-                if (remoteContext != null && !remoteContext.IsInvalid)
+                if (remoteContext == null || remoteContext.IsInvalid)
                {
-                    remoteContext.DangerousAddRef(ref gotReference);
-                    result = new X509Certificate2(remoteContext.DangerousGetHandle());
+                    return null;
                }

+                remoteContext.DangerousAddRef(ref gotReference);
+                result = new X509Certificate2(remoteContext.DangerousGetHandle());
+
                if (retrieveChainCertificates)
                {
                    chain ??= new X509Chain();