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

时间:2015-08-18 13:16:17

标签: java php encryption aes porting

我有问题转换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();
    }

}

1 个答案:

答案 0 :(得分:2)

您的代码存在一些问题。

  • MCRYPT_RIJNDAEL_128是AES而不是DES。这是两种完全不同的算法。
  • Cipher.getInstance("DES");可能默认为Cipher.getInstance("DES/ECB/PKCS5Padding");,具体取决于您的默认JCE提供程序
  • 您没有使用自己生成的key,而是直接使用非密钥的密码。此外,您的密钥不应该是十六进制编码,因为您还在PHP中使用原始密钥。
  • 盐不是IV。
  • 你没有从恢复的明文末尾切掉哈希来验证它。

在原始PHP代码中有些令人畏惧的事情:

  • 现在单次通过SHA-256还不够好。您的密码需要非常长(50个以上的随机字符)才能保证安全。使用PBKDF2,bcrypt,scrypt,成本高或迭代,以便能够使用更短的密码。
  • 似乎IV实际上附加到Base64编码的密文,就像没有填充字节的Base64编码的字符串一样。这非常不寻常,可能会导致误解。
  • 使用MCrypt的零填充而不是使用PKCS#7 padding
  • 检查明文的完整性是好的,但您应该检查密文的完整性,因为encrypt-then-MAC通常更好。 MD5也不够好。您应该至少使用HMAC-SHA256。