循环中的连接无符号char *

Question

我正在使用库OpenSSL通过AE进行加密。由于AES是一个块密码，因此我需要将数据分成16个字节。因此，如果我想恢复消息，我需要在程序结束时团结块。

这是我的代码：

#include <string.h>
#include <stdlib.h>
#include <stdio.h>
#include <openssl/aes.h>

char key[] = "secretkey123";

int main() {
    unsigned char text_slice[128];
    unsigned char enc_slice[128];
    unsigned char dec_slice[128];
    
    unsigned char in[160];
    
    unsigned char enc_out[160] = "";
    unsigned char dec_out[160] = "";
    
    int i;
    int k = 10;
    for (i = 0; i < 16 * k; i++) {
        in[i] = 'A' + (rand() % 26);
    }
    in[160] = '\0';
    printf("IN:%s\n", in);
    
    AES_KEY enc_key, dec_key;
    
    AES_set_encrypt_key(key, 128, &enc_key);
    
    AES_set_decrypt_key(key, 128, &dec_key);
    
    for (i = 0; i < k; i++) {
        text_slice[0] = '\0';
        enc_slice[0] = "\0";
        dec_slice[0] = "\0";
        memcpy(&text_slice[0], &in[15 * i], 15);
        text_slice[16] = "\0";
        printf("TEXT SLICE: %s \n", text_slice);
        AES_encrypt(text_slice, enc_slice, &enc_key);
        memcpy(&enc_out[16 * i], &enc_slice[0], 16);
    }
    printf("ENC:%s\n", enc_out);
    for (i = 0; i < k; i++) {
        text_slice[0] = '\0';
        enc_slice[0] = "\0";
        dec_slice[0] = "\0";
        memcpy(enc_slice, &enc_out[16 * i], 16);
        enc_slice[16] = "\0";
    
        AES_decrypt(enc_slice, dec_slice, &dec_key);
        printf("Dec slice:%s \n", dec_slice);
        memcpy(&dec_out[16 * i], &dec_slice[0], 16);
    }
    printf("DEC OUT:%s\n", dec_out);
    
    return 0;
}

程序的输出如下：

IN:NWLRBBMQBHCDARZOWKKYHIDDQSCDXRJMOWFRXSJYBLDBEFSARCBYNECDYGGXXPKLORELLNMPAPQFWKHOPKMCOQHNWNKUEWHSQMGBBUQCLJJIVSWMDKQTBXIXMVTRRBLJPTNSNFWZQFJMAFADRRWSOFSBCNUVQHFF
TEXT SLICE: NWLRBBMQBHCDARZ 
TEXT SLICE: OWKKYHIDDQSCDXR 
TEXT SLICE: JMOWFRXSJYBLDBE 
TEXT SLICE: FSARCBYNECDYGGX 
TEXT SLICE: XPKLORELLNMPAPQ 
TEXT SLICE: FWKHOPKMCOQHNWN 
TEXT SLICE: KUEWHSQMGBBUQCL 
TEXT SLICE: JJIVSWMDKQTBXIX 
TEXT SLICE: MVTRRBLJPTNSNFW 
TEXT SLICE: ZQFJMAFADRRWSOF 
ENC:j�Q���

              

Answer 1

钥匙必须至少具有16个字节，因为您将许多位等于128英寸的位：

AES_set_encrypt_key(key, 128, &enc_key);
AES_set_decrypt_key(key, 128, &dec_key);

您在这里的行为不确定：

    in[160] = '\0';

这些行没有道理：

    enc_slice[0] = "\0";
    dec_slice[0] = "\0";
    text_slice[16] = "\0";
    enc_slice[16] = "\0";

为什么您要编码15个字符而不是16个字符的块？

    memcpy(&text_slice[0], &in[15 * i], 15);

这是一个修改版本：

#include <string.h>
#include <stdlib.h>
#include <stdio.h>
#include <openssl/aes.h>

char key[] = "secretkey1234567";

int main() {
    unsigned char text_slice[128];
    unsigned char enc_slice[128];
    unsigned char dec_slice[128];
    unsigned char in[160];
    unsigned char enc_out[160];
    unsigned char dec_out[160];
    
    int i, k = 10;
    for (i = 0; i < 16 * k; i++) {
        in[i] = 'A' + (rand() % 26);
    }
    printf("IN: \"%.160s\"\n", (char *)in);
    
    AES_KEY enc_key, dec_key;
    AES_set_encrypt_key(key, 128, &enc_key);
    AES_set_decrypt_key(key, 128, &dec_key);
    
    for (i = 0; i < k; i++) {
        memcpy(text_slice, &in[16 * i], 16);
        printf("TEXT SLICE: %.16s\n", (char *)text_slice);
        AES_encrypt(text_slice, enc_slice, &enc_key);
        memcpy(&enc_out[16 * i], enc_slice, 16);
    }
    printf("ENC:");
    for (i = 0; i < 16 * k; i++) {
       printf(" %02X\n", enc_out[i]);
    }
    printf("\n");
    for (i = 0; i < k; i++) {
        memcpy(enc_slice, &enc_out[16 * i], 16);
        AES_decrypt(enc_slice, dec_slice, &dec_key);
        printf("Dec slice: %.16s \n", (char *)dec_slice);
        memcpy(&dec_out[16 * i], dec_slice, 16);
    }
    printf("DEC OUT: \"%.160s\"\n", (char *)dec_out);
    return 0;
}

Answer 2

该代码中有很多问题。

AES_SET_ENCRYPT_KEY和aes_set_decrypt_key不是密钥推导函数，第二个参数应该代表第一个参数（userkeykey）的位中的长度。

如果您提供一个任意的字符串常数，例如“ secretKey123”，长度不是128，则可能会导致内存损坏或创建非常不安全的上下文以进行加密。

通常，对于基于用户密码的加密，我们使用键推导功能从任何密码生成一个密钥（例如pbkdf2或scrypt）。

该行也很不好：

in[160] = '\0';

由于缓冲区的大小为160，因此最后一个索引应为160-1。如果您在索引160上写作，则实际上是溢出的。

另外，在编写C代码时，您必须真正注意到单引号和双引号之间的区别。

这很不好：

enc_slice[0] = "\0";
dec_slice[0] = "\0";

如果您使用双引号，则实际上并未在enc_slice和dec_slice的第一个索引上写下值0。您正在编写一个空的仅读取常数字符串的地址。

另外：

unsigned char enc_out[160] = "";
unsigned char dec_out[160] = "";

如果您想拥有空的缓冲区，则应该做：

unsigned char enc_out[160] = {0};
unsigned char dec_out[160] = {0};

希望这是改进代码并使其正常工作的好开始。