System.Security.Cryptography.Rsa.Tests.RSAXml Test Suite Fails with Strings Differ/Containers Differ #117665

Activity

1. mdh1418

   added

   blocking-clean-ciBlocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms'Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms'

   Known Build ErrorUse this to report build issues in the .NET Helix tabUse this to report build issues in the .NET Helix tab

   on Jul 15, 2025
2. dotnet-policy-service

   added

   untriagedNew issue has not been triaged by the area ownerNew issue has not been triaged by the area owner

   on Jul 15, 2025
3. github-actions

   added

   area-System.Security

   on Jul 15, 2025
4. dotnet-policy-service commented on Jul 15, 2025

   

   dotnet-policy-serviceBot

   on Jul 15, 2025 – with Dotnet Policy Service

   Contributor

   More actions

   Tagging subscribers to this area: @dotnet/area-system-security, @bartonjs, @vcsjones
   See info in area-owners.md if you want to be subscribed.
5. vcsjones commented on Jul 15, 2025

   

   vcsjones

   on Jul 15, 2025

   Member

   More actions

   Posting (one of) the failures so others don't need to go log spelunking.

   Either way, it seems like RSA keys are being serialized differently. This was on mcr.microsoft.com/dotnet-buildtools/prereqs:alpine-3.22-helix-amd64.

   ystem.Security.Cryptography.Rsa.Tests.RSAXml.TestWriteUnusualExponentParameters(includePrivateParameters: True) [FAIL]
         Assert.Equal() Failure: Strings differ
                     ↓ (pos 1)
         Expected: "9rqCgyYMOZEbi12xig/zanjVWagNZCk90Aw1h1YAm"···
         Actual:   "9AqAgAYAOAEAiA2AiA/AaAjAWAgAZAkA0AwAhAYAm"···
                     ↑ (pos 1)
         Stack Trace:
           /_/src/libraries/Common/tests/System/Security/Cryptography/AlgorithmImplementations/RSA/RSAXml.cs(1459,0): at System.Security.Cryptography.Rsa.Tests.RSAXml.AssertNextElement(IEnumerator`1 iter, String localName, String expectedValue, String alternateValue)
           /_/src/libraries/Common/tests/System/Security/Cryptography/AlgorithmImplementations/RSA/RSAXml.cs(1405,0): at System.Security.Cryptography.Rsa.Tests.RSAXml.TestWriteXml(RSAParameters& keyParameters, Boolean includePrivateParameters, String expectedModulus, String expectedExponent, String expectedD, String expectedP, String expectedQ, String expectedDP, String expectedDQ, String expectedInverseQ, String alternateD)
           /_/src/libraries/Common/tests/System/Security/Cryptography/AlgorithmImplementations/RSA/RSAXml.cs(942,0): at System.Security.Cryptography.Rsa.Tests.RSAXml.TestWriteUnusualExponentParameters(Boolean includePrivateParameters)
              at System.RuntimeMethodHandle.InvokeMethod(ObjectHandleOnStack target, Void** arguments, ObjectHandleOnStack sig, BOOL isConstructor, ObjectHandleOnStack result)
              at System.RuntimeMethodHandle.InvokeMethod(ObjectHandleOnStack target, Void** arguments, ObjectHandleOnStack sig, BOOL isConstructor, ObjectHandleOnStack result)
           /_/src/coreclr/System.Private.CoreLib/src/System/Reflection/MethodBaseInvoker.CoreCLR.cs(36,0): at System.Reflection.MethodBaseInvoker.InterpretedInvoke_Method(Object obj, IntPtr* args)
           /_/src/libraries/System.Private.CoreLib/src/System/Reflection/MethodBaseInvoker.cs(174,0): at System.Reflection.MethodBaseInvoker.InvokeDirectByRefWithFewArgs(Object obj, Span`1 copyOfArgs, BindingFlags invokeAttr)
       System.Security.Cryptography.Rsa.Tests.RSAXml.TestWriteUnusualExponentParameters(includePrivateParameters: False) [FAIL]
   

   👍React with 👍1Reacted by Mitchell Hwang
6. bartonjs commented on Jul 15, 2025

   

   bartonjs

   on Jul 15, 2025

   Member

   More actions

   That seems... bad?

   -f6 ba 82 83 26 0c 39 91 1b 8b 5d b1 8a 0f f3 6a 78 d5 59 a8 0d 64 29 3d d0 0c 35 87 56 00 ...
   +f4 0a 80 80 06 00 38 01 00 88 0d 80 88 0f c0 68 08 c0 58 08 00 64 09 00 d0 0c 00 84 06 00 ...

   It's like something is clearing every other 6 bits?

   • F4BA XOR F40A is 02B0: 0b0000_0010_1011_0000
   • 8B5D XOR 880D is 0350: 0b0000_0011_0101_0000
7. vcsjones commented on Jul 15, 2025

   

   vcsjones

   on Jul 15, 2025

   Member

   More actions

   @bartonjs Since base64 is a 6-bit map... I wonder if this is a base64 encoding problem.
8. bartonjs commented on Jul 15, 2025

   

   bartonjs

   on Jul 15, 2025

   Member

   More actions

   And here I was wondering what sort of data mutation test we had that was accidentally messing up an RSAParameter. Something bad happening in Base64 seems better aligned to the problem... and matches that every other character in the Actual is A (000000)
9. vcsjones commented on Jul 18, 2025

   

   vcsjones

   on Jul 18, 2025

   Member

   More actions

   Same thing in other runs:

   Expected: "myxwX6kQNx+LSMao1StC1p5rKCEwcBjzI136An3B/"···
   Actual:   "mAxAXAkANA+ASAaA1AtA1A5AKAEAcAjAIA3AAA3A/"···
   
10. vcsjones

    added

    area-System.Runtime

    and removed

    area-System.Security

    on Jul 18, 2025
11. dotnet-policy-service commented on Jul 18, 2025

    

    dotnet-policy-serviceBot

    on Jul 18, 2025 – with Dotnet Policy Service

    Contributor

    More actions

    Tagging subscribers to this area: @dotnet/area-system-runtime
    See info in area-owners.md if you want to be subscribed.
12. vcsjones commented on Jul 18, 2025

    

    vcsjones

    on Jul 18, 2025

    Member

    More actions

    Marking this as System.Runtime because this suspiciously looks like a base64 encoding problem. Every other six bits are zeroed.
13. jeffhandley

    added this to the 10.0.0 milestone on Jul 22, 2025
14. jeffhandley

    removed

    untriagedNew issue has not been triaged by the area ownerNew issue has not been triaged by the area owner

    on Jul 22, 2025
15. jeffhandley commented on Jul 22, 2025

    

    jeffhandley

    on Jul 22, 2025

    Member

    More actions

    And if this is a base64 encoding problem, then that could also be the cause of System.Xml.Tests failing (#117614).

    I'm marking this as blocking-release as it seems like a potential lurking product issue.

    👍React with 👍1Reacted by Mitchell Hwang
16. jeffhandley

    added

    blocking-release

    on Jul 22, 2025
17. pavelsavara commented on Jul 23, 2025

    

    pavelsavara

    on Jul 23, 2025

    Member

    More actions

    Maybe related ?
    dotnet/aspnetcore#62828
18. bartonjs commented on Jul 29, 2025

    

    bartonjs

    on Jul 29, 2025

    Member

    More actions

    This was unified into #118143
19. bartonjs

    closed this as a duplicateof Data Corruption in System.Buffers.Text.Base64.EncodeToUtf8 #118143 on Jul 29, 2025
20. github-actions

    locked and limited conversation to collaborators on Aug 29, 2025