fix(deps): update dependency uuid to v14 [security]#4681
fix(deps): update dependency uuid to v14 [security]#4681openshift-merge-bot[bot] merged 1 commit intomainfrom
Conversation
|
Hi @renovate[bot]. Thanks for your PR. I'm waiting for a redhat-developer member to verify that this patch is reasonable to test. If it is, they should reply with Regular contributors should join the org to skip this step. Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
a422f21 to
024650e
Compare
024650e to
04d3ef6
Compare
04d3ef6 to
b6efdf6
Compare
|
The container image build workflow finished with status: |
b6efdf6 to
089bf47
Compare
089bf47 to
45730e8
Compare
|
/ok-to-test |
45730e8 to
78e674c
Compare
78e674c to
97d78d7
Compare
d9457db to
e074f15
Compare
|
The container image build workflow finished with status: |
|
❌ The last analysis has failed. |
|
/test e2e-ocp-helm |
|
/retest |
e074f15 to
714bb96
Compare
714bb96 to
2fda2af
Compare
2fda2af to
8160458
Compare
8160458 to
b7360b5
Compare
b7360b5 to
10b66fc
Compare
10b66fc to
ef78b85
Compare
Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
ef78b85 to
351aec9
Compare
|
The container image build workflow finished with status: |
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #4681 +/- ##
===========================================
+ Coverage 40.98% 69.56% +28.58%
===========================================
Files 119 109 -10
Lines 2223 4705 +2482
Branches 560 511 -49
===========================================
+ Hits 911 3273 +2362
- Misses 1306 1432 +126
+ Partials 6 0 -6
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
|
/retest |
|
/lgtm |



This PR contains the following updates:
11.1.0→14.0.0Warning
Some dependencies could not be looked up. Check the Dependency Dashboard for more information.
uuid: Missing buffer bounds check in v3/v5/v6 when buf is provided
GHSA-w5hq-g745-h8pq
More information
Details
Summary
v3,v5, andv6accept external output buffers but do not reject out-of-range writes (smallbufor largeoffset).By contrast,
v4,v1, andv7explicitly throwRangeErroron invalid bounds.This inconsistency allows silent partial writes into caller-provided buffers.
Affected code
src/v35.ts(v3/v5path) 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(coversv3andv5)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.0Compare Source
⚠ BREAKING CHANGES
Bug Fixes
v12.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 is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.