*** glibc detected *** free(): invalid pointer

Question

У меня есть следующий код, который вызывает ошибку *** glibc detected *** free(): invalid pointer всякий раз, когда я запускаю код.

main.h:

#ifndef PTHREAD_CALC_H_ 
#define PTHREAD_CALC_H_ 

void* task(void*); 

#endif 

main.cxx:

#include <stdio.h> 
#include <stdlib.h> 
#include <sys/time.h> 
#include <pthread.h> 
#include "main.h" 

int main(int argc, char* argv[]) { 

    pthread_t *threads = (pthread_t*)malloc(sizeof(pthread_t)*2); 
    double *temp; 

    double sum = 0.0; 
    for (int j = 0; j < 2; j++) { 
     pthread_create(&(threads[j]), NULL, task, NULL); 
    } 

    for (int j = 0; j < 2; j++) { 
     pthread_join(threads[j], (void**)(&temp)); 
     sum += *temp; 
    } 

    free(threads); 
    free(temp); 

    return 0; 
} 

void* task(void *data) { 
    double sum = 5; 
    pthread_exit((void*)&sum); 
    return NULL; 
} 

Я с трудом определяют, что является причиной ошибки. Любая помощь очень ценится. Если я могу предоставить что-нибудь еще, чтобы помочь выявить проблему, пожалуйста, дайте мне знать.

Спасибо

Редактировать

Ради завершенности, здесь результирующий код, который выполняется, как ожидалось:

main.cxx:

#include <stdio.h> 
#include <stdlib.h> 
#include <sys/time.h> 
#include <pthread.h> 
#include "main.h" 

int main(int argc, char* argv[]) { 

    pthread_t *threads = (pthread_t*)malloc(sizeof(pthread_t)*2); 
    double *temp; 

    double sum = 0.0; 
    for (int j = 0; j < 2; j++) { 
     pthread_create(&(threads[j]), NULL, task, NULL); 
    } 

    for (int j = 0; j < 2; j++) { 
     pthread_join(threads[j], (void**)&temp); 
     sum += temp; 
     delete temp; 
    } 

    free(threads); 
    return 0; 
} 

void* task(void *data) { 
    double* sum = new double; 
    *sum = 5.0; 
    pthread_exit(static_cast<void*>(sum)); 
} 

Answer 1

В настоящее время задача вашего потока возвращает некоторое значение в стеке потока. Когда нить заканчивается, нет никакой гарантии, что * temp указывает на что-то действительное.

Таким образом, после этого вызова

pthread_join(threads[j], (void**)(&temp)); 

Темп указывает на старое место суммы в потоке, но если поток заканчивает он больше не существует. И использование этого приведет к неопределенному поведению.

, но позже вы освобождаете темп, который указывает на двойник на другом стеке, но есть свободные, так как распределения стека бесплатны автоматически, когда они выходят за рамки.

free(temp); 

хочет вы можете сделать, это:

void* task(void *data) { 
    double* sum = new double; 
    *sum = 5; 
    pthread_exit(static_cast<void*>(sum)); 
} 

, а затем в основном после присоединения нити

delete temp; 

Answer 2

В своем коде вы объявляете :

double *temp; 

Вы никогда не malloc по адресу, но позже вы его бесплатно. Это приведет к ошибке. Удалите свободный (temp), и он должен работать. На самом деле, хотя вы ввели новые ошибки при разыменовании * temp без хранения.