diff --git a/SecureCoding.Test/SecureReflection/SecureAssemblyLoaderTests.cs b/SecureCoding.Test/SecureReflection/SecureAssemblyLoaderTests.cs index 1f20f0c..1708ddc 100644 --- a/SecureCoding.Test/SecureReflection/SecureAssemblyLoaderTests.cs +++ b/SecureCoding.Test/SecureReflection/SecureAssemblyLoaderTests.cs @@ -77,7 +77,7 @@ public void SecureAssemblyLoadFile_X509Certificate2Argument_Failure() Assert.ThrowsException(() => SecureAssembly.LoadFile(string.Empty, default(X509Certificate2))); Assert.ThrowsException(() => SecureAssembly.LoadFile("ValidString", new X509Certificate2[0])); - var selfSignedCertificate = new X509Certificate2(selfSignedCertificateCerPath); + var selfSignedCertificate = new X509Certificate2(selfSignedCertificatePfxPath, selfSignedCertificatePassword); // Signed with different Certificate test Assert.ThrowsException(() => SecureAssembly.LoadFrom(signedTestDllPath, selfSignedCertificate)); diff --git a/SecureCoding/SecureReflection/SecureAssembly.cs b/SecureCoding/SecureReflection/SecureAssembly.cs index 3c41560..fc3d5e0 100644 --- a/SecureCoding/SecureReflection/SecureAssembly.cs +++ b/SecureCoding/SecureReflection/SecureAssembly.cs @@ -627,8 +627,9 @@ private static bool TryValidateSignature(string fileName, bool verifyCertificate return result == 0; // 0 means signature is valid } + #region WinTrust -#pragma warning disable IDE0044 // Add readonly modifier +#pragma warning disable S2933 // Add readonly modifier /// /// Represents possible results returned by the WinVerifyTrust function, @@ -825,7 +826,7 @@ public void Dispose() } } - #pragma warning restore IDE0044 // Add readonly modifier +#pragma warning restore S2933 // Add readonly modifier #endregion } }