Corrected memory leak of task refs in client deliver_message on timeo…

njouanin / hbmqtt

MQTT client/broker using Python asynchronous I/O

MIT License

795 stars 188 forks source link

Corrected memory leak of task refs in client deliver_message on timeo… #204

Closed shipmints closed 4 years ago

shipmints commented 4 years ago

…ut causing client_tasks to grow forever, in plugin/manager causing _fired_events to grow forever

Per #203

coveralls commented 4 years ago

Coverage remained the same at ?% when pulling 138313a3eb10f16661e3fc56eddac06dccc2aef6 on shipmints:fix-mem-leaks into f76be85a0ade2d612136fbfba7ded7427b82206c on beerfactory:master.

coveralls commented 4 years ago

Coverage remained the same at ?% when pulling 138313a3eb10f16661e3fc56eddac06dccc2aef6 on shipmints:fix-mem-leaks into f76be85a0ade2d612136fbfba7ded7427b82206c on beerfactory:master.

shipmints commented 4 years ago

Thank you for the merge @njouanin We are tracking other potential leaks and one amendment to the PR which I believe is a better patch. Stay tuned another PR on that branch coming shortly.