1

我正在检查源代码并看到这个for循环：

for (int i=0; i < result1.length(); i++) {
    unsigned char *buff = ascii_to_utf8((unsigned char)result1.at(i));
    result.append((char*)buff);
    free (buff);
}

已改为：

for (int i=0; i < result1.length(); i++)
    result.append((char*)ascii_to_utf8((unsigned char)result1.at(i)));

我能看到的唯一区别是在第二个代码中没有中间变量。

我的问题是，我看不到的两个代码片段之间有区别吗？freeseconde 代码中没有任何内容吗？

------------ 编辑 ------------ 这是 ascii_to_utf8 的源代码：

unsigned char* InvoiceXML::ascii_to_utf8(unsigned char c)
{
    unsigned char *out;

    if(c < 128)
    {
        out = (unsigned char *)calloc(2, sizeof(char));
        out[0] = c;
        out[1] = '\0';
    }
    else
    {
        out = (unsigned char *)calloc(3, sizeof(char));
        out[0] = (c >> 6) | 0xC0;
        out[1] = (c & 0x3F) | 0x80;
        out[2] = '\0';
    }

    return out; 
}
4

3 回答 3

3

这取决于 ascii_to_utf8 做什么；它几乎肯定会分配内存，在这种情况下，您确实需要那里的“免费”。所以第二个片段会泄漏内存。

于 2012-12-06T15:26:52.387 回答
1

如果ascii_to_utf8更改为不分配内存（例如，通过对结果使用静态 char 数组），则调用者不应删除返回的值。

于 2012-12-06T15:44:38.867 回答
1

如果ascii_to_utf8确实分配了内存，那么：

第一个片段：如果您的result对象没有在append函数中复制，那显然是错误的——您将存储指向已释放值的指针。

第二个片段：如果您的result对象确实在函数中复制append，那是错误的；如果不是，那么您需要在free()某处获取该数据，但此代码段是正确的。

此外，如果`ascii_to_utf8确实通过 分配内存new，则应delete改为使用。

否则（ascii_to_utf8不分配内存）：

第一个片段是错误的 - 您不应free静态分配内存。

第二个片段是正确的。

于 2012-12-06T15:44:38.993 回答