void (...) is casted to void *(...) in mg_start

Casting void (...) to void *(...), which mg_start_thread and pthread_create 
accept, is unsafe.

The only reason why compiler doesn't give a warning is, master_thread and 
worker_thread are casted to mg_thread_function_t (which has the correct 
signature) explicitly.

The fix is to change the signatures of master_thread and worker_thread to 
return void*.

A (rather simple) patch is attached.

Original issue reported on code.google.com by pas.n...@gmail.com on 8 Jan 2013 at 2:58

Attachments:

diff.patch

mlfazevedo / mongoose

void (...) is casted to void *(...) in mg_start #392