使用 salt 将 Rijndael 解密从 PHP 移植到 Java

Question

我在将带有解密示例的 php 代码转换为 java 时遇到问题。 这是php代码：

function decrypt($encrypted, $password, $salt='2#g+XK^Sc3"4ABXbvwF8CPD%en%;9,c(') {
    // Build a 256-bit $key which is a SHA256 hash of $salt and $password.
    $key = hash('SHA256', $salt . $password, true);
    // Retrieve $iv which is the first 22 characters plus ==, base64_decoded.
    $iv = base64_decode(substr($encrypted, 0, 22) . '==');
    // Remove $iv from $encrypted.
    $encrypted = substr($encrypted, 22);
    // Decrypt the data.  rtrim won't corrupt the data because the last 32 characters are the md5 hash; thus any \0 character has to be padding.
    $decrypted = rtrim(mcrypt_decrypt(MCRYPT_RIJNDAEL_128, $key, base64_decode($encrypted), MCRYPT_MODE_CBC, $iv), "\0\4");
    // Retrieve $hash which is the last 32 characters of $decrypted.
    $hash = substr($decrypted, -32);
    // Remove the last 32 characters from $decrypted.
    $decrypted = substr($decrypted, 0, -32);
    // Integrity check.  If this fails, either the data is corrupted, or the password/salt was incorrect.
    if (md5($decrypted) != $hash) return false;
    // Yay!
    return $decrypted;
}

这是我所做的 java 代码。但这不起作用。

private static String password = "AxkbK2jZ5PMaeNZWfn8XRLUWF2waGwH2EkAXxBDU6aZ";
private static String salt = "2#g+XK^Sc3\"4ABXbvwF8CPD%en%;9,c(";
private static String text = "Fm+Zfufqe3DjRQtWcYdw9g9oXriDjrAkRrBLhEfu7fCtT4BzD0gw7D+8KxrcbbgJm26peTUWHU2k4YJ4KqCSRQN3NPzuXwlJ4mC4444Edg3Q==";

public String decrypt(String pass, String encr) {

    try {
        int i = 0;

        String key = hash();
        byte[] iv = Base64.decodeBase64(text.substring(0, 22) + "==");

        Cipher cipher = Cipher.getInstance("DES");
        SecretKeySpec keySpec = new SecretKeySpec(password.getBytes(), "DES");
        IvParameterSpec ivSpec = new IvParameterSpec(salt.getBytes());
        cipher.init(Cipher.DECRYPT_MODE, keySpec, ivSpec);
        ByteArrayInputStream fis = new ByteArrayInputStream(iv);
        CipherInputStream cis = new CipherInputStream(fis, cipher);
        ByteArrayOutputStream fos = new ByteArrayOutputStream();
        // decrypting
        byte[] b = new byte[8];
        while ((i = cis.read(b)) != -1) {
            fos.write(b, 0, i);
        }
        fos.flush();
        fos.close();
        cis.close();
        fis.close();

        return fos.toString();
    } catch (Exception e) {
        e.printStackTrace();
    }

    return null;
}

private String hash() {
    StringBuffer sb = new StringBuffer();

    try {
        MessageDigest md = null;
        md = MessageDigest.getInstance("SHA-256");
        md.update((password + salt).getBytes());
        byte byteData[] = md.digest();

        for (int i = 0; i < byteData.length; i++) {
            sb.append(Integer.toString((byteData[i] & 0xff) + 0x100, 16).substring(1));
        }
    } catch (NoSuchAlgorithmException e) {
        e.printStackTrace();
    } finally {
        return sb.toString();
    }

}

Answer 1

您的代码存在一些问题。

• MCRYPT_RIJNDAEL_128 是 AES 而不是 DES。这是两种完全不同的算法。
• Cipher.getInstance("DES"); 可能默认为 Cipher.getInstance("DES/ECB/PKCS5Padding");，具体取决于您的默认 JCE 提供程序
  • ECB 和 CBC 完全不同 modes of operation。
  • PHP 用 0x00 字节填充明文，而 PKCS#5/PKCS#7 padding 添加 padding bytes which contain the number of added padding bytes（比较方法 1 和 4）。
  • 您需要使用Cipher.getInstance("AES/CBC/NoPadding");，并自行删除拖尾的 0x00 和 0x04 字节。
• 您没有使用您生成的key，而是直接使用不是密钥的密码。此外，您的密钥不应采用十六进制编码，因为您在 PHP 中也使用原始密钥。
• 盐不是静脉输液。
• 您没有从恢复的明文末尾切掉哈希来验证它。

---

在您的原始 PHP 代码中有些令人畏惧的地方：

• 如今，单遍 SHA-256 还不够好。您的密码必须非常长（50 多个随机字符）才能保证安全。使用高成本或迭代的 PBKDF2、bcrypt、scrypt，以便能够使用更短的密码。
• 似乎 IV 实际上作为 Base64 编码的字符串附加到 Base64 编码的密文中，没有填充字节。这是非常不寻常的，可能会导致误解。
• 使用 MCrypt 的零填充而不是使用 PKCS#7 padding。
• 检查明文的完整性很好，但您应该检查密文的完整性，因为encrypt-then-MAC 通常更好。 MD5也不够好。您应该至少使用 HMAC-SHA256。