Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Issues detected by Coverity in ncrypto #56611

Closed
targos opened this issue Jan 15, 2025 · 0 comments · Fixed by #56612
Closed

Issues detected by Coverity in ncrypto #56611

targos opened this issue Jan 15, 2025 · 0 comments · Fixed by #56612
Labels
crypto Issues and PRs related to the crypto subsystem.

Comments

@targos
Copy link
Member

targos commented Jan 15, 2025

@jasnell @nodejs/crypto

Resource leak

Image

Image

Image

Improper use of negative value

Image

Logically dead code

Image
@targos targos added the crypto Issues and PRs related to the crypto subsystem. label Jan 15, 2025
jasnell added a commit to jasnell/node that referenced this issue Jan 15, 2025
@jasnell
deps: fixup some minor coverity warnings
c25bfd4 
Fixes: nodejs#56611
@jasnell jasnell mentioned this issue Jan 15, 2025
Merged
nodejs-github-bot pushed a commit that referenced this issue Jan 19, 2025
@jasnell
deps: fixup some minor coverity warnings
a5ed762 
Fixes: #56611
PR-URL: #56612
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Ulises Gascón <[email protected]>
aduh95 pushed a commit that referenced this issue Jan 27, 2025
@jasnell @aduh95
deps: fixup some minor coverity warnings
359039c 
Fixes: #56611
PR-URL: #56612
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Ulises Gascón <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
crypto Issues and PRs related to the crypto subsystem.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

deps: fixup some minor coverity warnings jasnell/node
1 participant
@targos