How do I return a value from thread function?

Question

I have a struct like this:

struct data{ int x; int y; }

and I have a thread function that looks like this:

void *threadFunction(void *item){ data *myData = (data*) item; int first = 50; int second = 10; myData->x = first; myData->y = second; return(void*) myData; }

I call the thread function like this in main():

pthread_create(threadID, NULL, threadFunction, &item);

but when I want to get the values from my thread function back into main() using this:

struct data* returnedItem; pthread_join(threadID, (void**) returnedItem; cout << returnedItem->x << returnedItem->y;

I'm not sure what happens as my program seems to just do nothing. It compiles and runs, but it loops infinitely somewhere or just waits on something. I'm not sure what happens at all, I just don't get any sort of response. What am I doing wrong? Is there something I'm missing in main() to retrieve the value from the pthread_join statement? Maybe I screwed up the arguments in the statement?

Why bother at all. If you just return a pointer to the original thing, just keep using &item... — Kerrek SB
– Kerrek SB, Commented Oct 20, 2012 at 23:33

Kerrek SB · Accepted Answer · 2012-10-20 23:34:08Z

1

You're getting your pointers confused. Do it like this:

void * p; pthread_join(threadID, &p); data * returnedItem = static_cast<data *>(p);

While it's possible to convert pointers from and to void pointers, you can't just pretend that a variable of type X (in your case data *) is actually a variable of type Y (in your case void *). That's not allowed and undefined behaviour.

answered Oct 20, 2012 at 23:34

Kerrek SB

480k96 gold badges904 silver badges1.1k bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

ahabos Over a year ago

Thanks for the help, and it's beginning to make more sense, but my program is still non-responsive. Am I returning the pointer properly?

Kerrek SB Over a year ago

@ahabos: Yeah, that looks right, though you don't need the cast.

ahabos Over a year ago

I got it, turns out the error wasn't in passing or returning arguments. Thanks for your help though, I understand what I was doing more clearly now!

BigBoss · Accepted Answer · 2012-10-21 01:03:50Z

pthread_join in its second argument take a pointer to a void* to return result of thread, so you write pthread_join(threadID, (void**) returnedItem );. But think about it:

struct data* returnedItem; // what is the value of returnedItem?? possibly garbage, lets say it is 0xFEDCBA98 pthread_join(threadID, (void**) returnedItem );

You cast that garbase into (void**), now at some point pthread_join want to write the result of the thread into argument that provided by you so it say

* (void**) 0xFEDCBA98 = thread_return_value;

but wait! you make pthread_join to deference an invalid pointer and it may even generate something like segmentation fault or worse overwrite one of your most important information! So every time that a function need room to return something, you must provide a valid memory for it. Valid code can be:

pthread_join(threadID, (void**) &returnedItem );

Collectives™ on Stack Overflow

How do I return a value from thread function?

2 Answers 2

3 Comments

Comments

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

3 Comments

Comments

Related