'strcpy'与“malloc”？

Question

执行以下操作安全吗？

#include <stdio.h>
#include <malloc.h>
#include <string.h>

int main(void)
{
    char* msg;

    strcpy(msg, "Hello World!!!");  //<---------

    printf("%s\n", msg);

    return 0;
}

或者应该使用以下内容？

char* msg = (char*)malloc(sizeof(char) * 15);

Answer 1

strdup 为您执行 malloc 和 strcpy

char *msg = strdup("hello world");

Answer 2

您的原始代码没有分配 msg。尝试对其进行 strcpy 会很糟糕。在 strcpy 之前，您需要分配一些空间。您可以按照您的建议使用 malloc 或像这样在堆栈上分配空间：

char msg[15];

如果您 malloc 内存，您应该记住在某个时候释放它。如果在堆栈上分配内存，当超出范围（例如函数退出）时，内存将自动返回到堆栈。在这两种情况下，您都需要小心分配足够的空间，以便能够将最长的字符串复制到其中。您可能需要查看 strncpy 以避免数组溢出。

Answer 3

第一个版本不安全。并且，msg 应该指向“Hello World!!!”的有效内存位置。被复制。

char* msg = (char*)malloc(sizeof(char) * 15);
strcpy(msg, "Hello World!!!");

Answer 4

使用：

#define MYSTRDUP(str,lit) strcpy(str = malloc(strlen(lit)+1), lit)

现在它很简单并且符合标准：

char *s;
MYSTRDUP(s, "foo bar");

Answer 5

 char* msg;
 strcpy(msg, "Hello World!!!");  //<---------Ewwwww
 printf("%s\n", msg); 

这是UB。没有第二个想法。 msg 是一个野指针，尝试取消引用它可能会导致您的实现出现段错误。

msg 指向一个足够大的有效内存位置以容纳 “Hello World”。

尝试

char* msg = malloc(15);
strcpy(msg, "Hello World!!!");

或

char msg[20]; 
strcpy(msg, "Hello World!!!");

Answer 6

您需要分配空间。在strcpy 之前使用malloc。