Skip to content

Update OpenSSL to 3.5.8 in SPDK base Dockerfile to fix CVE-2026-14456 - #63

Merged
boddumanohar merged 2 commits into
masterfrom
fix/openssl-3.5.8-cve
Aug 20, 2026
Merged

Update OpenSSL to 3.5.8 in SPDK base Dockerfile to fix CVE-2026-14456#63
boddumanohar merged 2 commits into
masterfrom
fix/openssl-3.5.8-cve

Conversation

@boddumanohar

Copy link
Copy Markdown
Member

Summary

  • The ultra repo's daily Grype security scan (https://github.com/simplyblock/ultra/actions/runs/32365759477/job/96414790292) flagged openssl 3.5.7, fixed in 3.5.8/3.6.4/4.0.2, as CVE-2026-14456 (High).
  • That OpenSSL is built from source in this repo's docker/Dockerfile_spdk_base, which is used to produce simplyblock/spdk-core:nvmf-blocking-from-base-latest — the base image the ultra repo's Dockerfile_spdk_ultra builds from.
  • Bump the source build from 3.5.7 to 3.5.8, following the same pattern as the prior 3.5.6 -> 3.5.7 bump (d090dff).

Test plan

  • Base image build workflow (docker-base-image-builder-amd.yml / -arm.yml) runs and publishes updated nvmf-blocking-from-base-latest
  • ultra repo's Security Scan workflow passes against the new base image

Grype's daily security scan of the ultra repo's built image flagged
openssl 3.5.7 (baked into simplyblock/spdk-core:nvmf-blocking-from-base-latest
via this Dockerfile) for CVE-2026-14456, fixed in 3.5.8/3.6.4/4.0.2.
Bump the source build to 3.5.8, same pattern as the prior 3.5.6 -> 3.5.7
bump (d090dff).
@mxsrc
mxsrc self-requested a review August 20, 2026 12:33

@mxsrc mxsrc left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Could you introduce a quick argument for the Version? Avoids missing occurences in the future.

The version string was repeated 4 times in the build RUN command,
making it easy to miss an occurrence on future CVE bumps. Centralize
it as ARG OPENSSL_VERSION at the top of the builder stage.
@boddumanohar
boddumanohar requested a review from mxsrc August 20, 2026 13:15

@mxsrc mxsrc left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thanks!

@boddumanohar
boddumanohar merged commit 37c10d9 into master Aug 20, 2026
1 check passed
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.

2 participants