Fix the tests.

mono · Aug 14, 2019 · 29b1ac19c961b959a09097dbc0fe4cd567cc5298 · 29b1ac1
1 parent 71e34ed
commit 29b1ac19c961b959a09097dbc0fe4cd567cc5298
Unified Split

Showing with 6 additions and 3 deletions.

+6 −3 mcs/class/System/Mono.Btls/X509CertificateImplBtls.cs
diff --git a/mcs/class/System/Mono.Btls/X509CertificateImplBtls.cs b/mcs/class/System/Mono.Btls/X509CertificateImplBtls.cs
@@ -198,11 +198,14 @@ protected override void Dispose (bool disposing)
 			set {
 				if (nativePrivateKey != null)
 					nativePrivateKey.Dispose ();
-				if (value == null) {
+				try {
+					// FIXME: there doesn't seem to be a public API to check whether it actually
+					//        contains a private key (apart from RSAManaged.PublicOnly).
+					if (value != null)
+						nativePrivateKey = MonoBtlsKey.CreateFromRSAPrivateKey ((RSA)value);
+				} catch {
 					nativePrivateKey = null;
-					return;
 				}
-				nativePrivateKey = MonoBtlsKey.CreateFromRSAPrivateKey ((RSA)value);
 			}
 		}