joaquinfontela / Taller-95.08-TP3

0 stars 0 forks source link

Eliminar operator=(Socket&&) #12

Open mlafroce opened 3 years ago

mlafroce commented 3 years ago

https://github.com/joaquinfontela/tp3-taller/blob/c6ff6c4b736c5cd5d78e25d1845f6c91f3891923/common_src/Socket.cpp#L13-L15 Ojo, por un lado no estás chequeando que this != &other, por otro lado, si está conectado this, lo estás pisando y leakeando un socket. Mejor eliminar el operador=