Question

我目前正在尝试使用以下MD5库在C中将两个字符串一起哈希： https://openwall.info/wiki/people/solar/software/public-domain-source-code/md5

但是，当我运行我的代码（它编译时没有警告）时，出现内存错误（segfault或类似错误）。

错误在哪里？谁能指出我正确的方向？

代码：

#include "md5.h"
#include <string.h>
#include <stdio.h>
#include <stdlib.h>

char* hashfunc (char* word1, char* word2){
    //concat both words
    strncat(word1,word2,100);
    MD5_CTX md5;
    MD5_Init(&md5);
    MD5_Update(&md5,word1,strlen(word1));
    unsigned char* digest;
    MD5_Final(digest,&md5);

    return digest;
}

int main(){
    char* a = "dick";
    char* b = "duenn";

    char* ret = hashfunc(a,b);
    printf("MD5 hash: %s\n",ret);
}

谢谢。

Answer 1

C的最大不便之一可能是字符串处理。这都是非常低的水平。 char*不只是一个可以修改和写入的字符串；它只是指向内存的指针。

有几种方法可以更正代码，但这是我建议的解决方法：

// hashes two words and writes the digest to output.
void hashfunc(char* word1, char* word2, char* output) {
    //concat both words
    char concat[100];
    strcpy(concat, word1);
    strcat(concat, word2);

    MD5_CTX md5;
    MD5_Init(&md5);
    MD5_Update(&md5,concat,strlen(concat));
    MD5_Final(output,&md5);
}

更改为：

使用临时工作缓冲区将单词结合在一起以进行哈希处理。
不返回值，而是选择让用户传递缓冲区以处理返回值。

第二个更改的好处是使用者可以在堆栈上使用内存，而不必总是被迫处理函数内部的内存分配。（另一种方法是分配内存并返回内存，这比较慢，必须手动释放）。

我也不完全确定您使用的MD5库如何工作，但是您可以完全避免手动串联：

void hashfunc(char* word1, char* word2, char* output) {
    MD5_CTX md5;
    MD5_Init(&md5);
    MD5_Update(&md5,word1,strlen(word1));
    MD5_Update(&md5,word2,strlen(word2));
    MD5_Final(output,&md5);
}

如果您需要一个用法示例：

char ret[100];
hashfunc(a, b, ret);

C

1 个答案: