bug

[jyxjrh]

cypher_t* plain = (cypher_t*)malloc(data_in->len_data);
这个解密的地方，内存会越界

[daydaup]

你好， 请问你是？ | | From：张隆基 邮箱：patekz@163.com | 签名由 网易邮箱大师 定制 On 02/23/2019 14:37, jyxjrh wrote: cypher_t* plain = (cypher_t*)malloc(data_in->len_data); 这个解密的地方，明显有问题啊 — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

[jyxjrh]

哦，我是网上找加密算法找到这里的。下了之后看了下，感觉这里有问题。
aes_cbc.c 这个文件的第59行：cypher_t* aes_cbc_decrypt(uint8_t* key, cypher_t* data_in)
{
cypher_t* cypher_padding = block_padding(data_in);
cypher_t* plain = (cypher_t*)malloc(sizeof(uint8_t)+data_in->len_data); // 这里分配的大小应该加上sizeof(uint8_t)

目前的写法，后面会写越界。

[daydaup]

那你能描述下你觉得是什么问题？ | | From：张隆基 邮箱：patekz@163.com | 签名由 网易邮箱大师 定制 On 02/24/2019 14:51, jyxjrh wrote: 哦，我是网上找加密算法找到这里的。下了之后看了下，感觉这里有问题。 aes_cbc.c 这个文件的第59行： — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

[jyxjrh]

aes_cbc.c文件第59行这里。cypher_t* plain = (cypher_t*)malloc(sizeof(uint8_t)+data_in->len_data); // 这里分配的大小应该加上sizeof(uint8_t)
这个结构里，有len_data和data两部分，但是只分配了len_data的长度，后面使用data的时候会写越界。

[daydaup]

cypher_t* plain = (cypher_t*)malloc(sizeof(uint8_t) +data_in->len_data); 这样，你在博客下面评论下改过来吧。 | | From：张隆基 邮箱：patekz@163.com | 签名由 网易邮箱大师 定制 On 02/24/2019 15:27, jyxjrh wrote: aes_cbc.c文件第59行这里。cypher_t* plain = (cypher_t*)malloc(data_in->len_data); // 这里分配的大小应该加上sizeof(uint8_t) — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.