Search code examples
asp.net-core.net-corersacode-signingmultiplatform

RSA and SignedCms on ASP .Net Core; multiplatform approach needed


I am trying to find a way to do certificate signature of an APK SF file, in a way that works multiplatform. Without success at the minute; exaplanation of what I am doing.

Signing ( common for both )

I am doing a signature with issuer and serial number with:

private static byte[] GetSigned(byte[] sfData, X509Certificate cert){
   X509Certificate2 certificate = new X509Certificate2(cert.GetEncoded());
   RSA rsaPriv = Certificate.ToRSA(cert.KeyPair.Private as RsaPrivateCrtKeyParameters);
   X509Certificate2 certWithKey = RSACertificateExtensions.CopyWithPrivateKey(certificate, rsaPriv);

   ContentInfo content = new ContentInfo(sfData);
   SignedCms signedCms = new SignedCms(content, true);
   CmsSigner signer = new CmsSigner(SubjectIdentifierType.IssuerAndSerialNumber, certWithKey);
   signedCms.ComputeSignature(signer);
   return signedCms.Encode();
}

Windows solution

public static RSA ToRSA(RsaPrivateCrtKeyParameters privKey)
        {
            return CreateRSAProvider(ToRSAParameters(privKey));
        }

        private static RSA CreateRSAProvider(RSAParameters rp)
        {
            CspParameters csp = new CspParameters
            {
                KeyContainerName = string.Format("BouncyCastle-{0}", Guid.NewGuid()),
                Flags = CspProviderFlags.UseMachineKeyStore
            };

            // This is a workaround to fallback to user keystore while not machine is available;
            // as otherwise it's impossible having something working on Azure and locally.
            // It's more a bug of this cryptography stuff on ASP .Net core..
            RSACryptoServiceProvider rsaCsp;
            try
            {
                rsaCsp = new RSACryptoServiceProvider(csp);
            }catch(Exception ex)
            {
                csp.Flags = CspProviderFlags.NoFlags;
                rsaCsp = new RSACryptoServiceProvider(csp);
            }

            rsaCsp.ImportParameters(rp);
            return rsaCsp;
        }

        private static RSAParameters ToRSAParameters(RsaPrivateCrtKeyParameters privKey)
        {
            RSAParameters rp = new RSAParameters();
            rp.Modulus = privKey.Modulus.ToByteArrayUnsigned();
            rp.Exponent = privKey.PublicExponent.ToByteArrayUnsigned();
            rp.P = privKey.P.ToByteArrayUnsigned();
            rp.Q = privKey.Q.ToByteArrayUnsigned();
            rp.D = ConvertRSAParametersField(privKey.Exponent, rp.Modulus.Length);
            rp.DP = ConvertRSAParametersField(privKey.DP, rp.P.Length);
            rp.DQ = ConvertRSAParametersField(privKey.DQ, rp.Q.Length);
            rp.InverseQ = ConvertRSAParametersField(privKey.QInv, rp.Q.Length);
            return rp;
        }

Linux solution

Following this stackoverflow answer, I've used RSA; like:

public static RSA ToRSA(RsaPrivateCrtKeyParameters privKey)
        {
            var rp = ToRSAParameters(privKey);
            return RSA.Create(rp);
        }

While using Linux solution on Windows, I got the following exception

[31/01/2019 16:26:09 Informational] [xUnit.net 00:00:03.81]       System.ArgumentException : The CNG key handle being opened was detected to be ephemeral, but the EphemeralKey open option was not specified.
[31/01/2019 16:26:09 Informational] [xUnit.net 00:00:03.81]       Parameter name: keyHandleOpenOptions
[31/01/2019 16:26:09 Informational] [xUnit.net 00:00:03.81]       Stack Trace:
[31/01/2019 16:26:09 Informational] [xUnit.net 00:00:03.81]            at System.Security.Cryptography.CngKey.Open(SafeNCryptKeyHandle keyHandle, CngKeyHandleOpenOptions keyHandleOpenOptions)
[31/01/2019 16:26:09 Informational] [xUnit.net 00:00:03.81]            at Internal.Cryptography.Pal.Windows.PkcsPalWindows.GetPrivateKey[T](X509Certificate2 certificate, Boolean silent, Boolean preferNCrypt)
[31/01/2019 16:26:09 Informational] [xUnit.net 00:00:03.81]            at Internal.Cryptography.Pal.Windows.PkcsPalWindows.GetPrivateKeyForSigning[T](X509Certificate2 certificate, Boolean silent)
[31/01/2019 16:26:09 Informational] [xUnit.net 00:00:03.81]            at System.Security.Cryptography.Pkcs.CmsSignature.RSAPkcs1CmsSignature.Sign(ReadOnlySpan`1 dataHash, HashAlgorithmName hashAlgorithmName, X509Certificate2 certificate, Boolean silent, Oid& signatureAlgorithm, Byte[]& signatureValue)
[31/01/2019 16:26:09 Informational] [xUnit.net 00:00:03.81]            at System.Security.Cryptography.Pkcs.CmsSignature.Sign(ReadOnlySpan`1 dataHash, HashAlgorithmName hashAlgorithmName, X509Certificate2 certificate, Boolean silent, Oid& oid, ReadOnlyMemory`1& signatureValue)
[31/01/2019 16:26:09 Informational] [xUnit.net 00:00:03.81]            at System.Security.Cryptography.Pkcs.CmsSigner.Sign(ReadOnlyMemory`1 data, String contentTypeOid, Boolean silent, X509Certificate2Collection& chainCerts)
[31/01/2019 16:26:09 Informational] [xUnit.net 00:00:03.81]            at System.Security.Cryptography.Pkcs.SignedCms.ComputeSignature(CmsSigner signer, Boolean silent)
[31/01/2019 16:26:09 Informational] [xUnit.net 00:00:03.81]         C:\workspace\kleidi\Kleidi\Signing\APK\ApkSigner.cs(176,0): at Kleidi.Signing.APK.ApkSigner.GetRSAData(ZipFile zip, Byte[] sfData, String rsaName, Bundle cert)
[31/01/2019 16:26:09 Informational] [xUnit.net 00:00:03.81]         C:\workspace\kleidi\Kleidi\Signing\APK\ApkSigner.cs(58,0): at Kleidi.Signing.APK.ApkSigner.Sign(Stream srcApkStream, Stream& dstApkStream, Bundle certBundle, String sharedKey, String generationId)
[31/01/2019 16:26:09 Informational] [xUnit.net 00:00:03.81]         C:\workspace\kleidi\KleidiTests\Signing\APK\ApkSignerTest.cs(23,0): at Kleidi.Signing.APK.ApkSignerTest.Valid_signature_and_certificate_match()

Question is..

Anyone knows a way that works for both? or should I infere the operative system is building me and then use one or other ? (sounds ugly)


Solution

  • It looks like the SignedCms class is having trouble opening the key that was manipulated via CopyWithPrivateKey. The quick-and-dirty workaround for anything weird happening in that situation is to make a temporary PFX export and reimport it, so replace

    X509Certificate2 certWithKey = RSACertificateExtensions.CopyWithPrivateKey(certificate, rsaPriv);
    

    with something like

    using (X509Certificate2 temp = certificate.CopyWithPrivateKey(rsaPriv))
    using (X509Certificate2 certWithKey = new X509Certificate2(temp.Export(X509ContentType.Pfx))
    {
        // Build CmsSigner and call ComputeSignature here.
    }
    

    FWIW: This is now a CoreFX bug: https://github.com/dotnet/corefx/issues/35120.