Appease mono

Author: Rob-Hague

.editorconfig

@@ -1,4 +1,4 @@
-// Avoid looking for .editorconfig files in parent directories
+// Avoid looking for .editorconfig files in parent directories
 root=true
 
 [*]
@@ -87,6 +87,10 @@ dotnet_diagnostic.S1144.severity = none
 # This is a duplicate of IDE0060.
 dotnet_diagnostic.S1172.severity = none
 
+# S1199: Nested code blocks should not be used
+# https://rules.sonarsource.com/csharp/RSPEC-1199
+dotnet_diagnostic.S1199.severity = none
+
 # S1481: Unused local variables should be removed
 # https://rules.sonarsource.com/csharp/RSPEC-1481
 #

src/Renci.SshNet/Security/Cryptography/DsaKey.cs

@@ -158,24 +158,32 @@ public DsaKey(BigInteger p, BigInteger q, BigInteger g, BigInteger y, BigInteger
             DSA = LoadDSA();
         }
 
-#pragma warning disable CA1859 // Use concrete types when possible for improved performance
 #pragma warning disable CA5384 // Do Not Use Digital Signature Algorithm (DSA)
         private DSA LoadDSA()
         {
+            DSA dsa;
+
 #if NETFRAMEWORK
             // On .NET Framework we use the concrete CNG type which is FIPS-186-3
             // compatible. The CryptoServiceProvider type returned by DSA.Create()
             // is limited to FIPS-186-1 (max 1024 bit key).
-            var dsa = new DSACng();
-#else
-            var dsa = DSA.Create();
+
+            // We give some consideration to a mono build on non-Windows.
+            if (Environment.OSVersion.Platform == PlatformID.Win32NT)
+            {
+                dsa = new DSACng();
+            }
+            else
 #endif
+            {
+                dsa = DSA.Create();
+            }
+
             dsa.ImportParameters(GetDSAParameters());
 
             return dsa;
         }
 #pragma warning restore CA5384 // Do Not Use Digital Signature Algorithm (DSA)
-#pragma warning restore CA1859 // Use concrete types when possible for improved performance
 
         internal DSAParameters GetDSAParameters()
         {