CODEWITHSUNDEEP
javaencryptionaessalt-cryptographycrypt
Prabu
Prabu

Reputation: 3728

Java AES encryption issue

Every time the encryption values changed by using AES, let anyone investigate the below code and let me know the issue

code:

private static final String secretKeys = "58BA833E57A51CBF9BF8BAB696BF9"

public static String encrypt() throws Exception {
        byte[] salt = new byte[16];
        SecretKeyFactory factory = SecretKeyFactory.getInstance("PBKDF2WithHmacSHA256");
        PBEKeySpec pbeKeySpec = new PBEKeySpec(secretKeys.getChars(),salt,1000, 256);
        Key secretKey = factory.generateSecret(pbeKeySpec);
        byte[] key = new byte[32];
        byte[] iv = new byte[16];
        SecretKeySpec secret = new SecretKeySpec(key, "AES");
        Cipher cipher = Cipher.getInstance("AES/CBC/PKCS5Padding");
        cipher.init(Cipher.ENCRYPT_MODE, secret);
        byte[] result = cipher.doFinal("welcome".getBytes("UTF-8"));
        String s = Base64.getEncoder().encodeToString(result);
        return s
        }

Output first time I got the below-encrypted string

CZRIP35M4CnJtuDQ6YpmaQ==

The second time I got the below-encrypted string

/fylTjohAZDsnCaHhiZo3A==

I have three questions:

  1. why the encrypted string not a constant?

  2. how can I set the Blocksize? ( AES.BlockSize = 128;)

  3. How can I set the padding mode? (AES.Padding = PaddingMode.PKCS7;)

Upvotes: 1

Views: 330

Answers (3)

Daniel
Daniel

Reputation: 4211

Only adding to the answer provided by @micker, you need to invoke another version of Cipher.init(); one that takes the IV into account:

...
byte[] iv = new byte[16];
IvParameterSpec ivSpec = new IvParameterSpec(iv); // <= Wrap your IV bytes here.
SecretKeySpec secret = new SecretKeySpec(key, "AES");
Cipher cipher = Cipher.getInstance("AES/CBC/PKCS5Padding");
cipher.init(Cipher.ENCRYPT_MODE, secret, ivSpec); // <= Add IV here.
...

That being said, the implementation suffers from a slew of other issues (key being all zeroes, IV being all zeroes, first 4 line don't do anything for you (as @JBNizet pointed out)). I hope you are only using it to study how Java's encryption mechanics works.

Upvotes: 0

Andrei Sfat
Andrei Sfat

Reputation: 8606

  1. For the first question, @Freiheit already answered this. Long story short, based on the iv (initilization vector) which acts as a salt and will be different for each encryption. Having that said, encrypting the same plain text will result in different encrypted text, but the decryption (if necessary) will result back into the same plain text. IV is helpful to make the encryption predictable. Having stored the same password for 2 different users in a database will have different values, but will be the same password.

  2. With the current cipher configured, you already have 128 block size. You can read more about the different cypher transformation here. You can also find more information of the block sizes for different algorithms here

  3. You just need to change the Cipher.getInstance() to AES/CBC/PKCS7Padding

Upvotes: 3

micker
micker

Reputation: 928

1) the encrypted text is always different because the Cipher initialization is providing it's own IV since you are not providing one. You need to provide the IV you've "computed" in order to have a consistent output. Remember you never want to use an IV more than once for whatever this code is ultimately intended to do.

2) The keysize can be 128, 192 or 256 but the blocksize is always 128.

3) Java only provides PKCS5, but there is no difference in the implementation for AES. see what-is-the-difference-between-pkcs5-padding-and-pkcs7-padding

As was already pointed out there are several problems with the code provided such as the first lines not actually doing anything and the key and iv both being uninitialized. I would additionally suggest you use SecureRandom to initialize your key and iv. If you plan on using only a single AES key, this can be computed once and placed in the code or configuration file instead of running PBKDF2 every time.

Upvotes: 1

Related Questions