Implement {Try}HashData on asymmetric algorithms

Historically, the asymmetric algorithm base classes have had the hashing routines as abstract (or virtual+throw).

Now the base classes provide an implementation for these methods, reducing the amount of redundant effort on the part of derived types.

Author: vcsjones

src/libraries/Common/src/System/Security/Cryptography/DSACng.cs

@@ -9,7 +9,7 @@
 
 namespace System.Security.Cryptography
 {
-    public sealed partial class DSACng : DSA
+    public sealed partial class DSACng : DSA, IRuntimeAlgorithm
     {
         /// <summary>
         ///     Create a DSACng algorithm with a random 2048 bit key pair.
@@ -45,16 +45,6 @@ public override KeySizes[] LegalKeySizes
         public override string SignatureAlgorithm => "DSA";
         public override string? KeyExchangeAlgorithm => null;
 
-        // Need to override since base methods throw a "override me" exception: makes SignData/VerifyData function.
-        protected override byte[] HashData(byte[] data, int offset, int count, HashAlgorithmName hashAlgorithm) =>
-            HashOneShotHelpers.HashData(hashAlgorithm, new ReadOnlySpan<byte>(data, offset, count));
-
-        protected override byte[] HashData(Stream data, HashAlgorithmName hashAlgorithm) =>
-            HashOneShotHelpers.HashData(hashAlgorithm, data);
-
-        protected override bool TryHashData(ReadOnlySpan<byte> source, Span<byte> destination, HashAlgorithmName hashAlgorithm, out int bytesWritten) =>
-            HashOneShotHelpers.TryHashData(hashAlgorithm, source, destination, out bytesWritten);
-
         private void ForceSetKeySize(int newKeySize)
         {
             // Our LegalKeySizes value stores the values that we encoded as being the correct

src/libraries/Common/src/System/Security/Cryptography/DSAOpenSsl.cs

@@ -9,7 +9,7 @@
 
 namespace System.Security.Cryptography
 {
-    public sealed partial class DSAOpenSsl : DSA
+    public sealed partial class DSAOpenSsl : DSA, IRuntimeAlgorithm
     {
         // The biggest key allowed by FIPS 186-4 has N=256 (bit), which
         // maximally produces a 72-byte DER signature.
@@ -193,23 +193,6 @@ private SafeDsaHandle GenerateKey()
             return key;
         }
 
-        protected override byte[] HashData(byte[] data, int offset, int count, HashAlgorithmName hashAlgorithm)
-        {
-            // we're sealed and the base should have checked this already
-            Debug.Assert(data != null);
-            Debug.Assert(offset >= 0 && offset <= data.Length);
-            Debug.Assert(count >= 0 && count <= data.Length);
-            Debug.Assert(!string.IsNullOrEmpty(hashAlgorithm.Name));
-
-            return HashOneShotHelpers.HashData(hashAlgorithm, new ReadOnlySpan<byte>(data, offset, count));
-        }
-
-        protected override byte[] HashData(Stream data, HashAlgorithmName hashAlgorithm) =>
-            HashOneShotHelpers.HashData(hashAlgorithm, data);
-
-        protected override bool TryHashData(ReadOnlySpan<byte> data, Span<byte> destination, HashAlgorithmName hashAlgorithm, out int bytesWritten) =>
-            HashOneShotHelpers.TryHashData(hashAlgorithm, data, destination, out bytesWritten);
-
         public override byte[] CreateSignature(byte[] rgbHash!!)
         {
             SafeDsaHandle key = GetKey();

src/libraries/Common/src/System/Security/Cryptography/DSASecurityTransforms.cs

@@ -9,7 +9,7 @@ namespace System.Security.Cryptography
 {
     internal static partial class DSAImplementation
     {
-        public sealed partial class DSASecurityTransforms : DSA
+        public sealed partial class DSASecurityTransforms : DSA, IRuntimeAlgorithm
         {
             private SecKeyPair? _keys;
             private bool _disposed;
@@ -118,12 +118,6 @@ protected override byte[] HashData(byte[] data, int offset, int count, HashAlgor
                 return HashOneShotHelpers.HashData(hashAlgorithm, new ReadOnlySpan<byte>(data, offset, count));
             }
 
-            protected override byte[] HashData(Stream data, HashAlgorithmName hashAlgorithm) =>
-                HashOneShotHelpers.HashData(hashAlgorithm, data);
-
-            protected override bool TryHashData(ReadOnlySpan<byte> data, Span<byte> destination, HashAlgorithmName hashAlgorithm, out int bytesWritten) =>
-                HashOneShotHelpers.TryHashData(hashAlgorithm, data, destination, out bytesWritten);
-
             protected override void Dispose(bool disposing)
             {
                 if (disposing)

src/libraries/Common/src/System/Security/Cryptography/ECDsaAndroid.cs

@@ -10,7 +10,7 @@ namespace System.Security.Cryptography
 {
     internal static partial class ECDsaImplementation
     {
-        public sealed partial class ECDsaAndroid : ECDsa
+        public sealed partial class ECDsaAndroid : ECDsa, IRuntimeAlgorithm
         {
             // secp521r1 maxes out at 139 bytes, so 256 should always be enough
             private const int SignatureStackBufSize = 256;
@@ -240,15 +240,6 @@ protected override bool VerifyHashCore(
                 return verifyResult == 1;
             }
 
-            protected override byte[] HashData(byte[] data, int offset, int count, HashAlgorithmName hashAlgorithm) =>
-                HashOneShotHelpers.HashData(hashAlgorithm, new ReadOnlySpan<byte>(data, offset, count));
-
-            protected override byte[] HashData(Stream data, HashAlgorithmName hashAlgorithm) =>
-                HashOneShotHelpers.HashData(hashAlgorithm, data);
-
-            protected override bool TryHashData(ReadOnlySpan<byte> data, Span<byte> destination, HashAlgorithmName hashAlgorithm, out int bytesWritten) =>
-                HashOneShotHelpers.TryHashData(hashAlgorithm, data, destination, out bytesWritten);
-
             protected override void Dispose(bool disposing)
             {
                 if (disposing)

src/libraries/Common/src/System/Security/Cryptography/ECDsaCng.HashData.cs

@@ -5,7 +5,7 @@
 
 namespace System.Security.Cryptography
 {
-    public sealed partial class ECDsaCng : ECDsa
+    public sealed partial class ECDsaCng : ECDsa, IRuntimeAlgorithm
     {
         /// <summary>
         /// Create an ECDsaCng algorithm with a named curve.

src/libraries/Common/src/System/Security/Cryptography/ECDsaCng.cs

@@ -9,7 +9,7 @@
 
 namespace System.Security.Cryptography
 {
-    public sealed partial class ECDsaOpenSsl : ECDsa
+    public sealed partial class ECDsaOpenSsl : ECDsa, IRuntimeAlgorithm
     {
         // secp521r1 maxes out at 139 bytes, so 256 should always be enough
         private const int SignatureStackBufSize = 256;
@@ -250,15 +250,6 @@ protected override bool VerifyHashCore(
             return verifyResult == 1;
         }
 
-        protected override byte[] HashData(byte[] data, int offset, int count, HashAlgorithmName hashAlgorithm) =>
-            HashOneShotHelpers.HashData(hashAlgorithm, new ReadOnlySpan<byte>(data, offset, count));
-
-        protected override byte[] HashData(Stream data, HashAlgorithmName hashAlgorithm) =>
-            HashOneShotHelpers.HashData(hashAlgorithm, data);
-
-        protected override bool TryHashData(ReadOnlySpan<byte> data, Span<byte> destination, HashAlgorithmName hashAlgorithm, out int bytesWritten) =>
-            HashOneShotHelpers.TryHashData(hashAlgorithm, data, destination, out bytesWritten);
-
         protected override void Dispose(bool disposing)
         {
             if (disposing)

src/libraries/Common/src/System/Security/Cryptography/ECDsaOpenSsl.cs

@@ -9,7 +9,7 @@ namespace System.Security.Cryptography
 {
     internal static partial class ECDsaImplementation
     {
-        public sealed partial class ECDsaSecurityTransforms : ECDsa
+        public sealed partial class ECDsaSecurityTransforms : ECDsa, IRuntimeAlgorithm
         {
             private readonly EccSecurityTransforms _ecc = new EccSecurityTransforms(nameof(ECDsa));
 
@@ -135,15 +135,6 @@ public override bool VerifyHash(ReadOnlySpan<byte> hash, ReadOnlySpan<byte> sign
                     Interop.AppleCrypto.PAL_SignatureAlgorithm.EC);
             }
 
-            protected override byte[] HashData(byte[] data, int offset, int count, HashAlgorithmName hashAlgorithm) =>
-                HashOneShotHelpers.HashData(hashAlgorithm, new ReadOnlySpan<byte>(data, offset, count));
-
-            protected override byte[] HashData(Stream data, HashAlgorithmName hashAlgorithm) =>
-                HashOneShotHelpers.HashData(hashAlgorithm, data);
-
-            protected override bool TryHashData(ReadOnlySpan<byte> source, Span<byte> destination, HashAlgorithmName hashAlgorithm, out int bytesWritten) =>
-                HashOneShotHelpers.TryHashData(hashAlgorithm, source, destination, out bytesWritten);
-
             private void ThrowIfDisposed()
             {
                 _ecc.ThrowIfDisposed();

src/libraries/Common/src/System/Security/Cryptography/ECDsaSecurityTransforms.cs

@@ -0,0 +1,14 @@
+// Licensed to the .NET Foundation under one or more agreements.
+// The .NET Foundation licenses this file to you under the MIT license.
+
+using System.Diagnostics;
+using System.Formats.Asn1;
+using System.Numerics;
+
+namespace System.Security.Cryptography
+{
+    // Marker interface that goes on runtime-provided algorithms.
+    internal interface IRuntimeAlgorithm
+    {
+    }
+}

src/libraries/Common/src/System/Security/Cryptography/IRuntimeAlgorithm.cs

@@ -13,7 +13,7 @@ namespace System.Security.Cryptography
 {
     internal static partial class RSAImplementation
     {
-        public sealed partial class RSAAndroid : RSA
+        public sealed partial class RSAAndroid : RSA, IRuntimeAlgorithm
         {
             private const int BitsPerByte = 8;
 
@@ -647,15 +647,6 @@ private SafeRsaHandle GenerateKey()
                 return key;
             }
 
-            protected override byte[] HashData(byte[] data, int offset, int count, HashAlgorithmName hashAlgorithm) =>
-                HashOneShotHelpers.HashData(hashAlgorithm, new ReadOnlySpan<byte>(data, offset, count));
-
-            protected override byte[] HashData(Stream data, HashAlgorithmName hashAlgorithm) =>
-                HashOneShotHelpers.HashData(hashAlgorithm, data);
-
-            protected override bool TryHashData(ReadOnlySpan<byte> data, Span<byte> destination, HashAlgorithmName hashAlgorithm, out int bytesWritten) =>
-                HashOneShotHelpers.TryHashData(hashAlgorithm, data, destination, out bytesWritten);
-
             public override byte[] SignHash(byte[] hash, HashAlgorithmName hashAlgorithm, RSASignaturePadding padding)
             {
                 ArgumentNullException.ThrowIfNull(hash);