René's URL Explorer Experiment


Title: crypto.Hash.write swallows errors · Issue #63258 · nodejs/node · GitHub

Open Graph Title: crypto.Hash.write swallows errors · Issue #63258 · nodejs/node

X Title: crypto.Hash.write swallows errors · Issue #63258 · nodejs/node

Description: Version v25.9.0 Platform Linux 749dbb0e74fd 6.8.0-106-generic #106-Ubuntu SMP PREEMPT_DYNAMIC Fri Mar 6 07:58:08 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux Subsystem crypto What steps will reproduce the bug? I'm working on a software-fault-...

Open Graph Description: Version v25.9.0 Platform Linux 749dbb0e74fd 6.8.0-106-generic #106-Ubuntu SMP PREEMPT_DYNAMIC Fri Mar 6 07:58:08 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux Subsystem crypto What steps will reproduce t...

X Description: Version v25.9.0 Platform Linux 749dbb0e74fd 6.8.0-106-generic #106-Ubuntu SMP PREEMPT_DYNAMIC Fri Mar 6 07:58:08 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux Subsystem crypto What steps will reproduce t...

Opengraph URL: https://github.com/nodejs/node/issues/63258

X: @github

direct link

Domain: github.com


Hey, it has json ld scripts:
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"crypto.Hash.write swallows errors","articleBody":"### Version\n\nv25.9.0\n\n### Platform\n\n```text\nLinux 749dbb0e74fd 6.8.0-106-generic #106-Ubuntu SMP PREEMPT_DYNAMIC Fri Mar  6 07:58:08 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux\n```\n\n### Subsystem\n\ncrypto\n\n### What steps will reproduce the bug?\n\nI'm working on a software-fault-injection testing library that injected a fault into the crypto bindings. Specifically a fault here: https://github.com/nodejs/node/blob/a2aeb72bff19c89fe742f5a2db8cfc8d6407f830/deps/ncrypto/ncrypto.cc#L4326\nTo reproduce this issue, change that code to always return false for simplicity; indicating a failed crypto hash update action.\n\nThen execute the following JavaScript code:\n```js\nconst crypto = require('crypto');\n\nconst h = crypto.createHash('sha3-512');\nh.write('hello');\n//h.update('hello'); This line throws\nh.end();\n```\n\nIf we were to use the update call, then it throws an exception.\nHowever, the above code uses the write call. The write call returns true (which makes sense as the return value doesn't indicate success/failure). But I don't see an indication of an error: no exception, no error event emitted. It _seems_ that the error is silently swallowed.\n\nI did try using a callback for the write call:\n```js\nconst crypto = require('crypto');\n\nconst h = crypto.createHash('sha3-512');\nh.write('hello', console.log);\nh.end();\n```\n\nand this just logs `null`. Also no indication of an error.\n\n### How often does it reproduce? Is there a required condition?\n\nConsistently reproduces if false is returned from here:\n\nhttps://github.com/nodejs/node/blob/a2aeb72bff19c89fe742f5a2db8cfc8d6407f830/deps/ncrypto/ncrypto.cc#L4326\n\n### What is the expected behavior? Why is that the expected behavior?\n\nI had expected an exception, a false return value, or an error event emitted somewhere but none of these things seem to happen.\nI could be wrong though about my expectations and missing something.\n\n### What do you see instead?\n\nThe code runs to completion, and if checking the digest, it produces the same digest as if there was no data written to the Hash stream. This means that errors are silently swallowed and the wrong hash is produced.\n\n### Additional information\n\nThis issue was found using a hybrid static-dynamic analyzer I'm working on.","author":{"url":"https://github.com/ndossche","@type":"Person","name":"ndossche"},"datePublished":"2026-05-11T21:18:55.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":3},"url":"https://github.com/63258/node/issues/63258"}

route-pattern/_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format)
route-controllervoltron_issues_fragments
route-actionissue_layout
fetch-noncev2:ab3daadf-02cc-df8e-01fe-e7d9d7d466c5
current-catalog-service-hash81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114
request-idD802:327B79:3103F4C:459A37E:6A4D0B94
html-safe-nonceab80ca44120775114f405834a30213496c057887fac8077e30e912d2bcbfb58d
visitor-payloadeyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJEODAyOjMyN0I3OTozMTAzRjRDOjQ1OUEzN0U6NkE0RDBCOTQiLCJ2aXNpdG9yX2lkIjoiNTM5NzU1MTI2NTk5MDExODI5MiIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9
visitor-hmac60314e9ed33c9bac87bcd401d793f66b71b37a2183d3f0f8aa9ef42e512655ae
hovercard-subject-tagissue:4424375031
github-keyboard-shortcutsrepository,issues,copilot
google-site-verificationApib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I
octolytics-urlhttps://collector.github.com/github/collect
analytics-location///voltron/issues_fragments/issue_layout
fb:app_id1401488693436528
apple-itunes-appapp-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/nodejs/node/63258/issue_layout
twitter:imagehttps://opengraph.githubassets.com/abe607593961b01f0fa1c7b41e7b3f0b9b2c6677c04872f4cd608e9f71d702de/nodejs/node/issues/63258
twitter:cardsummary_large_image
og:imagehttps://opengraph.githubassets.com/abe607593961b01f0fa1c7b41e7b3f0b9b2c6677c04872f4cd608e9f71d702de/nodejs/node/issues/63258
og:image:altVersion v25.9.0 Platform Linux 749dbb0e74fd 6.8.0-106-generic #106-Ubuntu SMP PREEMPT_DYNAMIC Fri Mar 6 07:58:08 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux Subsystem crypto What steps will reproduce t...
og:image:width1200
og:image:height600
og:site_nameGitHub
og:typeobject
og:author:usernamendossche
hostnamegithub.com
expected-hostnamegithub.com
Nonebcb4661e6fb4fe8b394c51ea02ccdb2236d40dc59afc75a3bbba50bf6517134c
turbo-cache-controlno-preview
go-importgithub.com/nodejs/node git https://github.com/nodejs/node.git
octolytics-dimension-user_id9950313
octolytics-dimension-user_loginnodejs
octolytics-dimension-repository_id27193779
octolytics-dimension-repository_nwonodejs/node
octolytics-dimension-repository_publictrue
octolytics-dimension-repository_is_forkfalse
octolytics-dimension-repository_network_root_id27193779
octolytics-dimension-repository_network_root_nwonodejs/node
turbo-body-classeslogged-out env-production page-responsive
disable-turbofalse
browser-stats-urlhttps://api.github.com/_private/browser/stats
browser-errors-urlhttps://api.github.com/_private/browser/errors
releasefb70bd3c4b2bec429781b65419e912c66e2d5581
ui-targetfull
theme-color#1e2327
color-schemelight dark

Links:

Skip to contenthttps://github.com/nodejs/node/issues/63258#start-of-content
https://github.com/
Sign in https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fnodejs%2Fnode%2Fissues%2F63258
GitHub CopilotWrite better code with AIhttps://github.com/features/copilot
GitHub Copilot appDirect agents from issue to mergehttps://github.com/features/ai/github-app
MCP RegistryNewIntegrate external toolshttps://github.com/mcp
ActionsAutomate any workflowhttps://github.com/features/actions
CodespacesInstant dev environmentshttps://github.com/features/codespaces
IssuesPlan and track workhttps://github.com/features/issues
Code ReviewManage code changeshttps://github.com/features/code-review
GitHub Advanced SecurityFind and fix vulnerabilitieshttps://github.com/security/advanced-security
Code securitySecure your code as you buildhttps://github.com/security/advanced-security/code-security
Secret protectionStop leaks before they starthttps://github.com/security/advanced-security/secret-protection
Why GitHubhttps://github.com/why-github
Documentationhttps://docs.github.com
Bloghttps://github.blog
Changeloghttps://github.blog/changelog
Marketplacehttps://github.com/marketplace
View all featureshttps://github.com/features
Enterpriseshttps://github.com/enterprise
Small and medium teamshttps://github.com/team
Startupshttps://github.com/enterprise/startups
Nonprofitshttps://github.com/solutions/industry/nonprofits
App Modernizationhttps://github.com/solutions/use-case/app-modernization
DevSecOpshttps://github.com/solutions/use-case/devsecops
DevOpshttps://github.com/solutions/use-case/devops
CI/CDhttps://github.com/solutions/use-case/ci-cd
View all use caseshttps://github.com/solutions/use-case
Healthcarehttps://github.com/solutions/industry/healthcare
Financial serviceshttps://github.com/solutions/industry/financial-services
Manufacturinghttps://github.com/solutions/industry/manufacturing
Governmenthttps://github.com/solutions/industry/government
View all industrieshttps://github.com/solutions/industry
View all solutionshttps://github.com/solutions
AIhttps://github.com/resources/articles?topic=ai
Software Developmenthttps://github.com/resources/articles?topic=software-development
DevOpshttps://github.com/resources/articles?topic=devops
Securityhttps://github.com/resources/articles?topic=security
View all topicshttps://github.com/resources/articles
Customer storieshttps://github.com/customer-stories
Events & webinarshttps://github.com/resources/events
Ebooks & reportshttps://github.com/resources/whitepapers
Business insightshttps://github.com/solutions/executive-insights
GitHub Skillshttps://skills.github.com
Documentationhttps://docs.github.com
Customer supporthttps://support.github.com
Community forumhttps://github.com/orgs/community/discussions
Trust centerhttps://github.com/trust-center
Partnershttps://github.com/partners
View all resourceshttps://github.com/resources
GitHub SponsorsFund open source developershttps://github.com/sponsors
Security Labhttps://securitylab.github.com
Maintainer Communityhttps://maintainers.github.com
Acceleratorhttps://github.com/accelerator
GitHub Starshttps://stars.github.com
Archive Programhttps://archiveprogram.github.com
Topicshttps://github.com/topics
Trendinghttps://github.com/trending
Collectionshttps://github.com/collections
Enterprise platformAI-powered developer platformhttps://github.com/enterprise
GitHub Advanced SecurityEnterprise-grade security featureshttps://github.com/security/advanced-security
Copilot for BusinessEnterprise-grade AI featureshttps://github.com/features/copilot/copilot-business
Premium SupportEnterprise-grade 24/7 supporthttps://github.com/premium-support
Pricinghttps://github.com/pricing
Search syntax tipshttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
documentationhttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
Sign in https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fnodejs%2Fnode%2Fissues%2F63258
Sign up https://github.com/signup?ref_cta=Sign+up&ref_loc=header+logged+out&ref_page=%2F%3Cuser-name%3E%2F%3Crepo-name%3E%2Fvoltron%2Fissues_fragments%2Fissue_layout&source=header-repo&source_repo=nodejs%2Fnode
Reloadhttps://github.com/nodejs/node/issues/63258
Reloadhttps://github.com/nodejs/node/issues/63258
Reloadhttps://github.com/nodejs/node/issues/63258
Please reload this pagehttps://github.com/nodejs/node/issues/63258
nodejs https://github.com/nodejs
nodehttps://github.com/nodejs/node
Please reload this pagehttps://github.com/nodejs/node/issues/63258
Notifications https://github.com/login?return_to=%2Fnodejs%2Fnode
Fork 36k https://github.com/login?return_to=%2Fnodejs%2Fnode
Star 118k https://github.com/login?return_to=%2Fnodejs%2Fnode
Code https://github.com/nodejs/node
Issues 1.4k https://github.com/nodejs/node/issues
Pull requests 958 https://github.com/nodejs/node/pulls
Actions https://github.com/nodejs/node/actions
Projects https://github.com/nodejs/node/projects
Security and quality 0 https://github.com/nodejs/node/security
Insights https://github.com/nodejs/node/pulse
Code https://github.com/nodejs/node
Issues https://github.com/nodejs/node/issues
Pull requests https://github.com/nodejs/node/pulls
Actions https://github.com/nodejs/node/actions
Projects https://github.com/nodejs/node/projects
Security and quality https://github.com/nodejs/node/security
Insights https://github.com/nodejs/node/pulse
#63261https://github.com/nodejs/node/pull/63261
crypto.Hash.write swallows errorshttps://github.com/nodejs/node/issues/63258#top
#63261https://github.com/nodejs/node/pull/63261
https://github.com/ndossche
ndosschehttps://github.com/ndossche
on May 11, 2026https://github.com/nodejs/node/issues/63258#issue-4424375031
node/deps/ncrypto/ncrypto.cchttps://github.com/nodejs/node/blob/a2aeb72bff19c89fe742f5a2db8cfc8d6407f830/deps/ncrypto/ncrypto.cc#L4326
a2aeb72https://github.com/nodejs/node/commit/a2aeb72bff19c89fe742f5a2db8cfc8d6407f830
node/deps/ncrypto/ncrypto.cchttps://github.com/nodejs/node/blob/a2aeb72bff19c89fe742f5a2db8cfc8d6407f830/deps/ncrypto/ncrypto.cc#L4326
a2aeb72https://github.com/nodejs/node/commit/a2aeb72bff19c89fe742f5a2db8cfc8d6407f830
https://github.com
Termshttps://docs.github.com/site-policy/github-terms/github-terms-of-service
Privacyhttps://docs.github.com/site-policy/privacy-policies/github-privacy-statement
Securityhttps://github.com/security
Statushttps://www.githubstatus.com/
Communityhttps://github.community/
Docshttps://docs.github.com/
Contacthttps://support.github.com?tags=dotcom-footer

Viewport: width=device-width


URLs of crawlers that visited me.