Question

我正在尝试为arg_struct动态分配内存。如果我只是在堆栈上分配它，它可以正常工作，但不是动态的。动态地它在主函数中打印字符串，但是当它传递给线程函数时它不起作用。关于它为什么不起作用的任何想法？

#include <stdio.h>
#include <stdlib.h>
#include <pthread.h>


struct arg_struct {
    int *arg1;
    char *str;
};

void *print_the_arguments(void *arguments)
{
    struct arg_struct *args = (struct arg_struct *)arguments;
    printf("The result is : %s\n",args->str);
    pthread_exit(NULL);
    return NULL;
}

int main()
{
    pthread_t some_thread;
    struct arg_struct *args = malloc(sizeof(struct arg_struct));
    //struct arg_struct args; // uncommenting this and args.str WORKS (prints to thread function)
    args->str = malloc(sizeof(char)*6);
    args->str = "hello";
    //args.str = "hello";
    printf("printing from main: %s\n", args->str); // Prints here but not in other function

    if (pthread_create(&some_thread, NULL, &print_the_arguments, (void *)&args) != 0) {
        printf("Uh-oh!\n");
        return -1;
    }

    return pthread_join(some_thread, NULL); /* Wait until thread is finished */

}

Answer 1

此

29     if (pthread_create(&some_thread, NULL, &print_the_arguments, (void *)&args) != 0) {

应该是

29     if (pthread_create(&some_thread, NULL, &print_the_arguments, (void *)args) != 0) {

当你动态分配args时。另外，我建议

 25     args->str = strdup("hello");

并跳过malloc。

此代码：

 24     args->str = malloc(sizeof(char)*5);
 25     args->str = "hello";

导致内存泄漏，malloc的内存泄露。也许你的意思是

 24     args->str = malloc(sizeof(char)*5);
 25     strcpy(args->str, "hello");

但这也是不正确的。 5应该是6来解释空字符在字符串的末尾。 strdup()是malloc后跟strcpy的好快捷方式。

分配struct heap vs stack：Pthread_create

1 个答案: