Blowfish sur studio android pour le cryptage/décryptage

Question

J'ai cherché presque tous les sujets ici sur ce sujet mais je n'arrive toujours pas à comprendre ce qui ne va pas. Où j'ai créé 3 EditText sur mon application android: EditText où vous insérez la clé/mot de passe pour crypter et décrypter; EditText2 où est affiché le texte chiffré; EditText3 où il est affiché le texte déchiffré.

Comme c'est toujours un test précoce, j'ai mis le message ou la chaîne à Crypt comme variable dans l'application.

Le problème est que le cryptage donne quelque chose comme un algorithme blowfish, donc pas de problème là (ça se termine par 2 == donc je pense que ça marche bien). J'ai également essayé de décoder la chaîne avant le décryptage ou en utilisant le cryptage byte[] sans aucun résultat. Le déchiffrement ne restitue pas le texte String original, mais donne quelque chose de plus gros que le texte chiffré. Je n'ai pas de préférence sur le mode de blowfish donc j'ai commencé facile comme Blowfish/CFB/NoPadding.

str_key, str2 et str3 sont déclarés publics pour le moment. str2 définira le texte pour le champ EditText2 et str3 définira le texte pour le champ EditText3. Un exemple de sortie: Example of output generated from the app

Voici le code:

public void encrypt(){ 
    //encrypt 
    EditText mEdit = (EditText)findViewById(R.id.editText); 
    str_key = (String) mEdit.getText().toString(); 

    int iterationCount = 1000; 
    int keyLength = 256; 
    int saltLength = keyLength/8; 

    SecureRandom random = new SecureRandom(); 
    byte[] salt = new byte[saltLength]; 
    random.nextBytes(salt); 
    KeySpec keySpec = new PBEKeySpec(str_key.toCharArray(), salt, 
      iterationCount, keyLength); 
    SecretKeyFactory keyFactory = null; 
    try { 
     keyFactory = SecretKeyFactory.getInstance("PBKDF2WithHmacSHA1"); 
    } catch (NoSuchAlgorithmException e) { 
     e.printStackTrace(); 
    } 
    byte[] keyBytes = new byte[0]; 
    try { 
     keyBytes = keyFactory.generateSecret(keySpec).getEncoded(); 
    } catch (InvalidKeySpecException e) { 
     e.printStackTrace(); 
    } 
    SecretKey key = new SecretKeySpec(keyBytes, "Blowfish"); 

    Cipher cipher = null; 
    try { 
     cipher = Cipher.getInstance("Blowfish/CBC/PKCS5Padding"); 
    } catch (NoSuchAlgorithmException | NoSuchPaddingException e) { 
     e.printStackTrace(); 
    } 
    if (cipher == null || key == null) { 
     //throw new Exception("Invalid key or cypher"); 
     str2="error"; 
    } 
    else { 
     byte[] iv = new byte[cipher.getBlockSize()]; 
     random.nextBytes(iv); 
     IvParameterSpec ivParams = new IvParameterSpec(iv); 
     try { 
      cipher.init(Cipher.ENCRYPT_MODE, key,ivParams); 
     } catch (InvalidKeyException | InvalidAlgorithmParameterException e) { 
      e.printStackTrace(); 
     } 


     try { 
      raw = cipher.doFinal(message.getBytes("UTF-8")); 
     } catch (IllegalBlockSizeException | BadPaddingException | UnsupportedEncodingException e) { 
      e.printStackTrace(); 
     } 
     str2 = Base64.encodeToString(raw,Base64.DEFAULT); 

    } 


} 

Voici la fonction Décrypter:

public void decrypt(){ 
      int iterationCount = 1000; 
      int keyLength = 256; 
      int saltLength = keyLength/8; 
      SecureRandom random = new SecureRandom(); 
      byte[] salt = new byte[saltLength]; 
      random.nextBytes(salt); 
      KeySpec keySpec = new PBEKeySpec(str_key.toCharArray(), salt, iterationCount, keyLength); 
      SecretKeyFactory keyFactory = null; 
      try { 
       keyFactory = SecretKeyFactory.getInstance("PBKDF2WithHmacSHA1"); 
       } catch (NoSuchAlgorithmException e) { 
       e.printStackTrace(); 
       } 
       byte[] keyBytes = new byte[0]; 
       try { 
       keyBytes = keyFactory.generateSecret(keySpec).getEncoded(); 
       } catch (InvalidKeySpecException e) { 
        e.printStackTrace(); 
       } 
       SecretKey key = new SecretKeySpec(keyBytes, "Blowfish"); 



    Cipher cipher2 = null; 
    try { 
     cipher2 = Cipher.getInstance("Blowfish/CBC/PKCS5Padding"); 
    } catch (NoSuchAlgorithmException | NoSuchPaddingException e) { 
     e.printStackTrace(); 
    } 
    iv = new byte[cipher2.getBlockSize()]; 
    random.nextBytes(iv); 
    IvParameterSpec ivSpec = new IvParameterSpec(iv); 
    try { 
     cipher2.init(Cipher.DECRYPT_MODE, key, ivSpec); 
    } catch (InvalidKeyException | InvalidAlgorithmParameterException e) { 
     e.printStackTrace(); 
    } 
    byte[] decryptedBytes = null; 
    byte[] app= Base64.decode(str2,Base64.DEFAULT); 

    try { 
     decryptedBytes = cipher2.doFinal(app); 
    } catch (IllegalBlockSizeException | BadPaddingException e) { 
     e.printStackTrace(); 
    } 


    str3 = Base64.encodeToString(decryptedBytes,Base64.DEFAULT); 
} 

Answer 1

Vous encodez le résultat chiffré à base64 mais quand vous décryptez, vous prenez les octets simples de cette base64. Vous devriez plutôt décoder en base64 pour obtenir le tableau d'octets du texte chiffré, puis le déchiffrer.

Vous dérivez également votre clé directement à partir du tableau d'octets UTF8, ce qui est une très mauvaise façon de le faire. Vous devriez plutôt utiliser un KDF. PBKDF2 est le plus couramment utilisé ici.

La façon dont vous générez un IV (en n'en générant pas du tout) est également très médiocre. Il devrait être généré aléatoirement et préfixé au texte chiffré. Il n'a pas besoin d'être secret, juste imprévisible. Enfin, vous n'utilisez pas du tout un HMAC, donc n'importe qui pourrait changer le texte chiffré et vous ne le sauriez pas.