IllegalBlockSizeException при использовании aes/ecb/pkcs5padding для расшифровки массива байтов

Question

Я понимаю, что шифрование AES должно быть в блоках по 16, но у меня создалось впечатление, что с помощью Cipher.getInstance("AES/ECB/PKCS5PADDING"); для заполнения этого байта использовался массив байтов. Мой код ниже:

CipherUtils.java

private static byte[] key = { 
     0x74, 0x68, 0x69, 0x73, 0x49, 0x73, 0x41, 0x53, 0x65, 0x63, 0x72, 0x65, 0x74, 0x4b, 0x65, 0x79 
};//"thisIsASecretKey"; 

public static byte[] EncryptByteArray(byte[] array) 
{ 
    try 
    { 
     Cipher cipher = Cipher.getInstance("AES/ECB/PKCS5PADDING"); 
     SecretKeySpec secretKey = new SecretKeySpec(key, "AES"); 
     cipher.init(Cipher.ENCRYPT_MODE, secretKey); 

     return (cipher.doFinal(array)); 
    } 
    catch (Exception e) 
    { 
     e.printStackTrace(); 

    } 
    return null; 
} 

public static byte[] DecryptByteArray(byte[] array) 
{ 
    try 
    { 
     Cipher cipher = Cipher.getInstance("AES/ECB/PKCS5PADDING"); 
     SecretKeySpec secretKey = new SecretKeySpec(key, "AES"); 
     cipher.init(Cipher.DECRYPT_MODE, secretKey); 

     return cipher.doFinal(array); 
    } 
    catch (Exception e) 
    { 
     e.printStackTrace(); 

    } 
    return null; 
} 

Основная программа

 fis = new FileInputStream(path); 

     toDecrypt = new byte[fis.available()+1]; 

     int content; 
     int i = 0; 
     while ((content = fis.read()) != -1) { 

      // convert to byte and display it 
      toDecrypt[i] = (byte)content; 
      i += 1; 
     } 

     byte[] decryptedStr = CipherUtils.DecryptByteArray(toDecrypt); 

     FileOutputStream decryptedStream = new FileOutputStream(path); 
     decryptedStream.write (decryptedStr); 
     decryptedStream.close(); 

Файл в path был зашифрован с помощью функции в cipherutils.java и записываются в файл с использованием FileOutputStream.write

U pdate- Я создаю для Android с помощью Gradle.

Answer 1

Успели что-то придумаем с помощью @ JonSkeet в ответ (спасибо!):

 File file = new File(path); 
     fis = new FileInputStream(file); 

     toDecrypt = new byte[(int)file.length()]; 
     fis.read(toDecrypt); 

     byte[] decrypted = CipherUtils.DecryptByteArray(toDecrypt); 
     FileOutputStream decryptedStream = new FileOutputStream(bookPath); 
     decryptedStream.write (decrypted); 
     decryptedStream.close(); 

Как он отметил, мне не следовало использовать метод available(). Также был намного лучший способ записи в файл быстрее, чем повторение каждого байта! Согласно комментариям, я также изменил шифрование на режим CBC со случайным IV.

Answer 2

Это проблема:

toDecrypt = new byte[fis.available()+1]; 

Во-первых, вы используете available() метод, который никогда не хорошей идеей. Далее, даже если предположить, что - это, возвращая длину файла, вы добавляете 1 к нему - почему? Конечно, вы хотите просто байты в файле.

Самый простой способ сделать это просто использовать Files.readAllBytes:

byte[] toDecrypt = Files.readAllBytes(Paths.get(path)); 
// TODO: Change the method name to follow Java conventions 
byte[] decrypted = CipherUtils.DecryptByteArray(toDecrypt); 
Files.write(Paths.get(path), decrypted); 

Теперь вам не нужно беспокоиться о закрытии файла потоков, либо ... (Если бы удалось расшифровать, вы, вероятно, не смог бы написать, потому что у вас еще есть файл открыт для чтения в текущем коде)

Я также настоятельно рекомендую пересматривают ваше исключение «обработки»:.

• Catchi нг Exception почти всегда плохая идея
• Вызов e.printStackTrace() и затем продолжается, как будто ничего не произошло почти всегда плохая идея