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 将作为引用传递，并且您分配给它的新值将在您的方法之外传播。