search

feelfreelinux / cspot

A Spotify Connect player targeting, but not limited to embedded devices (ESP32).
Other
497 stars 46 forks source link

fix socket leak #86

Closed philippe44 closed 2 years ago

philippe44 commented 2 years ago

These are 2 closes of the Session socket: one when MercuryManager is stopped and one when Session is destroyed. I don't think it's safe on embedded system to close the same socket twice. Maybe MercuryManager should not close the session

feelfreelinux commented 2 years ago

double socket fix in https://github.com/feelfreelinux/cspot/commit/5ec881fb9c5704bae691379dd5f9b6b55604c037