-
Notifications
You must be signed in to change notification settings - Fork 220
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: crash on openwrt #640
Conversation
6a1f0ee
to
3c16ccf
Compare
❌ Your branch is currently out-of-sync to main. No worry, I will fix it for you. |
测试https://github.com/daeuniverse/dae/actions/runs/10908052824 未发现崩溃和CPU占用异常,数十次reload和大量udp发包后均工作正常。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧪 Since the PR has been fully tested, please consider merging it.
Co-authored-by: dae-prow[bot] <136105375+dae-prow[bot]@users.noreply.github.com> (cherry picked from commit cec5e71)
Co-authored-by: dae-prow[bot] <136105375+dae-prow[bot]@users.noreply.github.com>
Background
在 q.closed 的情况下 p.EmitTask 可能会触发死循环,导致吃尽 cpu。
Checklist
Full Changelogs
Issue Reference
Closes #638
Closes #636
Test Result