fix(deps): update dependency uuid [security]#8346
Conversation
|
/gcbrun |
There was a problem hiding this comment.
Code Review
This pull request updates the uuid dependency to version 11.1.1 across numerous package.json files. A review comment points out that handwritten/bigtable/package.json pins the dependency to a specific version, whereas other files use a caret range; it is recommended to use ^11.1.1 for consistency and to ensure future security patches are automatically included.
| "ts-loader": "^9.5.2", | ||
| "typescript": "^5.8.2", | ||
| "uuid": "8.0.0", | ||
| "uuid": "11.1.1", |
There was a problem hiding this comment.
The uuid dependency is pinned to a specific version (11.1.1), whereas other packages in this pull request and other dependencies in this file use caret ranges (e.g., ^11.1.1). For consistency and to ensure that future security patches within the same major version are automatically picked up, consider using a caret range.
| "uuid": "11.1.1", | |
| "uuid": "^11.1.1", |
462229f to
8f8eaea
Compare
|
/gcbrun |
This PR contains the following updates:
^9.0.0→^14.0.0^8.0.0→^14.0.0^10.0.0→^14.0.0^8.3.2→^14.0.08.0.0→11.1.1uuid: Missing buffer bounds check in v3/v5/v6 when buf is provided
CVE-2026-41907 / GHSA-w5hq-g745-h8pq
More information
Details
Summary
The
v3(),v5(), andv6()API methods (notuuidrelease versions) accept external output buffers but do not reject out-of-range writes (smallbufor largeoffset).By contrast,
v4(),v1(), andv7()API methods explicitly throwRangeErroron invalid bounds.This inconsistency allows silent partial writes into caller-provided buffers.
Affected code
src/v35.ts(v3()/v5()path) writesbuf[offset + i]without bounds validation.src/v6.tswritesbuf[offset + i]without bounds validation.Reproducible PoC
Observed:
v4() THREW RangeErrorv5() NO_THROWv6() NO_THROWExample partial overwrite evidence captured during audit:
Security impact
Suggested fix
Add the same guard used by
v4()/v1()/v7():Apply to:
src/v35.ts(coversv3()andv5())src/v6.tsSeverity
CVSS:4.0/AV:N/AC:L/AT:P/PR:N/UI:N/VC:N/VI:L/VA:N/SC:N/SI:N/SA:NReferences
This data is provided by the GitHub Advisory Database (CC-BY 4.0).
Release Notes
uuidjs/uuid (uuid)
v14.0.0Compare Source
Security
v3(),v5(), andv6()did not validate that writes would remain within the bounds of a caller-supplied buffer, allowing out-of-bounds writes when an invalidoffsetwas provided. ARangeErroris now thrown ifoffset < 0oroffset + 16 > buf.length.⚠ BREAKING CHANGES
cryptois now expected to be globally defined (requires node@20+) (#935)v13.0.2Compare Source
Bug Fixes
v13.0.1Compare Source
Bug Fixes
v13.0.0Compare Source
⚠ BREAKING CHANGES
Bug Fixes
v12.0.1Compare Source
v12.0.0Compare Source
⚠ BREAKING CHANGES
Features
Bug Fixes
v11.1.1Compare Source
v11.1.0Compare Source
Features
Uint8Arraysubtypes forbufferoption (#865) (a5231e7)v11.0.5Compare Source
Bug Fixes
v11.0.4Compare Source
Bug Fixes
v1(),v4(), andv7()(#845) (e0ee900)v11.0.3Compare Source
Bug Fixes
v11.0.2Compare Source
Bug Fixes
v11.0.1Compare Source
Bug Fixes
v11.0.0Compare Source
⚠ BREAKING CHANGES
Features
Bug Fixes
v10.0.0Compare Source
⚠ BREAKING CHANGES
Features
Bug Fixes
Configuration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.
This PR was generated by Mend Renovate. View the repository job log.