Java AESと自分のキーの使用

88

自分のキーでAESを使用して文字列を暗号化したい。しかし、キーのビット長に問題があります。私のコードをレビューして、修正/変更する必要があるものを確認できますか？

public static void main(String[] args) throws Exception {
    String username = "bob@google.org";
    String password = "Password1";
    String secretID = "BlahBlahBlah";
    String SALT2 = "deliciously salty";

    // Get the Key
    byte[] key = (SALT2 + username + password).getBytes();
    System.out.println((SALT2 + username + password).getBytes().length);

    // Need to pad key for AES
    // TODO: Best way?

    // Generate the secret key specs.
    SecretKeySpec secretKeySpec = new SecretKeySpec(key, "AES");

    // Instantiate the cipher
    Cipher cipher = Cipher.getInstance("AES");
    cipher.init(Cipher.ENCRYPT_MODE, secretKeySpec);

    byte[] encrypted = cipher.doFinal((secrectID).getBytes());
    System.out.println("encrypted string: " + asHex(encrypted));

    cipher.init(Cipher.DECRYPT_MODE, secretKeySpec);
    byte[] original = cipher.doFinal(encrypted);
    String originalString = new String(original);
    System.out.println("Original string: " + originalString + "\nOriginal string (Hex): " + asHex(original));
}

現在、「無効なAESキー長：86バイト」という例外が発生します。キーをパッドする必要がありますか？どうすればよいですか？

また、ECBまたはCBCに何かを設定する必要がありますか？

ありがとう

java  key  aes 
— バーニーペレス
ソース
6
— ジョアン・ポルテラ
16
ハハ面白い。実際にはランダムなソルトがありますが、質問をより明確にするためにコードをクリーンアップしました。そのため、変数の名前はSALT2です。しかし、これと同じ問題に遭遇し、コードをコピー/貼り付けするのが好きな他の人にとっては良いリファレンスです。
— バーニーペレス

回答:

125

編集：

コメントに書かれているように、古いコードは「ベストプラクティス」ではありません。反復回数の多いPBKDF2などのキー生成アルゴリズムを使用する必要があります。また、少なくとも部分的に非静的（「アイデンティティー」ごとに排他的な意味）のソルトを使用する必要があります。可能であれば、ランダムに生成して暗号文と一緒に保存します。

    SecureRandom sr = SecureRandom.getInstanceStrong();
    byte[] salt = new byte[16];
    sr.nextBytes(salt);

    PBEKeySpec spec = new PBEKeySpec(password.toCharArray(), salt, 1000, 128 * 8);
    SecretKey key = SecretKeyFactory.getInstance("PBKDF2WithHmacSHA1").generateSecret(spec);
    Cipher aes = Cipher.getInstance("AES");
    aes.init(Cipher.ENCRYPT_MODE, key);

===========

古い答え

SHA-1を使用してキーからハッシュを生成し、結果を128ビット（16バイト）にトリミングする必要があります。

さらに、文字列からバイト配列を生成しないでください 、プラットフォームのデフォルトのCharsetを使用 getBytes（）。したがって、パスワード "blaöä"は、異なるプラットフォームでは異なるバイト配列になります。

byte[] key = (SALT2 + username + password).getBytes("UTF-8");
MessageDigest sha = MessageDigest.getInstance("SHA-1");
key = sha.digest(key);
key = Arrays.copyOf(key, 16); // use only first 128 bit

SecretKeySpec secretKeySpec = new SecretKeySpec(key, "AES");

編集：キーサイズとして256ビットが必要な場合は、「Java暗号化拡張機能（JCE）無制限強度管轄ポリシーファイル」のOracleダウンロードリンクをダウンロードする必要があります。ハッシュとしてSHA-256を使用し、。 Arrays.copyOf行ます。「ECB」はデフォルトの暗号モードであり、「PKCS5Padding」はデフォルトのパディングです。次の形式を使用して、Cipher.getInstance文字列を通じて異なる暗号モードとパディングモードを使用できます： "Cipher / Mode / Padding"

CTSおよびPKCS5Paddingを使用するAESの場合、文字列は「AES / CTS / PKCS5Padding」です。

— mknjc
ソース
これは機能しますが、パスワードをハッシュ化し、最初の数ビットのみを使用します。これを行うためのより良い方法はありませんか？
— バーニーペレス
4
AESには128/192/256ビットの鍵が必要なので、鍵を生成するためのより良い方法はありません。キーをハッシュせず、入力をトリミングするだけの場合、最初の16/24/32バイトのみが使用されます。したがって、ハッシュを生成することが唯一の合理的な方法です。
— mknjc 2010
13
この回答は適切な鍵導出関数を使用していないため、本来のセキュリティほど安全ではないことに注意してください。少し古くなった鍵導出関数については、他の回答を参照してください-残念ながらまだ静的ソルトです。
— Maarten Bodewes 2013年
2
この回答は非常に悪い習慣であるため、削除することをお勧めします。適切な鍵導出関数を使用する必要があります-少なくともPBKDF2。
— ボリスザスパイダー
1
はい、マーティンが数年前に言ったように、答えは非常に悪いです。暗号とキー導出関数
— kelalaka
14

KeyGeneratorを使用してキーを生成する必要があります。

AESキーの長さは、使用する暗号に応じて128、192、および256ビットです。

こちらのチュートリアルをご覧ください

これはパスワードベースの暗号化のコードです。これはSystem.inから入力されたパスワードです。必要に応じて、保存されているパスワードを使用するように変更できます。 

        PBEKeySpec pbeKeySpec;
        PBEParameterSpec pbeParamSpec;
        SecretKeyFactory keyFac;

        // Salt
        byte[] salt = {
            (byte)0xc7, (byte)0x73, (byte)0x21, (byte)0x8c,
            (byte)0x7e, (byte)0xc8, (byte)0xee, (byte)0x99
        };

        // Iteration count
        int count = 20;

        // Create PBE parameter set
        pbeParamSpec = new PBEParameterSpec(salt, count);

        // Prompt user for encryption password.
        // Collect user password as char array (using the
        // "readPassword" method from above), and convert
        // it into a SecretKey object, using a PBE key
        // factory.
        System.out.print("Enter encryption password:  ");
        System.out.flush();
        pbeKeySpec = new PBEKeySpec(readPassword(System.in));
        keyFac = SecretKeyFactory.getInstance("PBEWithMD5AndDES");
        SecretKey pbeKey = keyFac.generateSecret(pbeKeySpec);

        // Create PBE Cipher
        Cipher pbeCipher = Cipher.getInstance("PBEWithMD5AndDES");

        // Initialize PBE Cipher with key and parameters
        pbeCipher.init(Cipher.ENCRYPT_MODE, pbeKey, pbeParamSpec);

        // Our cleartext
        byte[] cleartext = "This is another example".getBytes();

        // Encrypt the cleartext
        byte[] ciphertext = pbeCipher.doFinal(cleartext);
— ケイボッシュ
ソース
3
KeyGeneratorを使用して、パスワードでキーを生成するにはどうすればよいですか？パスワードに基づいて同じキーを生成したいのですが。後で文字列を復号化できます。
— バーニーペレス
あなたの話はAESではなくパスワードベースの暗号化です。PBEのサンプルプログラムで回答を更新しました
— Keibosh
5
代わりにPBEKDF2キージェネレーターを使用してみてくださいSecretKeyFactory。最新の暗号化には、文字列 "PBKDF2WithHmacSHA1"を使用してください。
— Maarten Bodewes 2013年
12
実際、この回答で使用されているすべての暗号化プリミティブは、確かに時代遅れ、MD5およびDESです。注意してください。
— Maarten Bodewes 2013年
MD5とDESは脆弱な暗号スイートであり、回避する必要があります
— atom88
6 
import java.security.Key;
import javax.crypto.Cipher;
import javax.crypto.spec.SecretKeySpec;
import sun.misc.*;
import java.io.BufferedReader;
import java.io.FileReader;

public class AESFile 
{
private static String algorithm = "AES";
private static byte[] keyValue=new byte[] {'0','2','3','4','5','6','7','8','9','1','2','3','4','5','6','7'};// your key

    // Performs Encryption
    public static String encrypt(String plainText) throws Exception 
    {
            Key key = generateKey();
            Cipher chiper = Cipher.getInstance(algorithm);
            chiper.init(Cipher.ENCRYPT_MODE, key);
            byte[] encVal = chiper.doFinal(plainText.getBytes());
            String encryptedValue = new BASE64Encoder().encode(encVal);
            return encryptedValue;
    }

    // Performs decryption
    public static String decrypt(String encryptedText) throws Exception 
    {
            // generate key 
            Key key = generateKey();
            Cipher chiper = Cipher.getInstance(algorithm);
            chiper.init(Cipher.DECRYPT_MODE, key);
            byte[] decordedValue = new BASE64Decoder().decodeBuffer(encryptedText);
            byte[] decValue = chiper.doFinal(decordedValue);
            String decryptedValue = new String(decValue);
            return decryptedValue;
    }

//generateKey() is used to generate a secret key for AES algorithm
    private static Key generateKey() throws Exception 
    {
            Key key = new SecretKeySpec(keyValue, algorithm);
            return key;
    }

    // performs encryption & decryption 
    public static void main(String[] args) throws Exception 
    {
        FileReader file = new FileReader("C://myprograms//plaintext.txt");
        BufferedReader reader = new BufferedReader(file);
        String text = "";
        String line = reader.readLine();
    while(line!= null)
        {
            text += line;
    line = reader.readLine();
        }
        reader.close();
    System.out.println(text);

            String plainText = text;
            String encryptedText = AESFile.encrypt(plainText);
            String decryptedText = AESFile.decrypt(encryptedText);

            System.out.println("Plain Text : " + plainText);
            System.out.println("Encrypted Text : " + encryptedText);
            System.out.println("Decrypted Text : " + decryptedText);
    }
}
— シャンカールマーシー
ソース
5
多分いくつかのより多くの説明文を追加します。
— DaGardner、2013
質問、keyValueバイト配列でを持つことの意味は何ですか？なぜそれがキーを作るのに使われているのを見ますか？SecretKey代わりにlikeを使用して何かを行うことはできますか？もしそうなら、どうですか？
— オースティン
@Mandrek、「plaintext.txt」ファイルの内容は暗号化されます。上記のロジックは、FileReaderコンストラクターの引数として読み取られるファイル内のデータ/メッセージを暗号化します。
— Shankar Murthy、
2

これは機能します。

public class CryptoUtils {

    private  final String TRANSFORMATION = "AES";
    private  final String encodekey = "1234543444555666";
    public  String encrypt(String inputFile)
            throws CryptoException {
        return doEncrypt(encodekey, inputFile);
    }


    public  String decrypt(String input)
            throws CryptoException {
    // return  doCrypto(Cipher.DECRYPT_MODE, key, inputFile);
    return doDecrypt(encodekey,input);
    }

    private  String doEncrypt(String encodekey, String inputStr)   throws CryptoException {
        try {

            Cipher cipher = Cipher.getInstance(TRANSFORMATION);

            byte[] key = encodekey.getBytes("UTF-8");
            MessageDigest sha = MessageDigest.getInstance("SHA-1");
            key = sha.digest(key);
            key = Arrays.copyOf(key, 16); // use only first 128 bit

            SecretKeySpec secretKeySpec = new SecretKeySpec(key, "AES");

            cipher.init(Cipher.ENCRYPT_MODE, secretKeySpec);

            byte[] inputBytes = inputStr.getBytes();     
            byte[] outputBytes = cipher.doFinal(inputBytes);

            return Base64Utils.encodeToString(outputBytes);

        } catch (NoSuchPaddingException | NoSuchAlgorithmException
                | InvalidKeyException | BadPaddingException
                | IllegalBlockSizeException | IOException ex) {
            throw new CryptoException("Error encrypting/decrypting file", ex);
       }
     }


    public  String doDecrypt(String encodekey,String encrptedStr) { 
          try {     

              Cipher dcipher = Cipher.getInstance(TRANSFORMATION);
              dcipher = Cipher.getInstance("AES");
              byte[] key = encodekey.getBytes("UTF-8");
              MessageDigest sha = MessageDigest.getInstance("SHA-1");
              key = sha.digest(key);
              key = Arrays.copyOf(key, 16); // use only first 128 bit

              SecretKeySpec secretKeySpec = new SecretKeySpec(key, "AES");

              dcipher.init(Cipher.DECRYPT_MODE, secretKeySpec);
            // decode with base64 to get bytes

              byte[] dec = Base64Utils.decode(encrptedStr.getBytes());  
              byte[] utf8 = dcipher.doFinal(dec);

              // create new string based on the specified charset
              return new String(utf8, "UTF8");

          } catch (Exception e) {

            e.printStackTrace();

          }
      return null;
      }
 }
— タラン
ソース
2

MD5、AES、パディングなし

import static javax.crypto.Cipher.DECRYPT_MODE;
import static javax.crypto.Cipher.ENCRYPT_MODE;
import static org.apache.commons.io.Charsets.UTF_8;
import java.security.InvalidKeyException;
import java.security.Key;
import java.security.MessageDigest;
import java.security.NoSuchAlgorithmException;
import java.util.Base64;
import javax.crypto.BadPaddingException;
import javax.crypto.Cipher;
import javax.crypto.IllegalBlockSizeException;
import javax.crypto.NoSuchPaddingException;
import javax.crypto.spec.SecretKeySpec;

public class PasswordUtils {

    private PasswordUtils() {}

    public static String encrypt(String text, String pass) {
        try {
            MessageDigest messageDigest = MessageDigest.getInstance("MD5");
            Key key = new SecretKeySpec(messageDigest.digest(pass.getBytes(UTF_8)), "AES");
            Cipher cipher = Cipher.getInstance("AES");
            cipher.init(ENCRYPT_MODE, key);

            byte[] encrypted = cipher.doFinal(text.getBytes(UTF_8));
            byte[] encoded = Base64.getEncoder().encode(encrypted);
            return new String(encoded, UTF_8);

        } catch (NoSuchAlgorithmException | NoSuchPaddingException | InvalidKeyException | IllegalBlockSizeException | BadPaddingException e) {
            throw new RuntimeException("Cannot encrypt", e);
        }
    }

    public static String decrypt(String text, String pass) {
        try {
            MessageDigest messageDigest = MessageDigest.getInstance("MD5");
            Key key = new SecretKeySpec(messageDigest.digest(pass.getBytes(UTF_8)), "AES");
            Cipher cipher = Cipher.getInstance("AES");
            cipher.init(DECRYPT_MODE, key);

            byte[] decoded = Base64.getDecoder().decode(text.getBytes(UTF_8));
            byte[] decrypted = cipher.doFinal(decoded);
            return new String(decrypted, UTF_8);

        } catch (NoSuchAlgorithmException | NoSuchPaddingException | InvalidKeyException | IllegalBlockSizeException | BadPaddingException e) {
            throw new RuntimeException("Cannot decrypt", e);
        }
    }
}
— マイク
ソース
Angular（ionic 4）でSecretKeySpecのような安全な鍵を作成する方法。
— Nitin Karale
0 
    byte[] seed = (SALT2 + username + password).getBytes();
    SecureRandom random = new SecureRandom(seed);
    KeyGenerator generator;
    generator = KeyGenerator.getInstance("AES");
    generator.init(random);
    generator.init(256);
    Key keyObj = generator.generateKey();
— Sonnykwe
ソース
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.