-
-
Notifications
You must be signed in to change notification settings - Fork 31.4k
Memory leak with util.TextDecoder #32424
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
Comments
addaleax
added a commit
to addaleax/node
that referenced
this issue
Mar 22, 2020
`ObjectTemplate`s are not garbage-collected like regular objects (for some reason). It is sufficient to create a single template anyway, so do that to address the memory leak. Fixes: nodejs#32424
2 tasks
See #32426 :) |
MylesBorins
pushed a commit
that referenced
this issue
Mar 26, 2020
`ObjectTemplate`s are not garbage-collected like regular objects (for some reason). It is sufficient to create a single template anyway, so do that to address the memory leak. Fixes: #32424 PR-URL: #32426 Reviewed-By: Gus Caplan <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]>
yoursunny
added a commit
to yoursunny/NDNts
that referenced
this issue
Apr 11, 2020
nodejs/node#32424 This is needed for Node < 13.12.0
targos
pushed a commit
to targos/node
that referenced
this issue
Apr 25, 2020
`ObjectTemplate`s are not garbage-collected like regular objects (for some reason). It is sufficient to create a single template anyway, so do that to address the memory leak. Fixes: nodejs#32424 PR-URL: nodejs#32426 Reviewed-By: Gus Caplan <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]>
targos
pushed a commit
that referenced
this issue
Apr 28, 2020
`ObjectTemplate`s are not garbage-collected like regular objects (for some reason). It is sufficient to create a single template anyway, so do that to address the memory leak. Fixes: #32424 PR-URL: #32426 Reviewed-By: Gus Caplan <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What steps will reproduce the bug?
Credit to @Hakerh400 on nodejs/help#2541:
How often does it reproduce? Is there a required condition?
This occurs whenever the built-in TextDecoder is used to decode a buffer.
What is the expected behavior?
Memory is reclaimed by the garbage collector, allowing the process to continue indefinitely.
What do you see instead?
After a short period of time, the process crashes with the following console output:
Additional information
TextEncoder#encode
.@exodus/text-encoding-utf8
with success)The text was updated successfully, but these errors were encountered: