char* message =malloc(sizeof(char))

这段代码中，使用了malloc函数动态分配了一个字节的内存空间，并将其地址赋值给了字符指针message。但是，这种分配方式是有问题的。

首先，sizeof(char)的值为1，因此这里分配的内存空间只有一个字节，这显然是不够用的。如果需要存储一个字符串，至少需要分配字符串长度加1个字节的内存空间，用于存储字符串结尾的空字符'\0'。

其次，这里没有对内存分配是否成功进行判断。如果内存分配失败，程序将会出错或崩溃。因此，在使用malloc函数分配内存空间时，应该对其返回值进行判断，以确保内存分配成功。

正确的分配内存空间的代码应该是这样的：

char* message = malloc((strlen("Hello, world!") + 1) * sizeof(char));
if (message == NULL) {
    printf("Failed to allocate memory.\n");
    return 1;
}
strcpy(message, "Hello, world!");

在这个代码中，首先使用strlen函数获取字符串"Hello, world!"的长度，然后加1个字节，分配足够的内存空间。接着对内存分配是否成功进行了判断，如果失败则输出错误信息并结束程序。最后使用strcpy函数将字符串"Hello, world!"复制到分配的内存空间中。