Android AES加密/解密忽略密码

Question

我想做一个简单的应用程序加密/解密与AES的消息。我的代码似乎现在工作，因为我得到的文本加密和解密没有任何问题。

我有一个输入字段、一个输入密码用于解密/加密的字段和一个输出字段。和两个按钮（加密/解密）。

问题是****，当我输入一条消息，设置密码并对其进行加密，然后试图引发无效的密码获取时，尽管输入的密码与我用于加密的密码不匹配，但消息还是会解密。

下面是我的密钥生成代码：

public void vers(){
    // Das Passwort bzw der Schluesseltext
    keyStr = keyedit.getText().toString();
    // byte-Array erzeugen

    try {
        key = (keyStr).getBytes("UTF-8");
    } catch (UnsupportedEncodingException e) {
        // TODO Auto-generated catch block
        e.printStackTrace();
    }
    // aus dem Array einen Hash-Wert erzeugen mit MD5 oder SHA

    try {
        sha = MessageDigest.getInstance("MD5");
    } catch (NoSuchAlgorithmException e) {
        // TODO Auto-generated catch block
        e.printStackTrace();
    }
    key = sha.digest(key);
    // nur die ersten 128 bit nutzen
    key = Arrays.copyOf(key, 16); 
    // der fertige Schluessel
    secretKeySpec = new SecretKeySpec(key, "AES");
}

这里是我加密消息的代码：

private void codealgo() {

    vers();

    // der zu verschl. Text
    text1 = input.getText().toString();


    // Verschluesseln

    try {
        cipher = Cipher.getInstance("AES");
    } catch (NoSuchAlgorithmException | NoSuchPaddingException e) {
        // TODO Auto-generated catch block
        e.printStackTrace();
    }
    try {
        cipher.init(Cipher.ENCRYPT_MODE, secretKeySpec);
    } catch (InvalidKeyException e) {
        // TODO Auto-generated catch block
        e.printStackTrace();
    }

    try {
        encrypted = cipher.doFinal(text1.getBytes());
    } catch (IllegalBlockSizeException e) {
        // TODO Auto-generated catch block
        e.printStackTrace();
    } catch (BadPaddingException e) {
        // TODO Auto-generated catch block
        e.printStackTrace();
    }

    // bytes zu Base64-String konvertieren (dient der Lesbarkeit)
    geheim = Base64.encodeToString(encrypted, Base64.NO_WRAP);


    // Ergebnis
    output.setText(geheim);

}

至少这里是我试图再次解密消息的代码：

private void decodealgo() {

        vers();

        geheim2 = input.getText().toString();

    // BASE64 String zu Byte-Array konvertieren
    data = Base64.decode(geheim2, Base64.NO_WRAP);


    // Entschluesseln

    try {
        cipher3 = Cipher.getInstance("AES");
    } catch (NoSuchAlgorithmException e) {
        // TODO Auto-generated catch block
        e.printStackTrace();
    } catch (NoSuchPaddingException e) {
        // TODO Auto-generated catch block
        e.printStackTrace();
    }
    try {
        cipher3.init(Cipher.DECRYPT_MODE, secretKeySpec2);
    } catch (InvalidKeyException e) {
        // TODO Auto-generated catch block
        Toast.makeText(getApplicationContext(), "Invalid key",
        Toast.LENGTH_SHORT).show();
        e.printStackTrace();
    }

    try {
        cipherData3 = cipher3.doFinal(data);
    } catch (IllegalBlockSizeException e) {
        // TODO Auto-generated catch block
        Toast.makeText(getApplicationContext(), "No valid encryption",
        Toast.LENGTH_SHORT).show();
    } catch (BadPaddingException e) {
        // TODO Auto-generated catch block
        Toast.makeText(getApplicationContext(), "Key invalid format",
        Toast.LENGTH_SHORT).show();
        e.printStackTrace();
    }

    try {
        text3 = new String(cipherData3, "UTF-8");
    } catch (UnsupportedEncodingException e) {
        // TODO Auto-generated catch block
        e.printStackTrace();
    }


    // Klartext
    output.setText(text3);

}

请告诉我为什么密码会被解密过程忽略，以及为什么一个随机条目会显示用密码加密的消息。

Answer 1

您似乎对几乎任何变量都使用字段。您可以将此等同于为应用程序使用全局变量，尽管这是在类级别上执行的。类的字段数量应该最小化（尽管不应该将同一个字段用于多个目的）。相反，您应该对不需要在方法之间共享的任何内容使用局部变量。

例如，所有与键相关的变量当前都是字段。您只需要一个secretkey实例来加密/解密，所以所有这些变量都应该在本地变量中（不过，您很好地将密钥派生放在一个单独的方法中）。此外，应该只将密钥用作encrypt和decrypt方法的参数。这样就不会突然将某个字段保留到以前的值。共享cipher实例总是一个坏主意，尤其是在不能保证使用相同密钥的情况下。

最后，请注意“aes/ECB/pkcs5padding”（可能是cipher.getinstance（“aes”）所暗示的）可能会抛出badpaddingexception，但这并不保证，即使密钥或密文不正确。为了保证密文的完整性和真实性，您必须在密文上使用(H)MAC。ECB模式本身只提供一定程度的机密性。最简单的方法可能是切换到GCM模式加密。

最后，密码不是密钥，请搜索PBKDF2以获得更多信息。不要使用MD5。