Problème de chiffrement avec AES : argument (apparemment) null

Version imprimable

Bonjour,

J'ai implémenté une classe pour faire le déchiffrement en AES qui se présente comme ceci:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
public string decryptWithAES(string cypherText, string passPhrase, byte[] randomNumber)
        {
            string decryptedData = null;
            AesCryptoServiceProvider aes;
            try
            {
                aes = new AesCryptoServiceProvider();
            }
            catch (System.PlatformNotSupportedException PlatNotSupExc)
            {
                throw;
            }
 
            try
            {
                aes.BlockSize = 128;
                aes.KeySize = 256;
                aes.Mode = CipherMode.CFB;
                aes.Padding = PaddingMode.PKCS7;
            }
            catch (System.Security.Cryptography.CryptographicException CryptoExc)
            {
                throw;
            }
 
            Rfc2898DeriveBytes derivedBytes;
            try
            {
                derivedBytes = new Rfc2898DeriveBytes(passPhrase, randomNumber, 20000);
            }
            catch (System.ArgumentNullException ArgNullExc)
            {
                throw;
            }
            catch (System.ArgumentException ArgExc)
            {
                throw;
            }
 
            try
            {
                aes.Key = derivedBytes.GetBytes(32);
                aes.IV = derivedBytes.GetBytes(16);
            }
            catch (System.Security.Cryptography.CryptographicException CryptoExc)
            {
                throw;
            }
            catch (System.ArgumentNullException ArgNullExc)
            {
                throw;
            }
            catch (System.ArgumentOutOfRangeException ArgOutOfRangeExc)
            {
                throw;
            }
 
            byte[] myData;
            try
            {
                myData = System.Text.Encoding.UTF8.GetBytes(cypherText);
            }
            catch (System.ArgumentNullException ArgNullExc)
            {
                throw;
            }
            catch (System.Text.EncoderFallbackException EncFallBackExc)
            {
                throw;
            }
 
            try
            {
                ICryptoTransform decryptor = aes.CreateDecryptor(aes.Key, aes.IV);
 
                using (MemoryStream msDecrypt = new MemoryStream(myData))
                {
                    using (CryptoStream csDecrypt = new CryptoStream(msDecrypt, decryptor, CryptoStreamMode.Read))
                    {
                        using (StreamReader swDecrypt = new StreamReader(csDecrypt))
                        {
                            decryptedData = swDecrypt.ReadToEnd();
                        }
                        csDecrypt.Close();
                    }
                    msDecrypt.Close();
                }
            }
            catch (System.OutOfMemoryException OutOfMemExc)
            {
                throw;
            }
            catch (System.IO.IOException IOExc)
            {
                throw;
            }
            catch (System.ArgumentNullException ArgNullExc)
            {
                throw;
            }
            catch (System.ArgumentException ArgExc)
            {
                throw;
            }
 
            return decryptedData;
        }

J'ai une erreur lorsque j'arrive dans ce bloque-ci :

Code:

1
2
3
4
using (StreamReader swDecrypt = new StreamReader(csDecrypt))
                        {
                            decryptedData = swDecrypt.ReadToEnd();
                        }

Ou j'obtiens l'erreur suivante:

Code:

Value cannot be null. Parameter name: inputBuffer

Est-ce que vous avez une idée de l'erreur? Serait-il possible qu'AES ait une limitation de la taille du cipher? Car je ne vois absolument pas ou est mon erreur :/

Merci bien :)

L.

24/04/2013, 21h45
Leelith

Bon je crois avoir trouver d'ou vient mon erreur. Quand je stock mes valeurs encryptée, j'utilise Convert.ToBase64String, ce qui je pense ne prend pas la totalité du array byte[].

Quelle est la bonne méthode du coup pour stocker du texte chiffré ?
24/04/2013, 21h47
tomlev

Poste la stacktrace de l'exception, qu'on voit un peu mieux d'où ça vient

Et voilà le stacktrace :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 System.ArgumentNullException was caught HResult=-2147467261 Message=Value cannot be null. Parameter name: inputBuffer Source=System.Core ParamName=inputBuffer StackTrace: at System.Security.Cryptography.CapiSymmetricAlgorithm.TransformFinalBlock(Byte[] inputBuffer, Int32 inputOffset, Int32 inputCount) at System.Security.Cryptography.CryptoStream.FlushFinalBlock() at System.Security.Cryptography.CryptoStream.Dispose(Boolean disposing) at System.IO.Stream.Close() at System.IO.Stream.Dispose() at MyProject.Security.CryptographyUtility.decryptWithAES(String cypherText, String passPhrase, Byte[] randomNumber) in (...)\CryptographyUtility.cs:line 347 InnerException:
Ca arrive quand je suis sur cette ligne : decryptedData = swDecrypt.ReadToEnd();

Mais par contre, je pense que cette erreur est due à une mauvaise conversation des bytes pour le stockage dans la base de donnée. Du coup je ne sais pas trop quelle conversion utiliser pour stocker ça de la bonne manière :/

Merci bien,

L.

24/04/2013, 22h19
tomlev

Bon, la stacktrace n'aide pas beaucoup...

Citation:

Envoyé par Leelith

Quand je stock mes valeurs encryptée, j'utilise Convert.ToBase64String, ce qui je pense ne prend pas la totalité du array byte[].

Qu'est-ce qui te fait dire ça ? ToBase64String donne une représentation fidèle des données d'entrée, il n'y a pas de perte... La seule possibilité pour que ça ne prenne pas la totalité du tableau de byte, ce serait que tu aies précisé les paramètres offset et length avec de mauvaises valeurs
24/04/2013, 22h44
Leelith

Non je n'ai pas précisé de valeurs particulières. Ou alors le problème serait comment je récupère les données ?

Actuellement je le fais avec System.Text.Encoding.UTF8.GetBytes(cypherText), est-ce que c'est correct?

L.
24/04/2013, 23h17
tomlev

Citation:

Envoyé par Leelith

Actuellement je le fais avec System.Text.Encoding.UTF8.GetBytes(cypherText), est-ce que c'est correct?

non, si tu as utilisé Convert.ToBase64String pour encoder les données cryptées, il faut utiliser Convert.FromBase64String pour convertir dans l'autre sens

Je me suis trompé, j'ai fais System.Text.Encoding.UTF8.GetBytes(plainText), et la même chose lors de la dechiffrement pour le texte chiffré (cf. code du premier poste).

Du coup il se pourrait que cette méthode ne fasse pas une conversion fidèle des données?

Voici le code de ma classe de chiffrement :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
public string encryptWithAES(string plainText, string passPhrase, byte[] randomNumber)
        {
            byte[] encryptedData = null;
            AesCryptoServiceProvider aes;
            try
            {
                aes = new AesCryptoServiceProvider();
            }
            catch (System.PlatformNotSupportedException platNotSupExc)
            {
                throw;
            }
 
            try
            {
                aes.BlockSize = 128;
                aes.KeySize = 256;
                aes.Mode = CipherMode.CFB;
                aes.Padding = PaddingMode.PKCS7;
            }
            catch (System.Security.Cryptography.CryptographicException CryptoExc)
            {
                throw;
            }
 
            Rfc2898DeriveBytes derivedBytes;
            try
            {
                derivedBytes = new Rfc2898DeriveBytes(passPhrase, randomNumber, 20000);
            }
            catch (System.ArgumentNullException ArgNullExc)
            {
                throw;
            }
            catch (System.ArgumentException ArgExc)
            {
                throw;
            }
            try
            {
                aes.Key = derivedBytes.GetBytes(32);
                aes.IV = derivedBytes.GetBytes(16);
            }
            catch (System.Security.Cryptography.CryptographicException CryptoExc)
            {
                throw;
            }
            catch (System.ArgumentNullException ArgNullExc)
            {
                throw;
            }
            catch (System.ArgumentOutOfRangeException ArgOutOfRangeExc)
            {
                throw;
            }
 
            try
            {
                byte[] myData = System.Text.Encoding.UTF8.GetBytes(plainText);
            }
            catch (System.ArgumentNullException ArgNullExc)
            {
                throw;
            }
            catch (System.Text.EncoderFallbackException EncFallExc)
            {
                throw;
            }
 
            ICryptoTransform encryptor;
            try
            {
                encryptor = aes.CreateEncryptor(aes.Key, aes.IV);
            }
            catch (System.ArgumentNullException ArgNullExc)
            {
                throw;
            }
            catch (System.ArgumentException ArgExc)
            {
                throw;
            }
 
            using (MemoryStream msEncrypt = new MemoryStream())
            {
                try
                {
                    using (CryptoStream csEncrypt = new CryptoStream(msEncrypt, encryptor, CryptoStreamMode.Write))
                    {
                        using (StreamWriter swEncrypt = new StreamWriter(csEncrypt))
                        {
                            swEncrypt.Write(plainText);
                            csEncrypt.FlushFinalBlock();
                        }
                        encryptedData = msEncrypt.ToArray();
                        csEncrypt.Close();
                    }
                    msEncrypt.Close();
                }
                catch (System.Security.Cryptography.CryptographicException ArgExc)
                {
                    throw;
                }
                catch (System.ObjectDisposedException ObjDisExc)
                {
                    throw;
                }
                catch (System.NotSupportedException NotSupExc)
                {
                    throw;
                }
                catch (System.IO.IOException IOExc)
                {
                    throw;
                }
                catch (System.ArgumentNullException ArgNullExc)
                {
                    throw;
                }
                catch (System.ArgumentException ArgExc)
                {
                    throw;
                }
            }
            try
            {
                return Convert.ToBase64String(encryptedData);
            }
            catch (System.ArgumentNullException ArgNullExc)
            {
                throw;
            }
        }

25/04/2013, 20h29
tomlev
J'ai l'impression que tu te mélanges un peu les pinceaux... donc je résume ce qu'il faut faire :

Chiffrement :
```
plainText -> (UTF8.GetBytes) -> plainBytes
          -> (chiffrement) -> cypherBytes
          -> (ToBase64) -> cipherText
```
Déchiffrement :
```
cipherText -> (FromBase64) -> cipherBytes
           -> (déchiffrement) -> plainBytes
           -> (UTF8.GetString) -> plainText
```
(comme tu as fait avec un StreamReader, ça devrait marcher aussi)

Citation:

Du coup il se pourrait que cette méthode ne fasse pas une conversion fidèle des données?

Toutes ces méthodes font une conversion fidèle des données, mais elles travaillent chacune sur un certain type de données, et si tu passes des données qui ne sont pas celles qui sont attendues, tu vas forcément obtenir n'importe quoi. Par exemple appeler UTF8.GetString sur des données binaires chiffrées n'a pas de sens, puisque ces données ne représentent pas du texte en UTF8. De même, appeler UTF8.GetBytes sur une chaine base64 va te donner quelque chose, mais qui ne correspond pas aux données binaires représentées par la chaine base64...

Après, il se peut qu'il y ait d'autres erreurs dans l'utilisation des classes de cryptographie... là dessus je ne peux pas vraiment t'aider, je ne suis pas du tout expert dans ce domaine. La méthode de chiffrement me semble correcte à première vue, mais chez moi elle pète une erreur ("The input data is not a complete block")

Au fait, pourquoi tous ces blocs catch avec juste un throw ? Si c'est pour mettre des breakpoints, il y a un moyen plus simple... Menu Debug -> Exceptions, et cocher la checkbox correspondant aux exceptions CLR. Le debugger s'arrêtera automatiquement quand une exception se produit
25/04/2013, 21h13
Leelith
Citation:
Envoyé par tomlev

J'ai l'impression que tu te mélanges un peu les pinceaux... donc je résume ce qu'il faut faire :

Chiffrement :

plainText -> (UTF8.GetBytes) -> plainBytes -> (chiffrement) -> cypherBytes -> (ToBase64) -> cipherText
Déchiffrement :

cipherText -> (FromBase64) -> cipherBytes -> (déchiffrement) -> plainBytes -> (UTF8.GetString) -> plainText
(comme tu as fait avec un StreamReader, ça devrait marcher aussi)

Toutes ces méthodes font une conversion fidèle des données, mais elles travaillent chacune sur un certain type de données, et si tu passes des données qui ne sont pas celles qui sont attendues, tu vas forcément obtenir n'importe quoi. Par exemple appeler UTF8.GetString sur des données binaires chiffrées n'a pas de sens, puisque ces données ne représentent pas du texte en UTF8. De même, appeler UTF8.GetBytes sur une chaine base64 va te donner quelque chose, mais qui ne correspond pas aux données binaires représentées par la chaine base64...
Effectivement, c'est même logique de devoir refaire les opérations en ordre inverse... C'est ça de coder jusqu'à pas d'heure :aie:

Citation:

Envoyé par tomlev

Après, il se peut qu'il y ait d'autres erreurs dans l'utilisation des classes de cryptographie... là dessus je ne peux pas vraiment t'aider, je ne suis pas du tout expert dans ce domaine. La méthode de chiffrement me semble correcte à première vue, mais chez moi elle pète une erreur ("The input data is not a complete block")

Au fait, pourquoi tous ces blocs catch avec juste un throw ? Si c'est pour mettre des breakpoints, il y a un moyen plus simple... Menu Debug -> Exceptions, et cocher la checkbox correspondant aux exceptions CLR. Le debugger s'arrêtera automatiquement quand une exception se produit

J'ai trouvé d'ou vient l'erreur de padding:

Code:

aes.Padding = PaddingMode.PKCS7;

Si tu ne mets pas de padding :

Code:

aes.Padding = PaddingMode.None;

Là il n'y a pas de problème... bizarre car je ne comprend pas du coup car il me semblait que c'était conseillé d'ajouter du padding à la fin?
25/04/2013, 21h26
tomlev

Citation:

Envoyé par Leelith

il me semblait que c'était conseillé d'ajouter du padding à la fin?

Aucune idée... comme je te l'ai dit, je n'y connais pas grand chose en crypto ;)
25/04/2013, 21h30
Leelith

Ok, je vais demander à quelqu'un qui s'y connait bien ;) Je posterais la réponse si jamais ça t'intéresse ou si ça peut intéresser quelqu'un :)
27/04/2013, 15h41
Leelith

J'ai trouvé le problème!

En fait, si tu choisi mode CFB (AesCryptoServiceProvider.Mode = CipherMode.CFB), tu peux spécifier un padding de type PKCS7 par exemple. Le mode de fonctionnement du CFB fait en sorte qu'il va gérer les blocks qui ne sont pas multiple de la taille de block définit (AesCryptoServiceProvider.BlockSize). Donc avec CFB, vu qu'il gère ça de manière autonome il n'y a pas besoin de faire appel à la méthode FlushFinalBlock() du stream ;)

Voilà voilà :)

L.