Problem with Cipher class in javax.crypto

[vinayak.top]

Hi all,

I am facing a problem when I try to encrypt and decrypt the simple string text with the Cipher class in javax.crypto. This problem comes with some combination of plainText and Key. An example combination is given below

The sample code is given below

Cipher cipher = Cipher.getInstance("DES/ECB/NoPadding");
String key= "hellovij";
String input = "Botlaaaa";


when i perform encryption and decryption i am getting the following output

Before Encryption input is :-->Botlaaaa
After Encryption cipher is :--> ?▐V¬◄♠┴
After Decryption input is :-->▄YA♫Γ╛♣Y (here it should be Botlaaa)


Please help me in solving this issue..
Thanks in advan:mad:ce.

[masijade]

Hi all,

I am facing a problem when I try to encrypt and decrypt the simple string text with the Cipher class in javax.crypto. This problem comes with some combination of plainText and Key. An example combination is given below

The sample code is given below

Cipher cipher = Cipher.getInstance("DES/ECB/NoPadding");
String key= "hellovij";
String input = "Botlaaaa";


when i perform encryption and decryption i am getting the following output

Before Encryption input is :-->Botlaaaa
After Encryption cipher is :--> ?▐V¬◄♠┴
After Decryption input is :-->▄YA♫Γ╛♣Y

Seeing the code might help. As well as knowing how you are displaying the decoded String. And, hopefully, you are "storing" the encoded bytes as bytes, and not as a String. There is a world of difference, especially where ciphers are concerned.

(here it should be Botlaaa)

No! Really? I never would have guessed.

Please help me in solving this issue..
Thanks in advan:mad:ce.

Take those above points into account.

[vinayak.top]

Thanks for your reply...


Here is my code


import javax.crypto.Cipher;
import javax.crypto.spec.SecretKeySpec;

public class Crypto
{
private Cipher cipher;
private String stringKey;
private String input;
private String cipherText;
private SecretKeySpec key;

public Crypto() throws Exception {
cipher = Cipher.getInstance("DES/ECB/NoPadding");
stringKey = "hellovij";
input = "Botlaaaa";

}
public static void main(String[] args) throws Exception
{


Crypto crypto = new Crypto();
crypto.key = new SecretKeySpec(crypto.stringKey.getBytes(), "DES");

System.out.println("Before Encryption input is :-->" + crypto.input );
crypto.cipherText = crypto.encrypt(crypto.input);
System.out.println("After Encryption cipher is :-->" + crypto.cipherText );
crypto.input = crypto.decrypt(crypto.cipherText);
System.out.println("After Decryption input is :-->" + crypto.input );


}
public String encrypt(String str) throws Exception {
cipher.init(cipher.ENCRYPT_MODE,key);
byte outputBytes[] = cipher.doFinal(str.getBytes());
return new String(outputBytes);
}
public String decrypt(String str) throws Exception{
cipher.init(cipher.DECRYPT_MODE,key);
byte outputBytes[] = cipher.doFinal(str.getBytes());
return new String(outputBytes);
}



}


Please need your help
Thank you very much

[masijade]

Like I said, don't do this

return new String(outputBytes);

You do know that Java Strings use two-byte characters, right? And that new String(byte[]) reads those bytes as if they were in the default system encoding which probably only uses single byte characters, right? So, doing new String has just added additional bytes to your encoding. As I said, handle the encoded bytes as a nyte array, do not make a String out of it.

[vinayak.top]

Thank u very much buddy...