2

最初我想将此uint8_t数组转换为charc 中的数组。我一直在尝试解决这个问题。我的第一个替代解决方案是将另一个类型值复制到临时值，将 tmp 值复制到可写字符，然后从内存中删除 tmp 值。顺便说一句，这用于伴随 blake 散列函数。这是我的代码片段：

char * bl(char *input)
{
    uint8_t  output[64];
    char msg[]= "";
    char *tmp;

    int dInt;

    memset(output,0,64);
    tmp = (char*) malloc(64);
    if (!tmp){
            exit( 1);
    }

    dInt = strlen(input);

    if (dInt > 0xffff){
            exit( 1);
    }
    uint8_t data[dInt];

    memset(data,0, dInt);
    strlcpy(data,input,dInt);
    uint64_t dLen =dInt;
    blake512_hash(output, data,dLen);

    int k;
    for (k=0;k<64;k++){
            tmp[k] = output[k];  //does this "copy" is buggy code?
    }

    memcpy(msg, tmp,64);
    //so here I can to delete tmp value
    // I dont want there were left unused value in memory
    // delete tmp;  
    free(tmp);

    return  msg;
}

我认为上面的代码仍然没有效率，那么您的意见，提示和修复是什么？之前非常感谢！

4

2 回答 2

6

首先，您永远不应该返回指向局部变量的指针，因为该变量将在函数退出时被销毁。您可能应该希望将输出数组传递给bl函数并使用它来输出字符串。

对于大多数情况（如果 uint8_t IS char，通常是这种情况），memcpy(msg, output, 64)应该就足够了。如果您想对此严格要求（坦率地说，如果您一直期望数组作为输出，blake512_hash则不应该首先返回数组），您可以简单地调用您的 for 循环并删除。uint8_tcharmsg[k] = (char)tmp[k]memcpy

于 2012-04-18T02:18:40.097 回答
0

这里有点不对劲。

dInt = strlen(input) + 1; // dInt is the size of the string including the terminating '\0'.

strlcpy确实使用大小，而不是 strlen。

味精=tmp；而不是释放 tmp。因为 msg 是 const char* "" （用 C++ 术语）。

于 2012-04-18T02:23:36.467 回答