Исключение неправильной длины Blowfish

Question

Я шифрую свой файл с помощью алгоритма Blowfish, но, похоже, я ничего о нем не знаю. Всякий раз, когда я пытаюсь Encipher() он выдаст исключение, которое говорит "Неверная длина". Я подумал, что длина должна быть нулевой, когда он получает мод с 8, и я думаю, что это означает, что должно быть 8 на 8 блоков потока для запуска шифрования. Что я должен делать?

Метод шифрования Blowfish:

public void Encipher(byte[] data, int length)
{
    uint xl, xr;
    if ((length % 8) != 0) <-- Exception Line
        throw new Exception("Invalid Length");
    for (int i = 0; i < length; i += 8)
    {
        // Encode the data in 8 byte blocks.
        xl = (uint)((data[i] << 24) | (data[i + 1] << 16) | (data[i + 2] << 8) | data[i + 3]);
        xr = (uint)((data[i + 4] << 24) | (data[i + 5] << 16) | (data[i + 6] << 8) | data[i + 7]);
        Encipher(ref xl, ref xr);
        // Now Replace the data.
        data[i] = (byte)(xl >> 24);
        data[i + 1] = (byte)(xl >> 16);
        data[i + 2] = (byte)(xl >> 8);
        data[i + 3] = (byte)(xl);
        data[i + 4] = (byte)(xr >> 24);
        data[i + 5] = (byte)(xr >> 16);
        data[i + 6] = (byte)(xr >> 8);
        data[i + 7] = (byte)(xr);
    }
}

Мой метод шифрования:

private void EncryptFile(string szFilePath, string szInfoFile, string szKey, string szEncryptedFile = "")
        {
            // Blowfish
            Blowfish alg = new Blowfish(Encoding.Unicode.GetBytes(szKey));

            // Open file
            System.IO.FileStream originalStream = System.IO.File.OpenRead(szFilePath);

            // Store original file length
            long originalLength = originalStream.Length;
            System.IO.File.WriteAllText(szInfoFile, originalLength.ToString());

            Byte[] buffer = new byte[originalStream.Length + (originalStream.Length % 8)];
            originalStream.Read(buffer, 0, buffer.Length);
            originalStream.Close();

            // Encrypt
            alg.Encipher(buffer, buffer.Length);

            string szEncFile;

            if (szEncryptedFile != string.Empty) szEncFile = szEncryptedFile; else szEncFile = szFilePath;

            System.IO.FileStream stream = new System.IO.FileStream(szEncFile, System.IO.FileMode.Create);
            stream.Write(buffer, 0, buffer.Length);
            stream.Close(); 
        }

Благодарю.

Answer 1

Если вы пытаетесь округлить значение до следующего значения, кратного 8, то вам следует сделать это вместо этого:

Byte[] buffer = new byte[originalStream.Length + (8-(originalStream.Length % 8))];

Answer 2

Peter Ritchie ответил на это. Однако есть пара идиоматических частей, которые вы должны рассмотреть ниже. Один это упаковка IDisposableреализованные классы (такие как FileStreams) в using блоки в целях обеспечения утилизации ресурсов в случае исключительных условий при обработке. Еще один if..then Вы положили в одну строку. Это действительно.. странно. Я заменил его на троичный оператор, который, кажется, соответствует использованию, которое вы используете. Удачи.

private void EncryptFile(string szFilePath, string szInfoFile, string szKey, string szEncryptedFile = "")
    {
        // Blowfish
        Blowfish alg = new Blowfish(Encoding.Unicode.GetBytes(szKey));

        // Open file
        using (System.IO.FileStream originalStream = System.IO.File.OpenRead(szFilePath))
        {
            // Store original file length
            long originalLength = originalStream.Length;
            System.IO.File.WriteAllText(szInfoFile, originalLength.ToString());

            Byte[] buffer = new byte[originalStream.Length + (originalStream.Length % 8)];
            originalStream.Read(buffer, 0, buffer.Length);
        }

        // Encrypt
        alg.Encipher(buffer, buffer.Length);

        string szEncFile;

        szEncFile = string.IsNullOrEmpty(szEncryptedFile) ? szFilePath : szEncryptedFile;

        using (System.IO.FileStream stream = new System.IO.FileStream(szEncFile, System.IO.FileMode.Create))
        {
            stream.Write(buffer, 0, buffer.Length);
        }
    }