-
Notifications
You must be signed in to change notification settings - Fork 29.1k
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
Request to backport "fix memory leak when nghttp2 hd threshold is reached" into v14.x #45295
Comments
v14.x is in maintenance mode, meaning no commit will land on it without a manual backport. Your best shot to see it being backported is to open a backport PR youself as documented in https://github.com/nodejs/node/blob/main/doc/contributing/backporting-to-release-lines.md. |
ywave620
added a commit
to ywave620/node
that referenced
this issue
Nov 4, 2022
landed |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Version
14x
Platform
linux,mac,window
Subsystem
http2
What steps will reproduce the bug?
As shown in #35233
Fix in #41502
How often does it reproduce? Is there a required condition?
Consistent reproducable
What is the expected behavior?
no crash
What do you see instead?
./src/node_http2.cc:522:virtual node::http2::Http2Session::~Http2Session(): Assertion `(current_nghttp2_memory_) == (0)' failed.
1: 0xa02dd0 node::Abort() [node]
Additional information
I can't find the fix commit(f98a785) in https://github.com/nodejs/node/blob/v14.x/doc/changelogs/CHANGELOG_V14.md, since this is a critial bug that will lead to crash and v14.x is a LTS, I request Node team to backport it.
The text was updated successfully, but these errors were encountered: