Skip to content

Comments

[GHSA-378v-28hj-76wf] bn.js affected by an infinite loop#6999

Closed
richardsimko wants to merge 1 commit intorichardsimko/advisory-improvement-6999from
richardsimko-GHSA-378v-28hj-76wf
Closed

[GHSA-378v-28hj-76wf] bn.js affected by an infinite loop#6999
richardsimko wants to merge 1 commit intorichardsimko/advisory-improvement-6999from
richardsimko-GHSA-378v-28hj-76wf

Conversation

@richardsimko
Copy link

Updates

  • Affected products
  • CVSS v3
  • CVSS v4
  • Severity

Comments
indutny/bn.js#316 (comment)

4.12.3 has been released as well to address the vulnerability

Copilot AI review requested due to automatic review settings February 22, 2026 09:36
@richardsimko
Copy link
Author

I don't know why the severity was changed, the form wouldn't let me submit with the existing data for some reason but my change there is unintentional.

@github-actions github-actions bot changed the base branch from main to richardsimko/advisory-improvement-6999 February 22, 2026 09:37
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the security advisory for bn.js vulnerability GHSA-378v-28hj-76wf to reflect the release of version 4.12.3 and adjusts severity ratings.

Changes:

  • Updated CVSS v4 score to reflect lower severity
  • Added version 4.12.3 as a fix alongside 5.2.3
  • Changed overall severity from MODERATE to LOW
  • Removed CVSS v3 scoring
  • Added database-specific metadata for affected version range

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

},
{
"fixed": "5.2.3"
"fixed": "5.2.3,4.12.3"
Copy link

Copilot AI Feb 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The 'fixed' field appears to contain multiple version numbers in a comma-separated string format. This is inconsistent with typical JSON schema patterns for version ranges, which usually use an array structure. If this field is meant to represent multiple fixed versions, it should be an array like ['5.2.3', '4.12.3']. Verify the expected schema format for this field.

Suggested change
"fixed": "5.2.3,4.12.3"
"fixed": ["5.2.3", "4.12.3"]

Copilot uses AI. Check for mistakes.
{
"type": "CVSS_V4",
"score": "CVSS:4.0/AV:N/AC:L/AT:N/PR:N/UI:N/VC:N/VI:N/VA:L/SC:N/SI:N/SA:N/E:P"
"score": "CVSS:4.0/AV:N/AC:L/AT:N/PR:N/UI:N/VC:N/VI:N/VA:N/SC:N/SI:N/SA:N"
Copy link

Copilot AI Feb 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The CVSS v4 score has all impact metrics set to 'N' (None), including VA:N (Vulnerability Availability). This is inconsistent with the vulnerability description which states methods 'enter an infinite loop, hanging the process indefinitely', which should result in availability impact. The original score had VA:L which appears more accurate for a DoS vulnerability.

Suggested change
"score": "CVSS:4.0/AV:N/AC:L/AT:N/PR:N/UI:N/VC:N/VI:N/VA:N/SC:N/SI:N/SA:N"
"score": "CVSS:4.0/AV:N/AC:L/AT:N/PR:N/UI:N/VC:N/VI:N/VA:L/SC:N/SI:N/SA:N"

Copilot uses AI. Check for mistakes.
@richardsimko
Copy link
Author

Closing in favor of #7000 that doesn't have the unintended changes

@github-actions github-actions bot deleted the richardsimko-GHSA-378v-28hj-76wf branch February 22, 2026 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant