C++ void pointers

Question

pthread_mutex_t mutexREAD;
int main(int argc, char *argv[]){
    pthread_t READERthreads;
    pthread_mutex_init(&mutexREAD, NULL);
    string *fname;
    cin>> *fname;
    pthread_create(&READERthreads, NULL, reader_thread, (void*) fname);
}
void *reader_thread(void *param){
    string fname = *(string *) param;
    cout<<"filename is "<< fname<<endl;
    ifstream myfile(fname.c_str());
    return NULL;
 }

上面的代码抛出分段错误。我可能搞砸了我的指针，但我不知道哪里出了问题，我该如何解决？

Answer 1

两个问题：崩溃的第一个也是可能的原因是你有一个指针fname但它实际上并没有指向任何地方。当您取消引用它时，这会导致未定义的行为。

除非你需要将不同的字符串传递给不同的线程，否则它不需要是一个指针，只需在调用pthread_create时使用地址运算符&即可。

当然，您可以改用std::thread，并按值传递字符串，而不必担心指针：

...
std::string fname;
std::cin >> fname;
std::thread READERthread{&reader_thread, fname);
READERthread.join();
...

和

void reader_thread(std::string fname)
{
    ...
}

---

第二个问题是，在退出进程之前，您不会等待线程完成。这将杀死线程。您要么需要加入线程，等待它退出。或者你可以分离它，只退出主线程(而不是进程(，这会让另一个线程在后台运行。

Answer 2

你声明了一个指向string的指针，只需使用string并传递其地址。

pthread_mutex_t mutexREAD;
int main(int argc, char *argv[]){
    pthread_t READERthreads;
    pthread_mutex_init(&mutexREAD, NULL);
    string fname;
    cin>> fname;
    pthread_create(&READERthreads, NULL, reader_thread, (void*) &fname);
    pthread_join(&READERthreads,NULL);
}
void *reader_thread(void *param){
    string fname = *(string *) param;
    cout<<"filename is "<< fname<<endl;
    ifstream myfile(fname.c_str());
    return NULL;
 }

另一个问题是你不等待线程终止，那么堆栈分配的字符串可能会在线程有时间使用它之前被释放......在生成线程中使用pthread_join。