Potential channel leak #325

Closed
opened 2025-12-29 01:27:04 +01:00 by adam · 2 comments
Owner

Originally created by @awsong on GitHub (Sep 15, 2022).

Bug description

https://github.com/juanfont/headscale/blob/main/protocol_common_poll.go#L169
After this line, the function returns and updateChan will be left open. The closing call is here: https://github.com/juanfont/headscale/blob/main/protocol_common_poll.go#L613 which won't be reached if line 169 returns.

Originally created by @awsong on GitHub (Sep 15, 2022). <!-- Headscale is a multinational community across the globe. Our common language is English. Please consider raising the bug report in this language. --> **Bug description** [https://github.com/juanfont/headscale/blob/main/protocol_common_poll.go#L169 ](url) After this line, the function returns and updateChan will be left open. The closing call is here: [https://github.com/juanfont/headscale/blob/main/protocol_common_poll.go#L613](url) which won't be reached if line 169 returns.
adam added the stalebug labels 2025-12-29 01:27:04 +01:00
adam closed this issue 2025-12-29 01:27:04 +01:00
Author
Owner

@github-actions[bot] commented on GitHub (Sep 30, 2023):

This issue is stale because it has been open for 180 days with no activity.

@github-actions[bot] commented on GitHub (Sep 30, 2023): This issue is stale because it has been open for 180 days with no activity.
Author
Owner

@github-actions[bot] commented on GitHub (Oct 14, 2023):

This issue was closed because it has been inactive for 14 days since being marked as stale.

@github-actions[bot] commented on GitHub (Oct 14, 2023): This issue was closed because it has been inactive for 14 days since being marked as stale.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/headscale#325