c++ - 可能的内存泄漏 : new char[strlen()]

Question

这是一个相当基本的问题，我很确定我知道答案，但鉴于错误的后果是我认为我应该问的段错误。我已经按以下方式使用 strlen() 和 new char[] 运算符很长一段时间了，只是注意到一些东西引发了危险信号:

void genericCopy(char *somestring, char *someOtherString) {
    someOtherString = new char[strlen(somestring)];
    strcpy(someOtherString,somestring);
}

我的问题是，作为一个字符串应该以 null 结尾，我应该这样做吗:

void genericCopy(char *somestring, char *someOtherString) {
    someOtherString = new char[strlen(somestring)+1];
    strcpy(someOtherString,somestring);
    someOtherString[strlen(someOtherString)] = '\0';
}

到目前为止，我对第一种方法从未遇到过问题，但这并不意味着我做对了。由于 strlen() 返回的长度是不带空终止符的字符串中的字符数，因此 new 不会为 '/0' 保留空间......至少我不认为是的。

Answer 1

首先你要知道你这个函数写起来没意义，用strdup就可以了(如果您的系统可用)。

但是，是的，您需要一个额外的字节来存储 \0，因此请始终执行类似 new char[strlen(somestring)+1]; 的操作。但是，不需要手动添加\0； strcpy 已经这样做了。

你应该使用类似 Valgrind 的东西在您的代码中发现此错误和类似错误。

然而，您的代码中还有一个问题；你的代码总是会泄漏 someOtherString；它不会返回到您调用它的地方。您要么需要将方法更改为:

char *genericCopy(char *something) {
    char *copy = new char[strlen(somestring)+1];
    strcpy(copy,somestring);
    return copy;
}

然后获取拷贝如下:

copy = genericCopy(something);

或者您需要将方法更改为:

void genericCopy(char *something, char **copy) {
    *copy = new char[strlen(somestring)+1];
    strcpy(*copy,somestring);
}

并将其命名为:

genericCopy(something, &copy);

如果您将使用 C++，您也可以将方法原型(prototype)更改为:

void genericCopy(char* somestring, char*& someOtherString)

并将其命名为:

genericCopy(something, copy);

然后 someOtherString 将作为引用传递，您分配给它的新值将传播到您的方法之外。