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

**Leelith** · 24/04/2013, 17h20

Bonjour,

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

Code C# :

Sélectionner tout - Visualiser dans une fenêtre à part

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 C# :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Ou j'obtiens l'erreur suivante:

Code C# :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

**Leelith** · 24/04/2013, 21h45

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é ?

**tomlev** · 24/04/2013, 21h47

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

**Leelith** · 24/04/2013, 22h02

Et voilà le stacktrace :

Code C# :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

**tomlev** · 24/04/2013, 22h19

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

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

**Leelith** · 24/04/2013, 22h44

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.

**tomlev** · 24/04/2013, 23h17

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

**Leelith** · 25/04/2013, 18h03

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 C# :

Sélectionner tout - Visualiser dans une fenêtre à part

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;
            }
        }

**tomlev** · 25/04/2013, 20h29

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)

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

**Leelith** · 25/04/2013, 21h13

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

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 C# :

Sélectionner tout - Visualiser dans une fenêtre à part

aes.Padding = PaddingMode.PKCS7;

Si tu ne mets pas de padding :

Code C# :

Sélectionner tout - Visualiser dans une fenêtre à part

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?

**tomlev** · 25/04/2013, 21h26

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

**Leelith** · 25/04/2013, 21h30

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

**Leelith** · 27/04/2013, 15h41

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.

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

Framework .NET

Discussions similaires

Partager

Partager