Как исправить «Заполнение недопустимо и не может быть удалено» при шифровании и дешифровании в одном и том же процессе - PullRequest
1 голос
/ 01 июня 2019

Я пытаюсь зашифровать и расшифровать файл XML. Программа пытается открыть, отредактировать и повторно зашифровать данные, но я получаю ошибку «System.Security.Cryptography.CryptographicException: заполнение недопустимо и не может быть удалено». Когда я шифрую, а затем дешифрую, запустив программу дважды, она работает просто отлично, но программа создает ошибку при выполнении их обоих.

Это очень распространенная ошибка, но также очень расплывчатая, со многими различными "сценариями". Я некоторое время искал один из способов исправить это, и я попробовал все «исправления» онлайн. Никто еще не работал.

class Program
    {
        static void Main(string[] args)
        {
        }
        public static void encFile(string input, string password)
        {
            GCHandle gch = GCHandle.Alloc(password, GCHandleType.Pinned);

            FileEncrypt(input, password);

            ZeroMemory(gch.AddrOfPinnedObject(), password.Length * 2);
            gch.Free();
        }
        public static string output;
        public static void decFile(string input, string password)
        {
            GCHandle gch = GCHandle.Alloc(password, GCHandleType.Pinned);

            // Decrypt the file
            output = FileDecrypt(input, password);

        }

        [DllImport("KERNEL32.DLL", EntryPoint = "RtlZeroMemory")]
        public static extern bool ZeroMemory(IntPtr Destination, int Length);

        /// <summary>
        /// Encrypts a file from its path and a plain password.
        /// </summary>
        /// <param name="inputFile"></param>
        /// <param name="password"></param>
        private static void FileEncrypt(string inputFile, string password)
        {
            File.WriteAllText(Path.GetDirectoryName(inputFile) + "\\" + Path.GetFileNameWithoutExtension(inputFile) + ".dat", Crypto.EncryptStringAES(File.ReadAllText(inputFile), "Password123!"));
        }

        /// <summary>
        /// Decrypts an encrypted file with the FileEncrypt method through its path and the plain password.
        /// </summary>
        /// <param name="inputFile"></param>
        /// <param name="outputFile"></param>
        /// <param name="password"></param>
        private static string FileDecrypt(string inputFile, string password)
        {
            return Crypto.DecryptStringAES(File.ReadAllText(inputFile), password);
        }
    }
    public class Crypto
    {

        //While an app specific salt is not the best practice for
        //password based encryption, it's probably safe enough as long as
        //it is truly uncommon. Also too much work to alter this answer otherwise.
        private static byte[] _salt = Encoding.ASCII.GetBytes("rxONBa*&e03!%76N9mBlbR#@Xl&A&w");

        /// <summary>
        /// Encrypt the given string using AES.  The string can be decrypted using 
        /// DecryptStringAES().  The sharedSecret parameters must match.
        /// </summary>
        /// <param name="plainText">The text to encrypt.</param>
        /// <param name="sharedSecret">A password used to generate a key for encryption.</param>
        public static string EncryptStringAES(string plainText, string sharedSecret)
        {
            if (string.IsNullOrEmpty(plainText))
                throw new ArgumentNullException("plainText");
            if (string.IsNullOrEmpty(sharedSecret))
                throw new ArgumentNullException("sharedSecret");

            string outStr = null;                       // Encrypted string to return
            RijndaelManaged aesAlg = null;              // RijndaelManaged object used to encrypt the data.

            try
            {
                // generate the key from the shared secret and the salt
                Rfc2898DeriveBytes key = new Rfc2898DeriveBytes(sharedSecret, _salt);

                // Create a RijndaelManaged object
                aesAlg = new RijndaelManaged();
                aesAlg.Key = key.GetBytes(aesAlg.KeySize / 8);
                aesAlg.Padding = PaddingMode.PKCS7;
                // Create a decryptor to perform the stream transform.
                ICryptoTransform encryptor = aesAlg.CreateEncryptor(aesAlg.Key, aesAlg.IV);

                // Create the streams used for encryption.
                using (MemoryStream msEncrypt = new MemoryStream())
                {
                    // prepend the IV
                    msEncrypt.Write(BitConverter.GetBytes(aesAlg.IV.Length), 0, sizeof(int));
                    msEncrypt.Write(aesAlg.IV, 0, aesAlg.IV.Length);
                    using (CryptoStream csEncrypt = new CryptoStream(msEncrypt, encryptor, CryptoStreamMode.Write))
                    {
                        using (StreamWriter swEncrypt = new StreamWriter(csEncrypt))
                        {
                            //Write all data to the stream.
                            swEncrypt.Write(plainText);
                        }
                    }
                    outStr = Convert.ToBase64String(msEncrypt.ToArray());
                }
            }
            finally
            {
                // Clear the RijndaelManaged object.
                if (aesAlg != null)
                    aesAlg.Clear();
            }

            // Return the encrypted bytes from the memory stream.
            return outStr;
        }

        /// <summary>
        /// Decrypt the given string.  Assumes the string was encrypted using 
        /// EncryptStringAES(), using an identical sharedSecret.
        /// </summary>
        /// <param name="cipherText">The text to decrypt.</param>
        /// <param name="sharedSecret">A password used to generate a key for decryption.</param>
        public static string DecryptStringAES(string cipherText, string sharedSecret)
        {
            if (string.IsNullOrEmpty(cipherText))
                throw new ArgumentNullException("cipherText");
            if (string.IsNullOrEmpty(sharedSecret))
                throw new ArgumentNullException("sharedSecret");

            // Declare the RijndaelManaged object
            // used to decrypt the data.
            RijndaelManaged aesAlg = null;

            // Declare the string used to hold
            // the decrypted text.
            string plaintext = null;

            try
            {
                // generate the key from the shared secret and the salt
                Rfc2898DeriveBytes key = new Rfc2898DeriveBytes(sharedSecret, _salt);

                // Create the streams used for decryption.                
                byte[] bytes = Convert.FromBase64String(cipherText);
                using (MemoryStream msDecrypt = new MemoryStream(bytes))
                {
                    // Create a RijndaelManaged object
                    // with the specified key and IV.
                    aesAlg = new RijndaelManaged();
                    aesAlg.Key = key.GetBytes(aesAlg.KeySize / 8);
                    // Get the initialization vector from the encrypted stream
                    aesAlg.IV = ReadByteArray(msDecrypt);
                    aesAlg.Padding = PaddingMode.PKCS7;
                    // Create a decrytor to perform the stream transform.
                    ICryptoTransform decryptor = aesAlg.CreateDecryptor(aesAlg.Key, aesAlg.IV);
                    using (CryptoStream csDecrypt = new CryptoStream(msDecrypt, decryptor, CryptoStreamMode.Read))
                    {
                        using (StreamReader srDecrypt = new StreamReader(csDecrypt))

                            // Read the decrypted bytes from the decrypting stream
                            // and place them in a string.
                            plaintext = srDecrypt.ReadToEnd();
                    }
                }
            }
            finally
            {
                // Clear the RijndaelManaged object.
                if (aesAlg != null)
                    aesAlg.Clear();
            }

            return plaintext;
        }

        private static byte[] ReadByteArray(Stream s)
        {
            byte[] rawLength = new byte[sizeof(int)];
            if (s.Read(rawLength, 0, rawLength.Length) != rawLength.Length)
            {
                throw new SystemException("Stream did not contain properly formatted byte array");
            }

            byte[] buffer = new byte[BitConverter.ToInt32(rawLength, 0)];
            if (s.Read(buffer, 0, buffer.Length) != buffer.Length)
            {
                throw new SystemException("Did not read byte array properly");
            }

            return buffer;
        }
    }

Выполнение следующего кода, который шифрует, а затем расшифровывает, приведет к ошибке:

static void Main(string[] args)
        {
            string password = "Password123!";
            encFile(@"C:\KaliPatriot\test.xml", password);
            decFile(@"C:\KaliPatriot\test.dat", password);
            Console.WriteLine(output);
        }

Но если вы попытаетесь сначала запустить этот код, который только шифрует файл:

static void Main(string[] args)
        {
            string password = "Password123!";
            encFile(@"C:\KaliPatriot\test.xml", password);
        }

, а затем расшифровывать файл в другом процессе после шифрования:

static void Main(string[] args)
        {
            string password = "Password123!";
            decFile(@"C:\KaliPatriot\test.dat", password);
            Console.WriteLine(output);
        }

Тогда программа выведет содержимое правильно, без ошибок.

Как можно зашифровать и расшифровать файл за один прогон, без каких-либо ошибок.

1 Ответ

1 голос
/ 01 июня 2019

Переменная password и параметр с одинаковым именем указывают на одну и ту же строку констант. Первый из ваших методов обнулит эту строку.

    static void Main(string[] args)
    {
        string password = "Password123!";

        encFile(@"C:\KaliPatriot\test.xml", password);
        // password is now "\0\0\0\0\0\0\0\0\0"            
        decFile(@"C:\KaliPatriot\test.dat", password);
        Console.WriteLine(output);
    }

Использование такой строки очень опасно. Предполагается (и предполагается), что строки являются неизменяемыми, и .NET также использует интернирование для строк.

Вам удалось получить UB (Неопределенное поведение) в C #, а не то, чем можно гордиться.

...