Мои функции шифрования/дешифрования AES не работают со случайными IVC.
Вопрос
Мне было скучно, и я написал оболочку для openSSL, чтобы выполнять шифрование AES с меньшими усилиями.Если я сделаю это так:http://pastebin.com/V1eqz4jp (ивек = 0)
Все работает нормально, но значение iec по умолчанию — все 0, что вызывает некоторые проблемы с безопасностью.Поскольку я все равно передаю данные обратно в виде строки, я подумал, почему бы не сгенерировать случайный IEC и не прикрепить его к началу, а затем убрать его обратно, когда я расшифровываю строку?Хотя почему-то это не работает.
Ну, на самом деле, это почти работает.Вроде расшифровывает середину строки, но не начало или конец:
String is: 0123456789ABCDEF0123456789ABCDEF0123456789ABCDEF0123456789ABCDEF0123456789ABCDEF
Encrypting..
���l%%1u���B!
�����`pN)�ɶ���[l�ӏ��{�Q�?�2�/�HԵ�y"�=Z�Cu����l%%1u���B!
Decrypting..
String is: �%���G*�5J�0��0123456789ABCDEF0123456789ABCDEF0123456789ABCDEF0123456789ABCDEF
Честно говоря, я понятия не имею, что происходит не так.Может быть, какая-то глупая ошибка, или, может быть, я что-то упускаю из виду AES?
Вот код:(Отредактировано с учетом решения Стива Джессопа для моей первой проблемы)
/*!
* Simple AES
* Brendan Long
* March 29, 2010
*
* Simplified encryption and decryption using OpenSSL's AES library.
* Remember to compile with -lcrypto and link against the library
* g++ (your stuff) -lcrypto simpleAes.cpp (or simpleAes.o)
*
* Implementation note: Using the default ivec (0) is not secure. For
* the full security that AES offers, use a different
* ivec each time (it does not need to be secret,
* just different.
*
* This code is released into the public domain. Yada yada..
* Read this for details: http://creativecommons.org/licenses/publicdomain/
*
* If for some reason public domain isn't good enough, you may use, alter,
* distribute or do anything else you want with this code with no restrictions.
*/
#include <openssl/aes.h>
#include <iostream>
#include <stdlib.h>
#include <time.h>
bool seed = true;
/*!
* Encrypts a string using AES with a 256 bit key
* Note: If the key is less than 32 bytes, it will be null padded.
* If the key is greater than 32 bytes, it will be truncated
* \param in The string to encrypt
* \param key The key to encrypt with
* \return The encrypted data
*/
std::string aes_encrypt(std::string in, std::string key){
// Seed the random number generator once
if(seed){
srand( (unsigned int) time(NULL));
seed = false;
}
// Generate a random ivec
unsigned char ivec[16];
for(int i=0; i<16; i++){
ivec[i] = (unsigned char) rand();
}
// Round up to AES_BLOCK_SIZE
size_t textLength = ((in.length() / AES_BLOCK_SIZE) + 1) * AES_BLOCK_SIZE;
// Always pad the key to 32 bits.. because we can
if(key.length() < 32){
key.append(32 - key.length(), '\0');
}
// Get some space ready for the output
unsigned char *output = new unsigned char[textLength];
// Generate a key
AES_KEY *aesKey = new AES_KEY;
AES_set_encrypt_key((unsigned char*)key.c_str(), 256, aesKey);
// Encrypt the data
AES_cbc_encrypt((unsigned char*)in.c_str(), output, in.length() + 1, aesKey, ivec, AES_ENCRYPT);
// Make the data into a string
std::string ret((char*) output, textLength);
// Add the ivec to the front
ret = std::string((char*)ivec, 16) + ret;
// Clean up
delete output;
delete aesKey;
return ret;
}
/*!
* Decrypts a string using AES with a 256 bit key
* Note: If the key is less than 32 bytes, it will be null padded.
* If the key is greater than 32 bytes, it will be truncated
* \param in The string to decrypt
* \param key The key to decrypt with
* \return The decrypted data
*/
std::string aes_decrypt(std::string in, std::string key){
// Get the ivec from the front
unsigned char ivec[16];
for(int i=0;i<16; i++){
ivec[i] = in[i];
}
in = in.substr(16);
// Always pad the key to 32 bits.. because we can
if(key.length() < 32){
key.append(32 - key.length(), '\0');
}
// Create some space for output
unsigned char *output = new unsigned char[in.length()];
// Generate a key
AES_KEY *aesKey = new AES_KEY;
AES_set_decrypt_key((unsigned char*)key.c_str(), 256, aesKey); // key length is in bits, so 32 * 8 = 256
// Decrypt the data
AES_cbc_encrypt((unsigned char*)in.c_str(), output, in.length(), aesKey, ivec, AES_DECRYPT);
// Make the output into a string
std::string ret((char*) output);
// Clean up
delete output;
delete aesKey;
return ret;
}
Решение 3
Мой друг разобрался в проблеме.Я делаю это:
- Сгенерировать случайное число и сохранить его в
ivec
- Зашифруйте данные с помощью
ivec
- Добавить ivec в начало выходных данных
Проблема в том, что шаг 2 меняет содержимое iec.По сути, я хранил случайные числа в начале строки.Решением было добавить это:
unsigned char ivec[16];
// set ivec to random numbers
std::string ivecString((char*) ivec, 16);
// encrypt data
return ivecString + encryptedData;
Другие советы
Вам следует сохранить ivec[16] в «выходных данных» ПЕРЕД шифрованием.Вот и все...
Еще хотелось бы добавить, что работать с char* вместо строки будет намного проще.
Эта строка неверна:
std::string ret((char*) output);
Расшифрованные данные не имеют нулевого терминатора, поскольку вы зашифровали in.length()
байты.Это объясняет мусор в конце, но не мусор в начале.Могут быть и другие проблемы.
Как правило, вы не можете обрабатывать выходные данные этапа шифрования как строку, если не выполните дополнительный шаг, например кодирование выходных данных в Base 64.Любой выходной байт может быть нулевым.