search

netaneldj / 7542-Taller-TP3

0 stars 0 forks source link

Esto es otra race condition #26

Open fedemgp opened 4 years ago

fedemgp commented 4 years ago

https://github.com/netaneldj/tp3/blob/8cda72ad130c619f38e1a2bd75dd0979eb313064/server_ClientManager.cpp#L64-L70

El atributo debería ser std::atomic o deberías tener un mutex para evitar race condition

fedemgp commented 4 years ago

Además, el stop debería hacer un cloase del socket aceptador así se evita la race condition que tenés en el destructor de ClientManager.