Change default CFB feedback size for DES
authorKevin Jones <kevin@vcsjones.com>
Wed, 3 May 2023 18:21:22 +0000 (14:21 -0400)
committerGitHub <noreply@github.com>
Wed, 3 May 2023 18:21:22 +0000 (14:21 -0400)
commitf1fcba4d0a8a37701755d3350365f7dae7039ab5
tree307f09188f18569d15e608714ef9eb6fb6de3ed4
parentc8f43d52f699c1da289a5cb36d371aec15901ab8
Change default CFB feedback size for DES

The previous default value, 64, doesn't work for DES because .NET only supports CFB8 for DES. Further, this more closely aligns the behavior of DES.Create() from .NET Framework to .NET, as DESCryptoServiceProvider also had a default feedback size of 8.

This change sets the default feedback size to 8, which is the only value that worked anyway, so it isn't a breaking change.

Types derived from DES will continue to have a default feedback size of 64, which is the default value for .NET Framework derived types. We are only changing the default for DESImplementation returned by DES.Create().
src/libraries/Common/tests/System/Security/Cryptography/AlgorithmImplementations/DES/DesTests.cs
src/libraries/System.Security.Cryptography/src/System/Security/Cryptography/DESCryptoServiceProvider.Unix.cs
src/libraries/System.Security.Cryptography/src/System/Security/Cryptography/DesImplementation.cs
src/libraries/System.Security.Cryptography/tests/DESTests.cs