Skip to content

chore(deps): update terraform modules - #1294

Open
renovate[bot] wants to merge 1 commit into
mainfrom
renovate/terraform-provider
Open

renovate[bot] wants to merge 1 commit into
mainfrom
renovate/terraform-provider

Conversation

@renovate

@renovate renovate Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Motivation

Automated dependency update by Renovate bot.

Description

This PR contains the following updates:

Package Type Update Change
external (source) required_provider patch >= 2.4.1>= 2.4.2
random (source) required_provider patch >= 3.9.0>= 3.9.1
time (source) required_provider patch 0.14.10.14.2
tls (source) required_provider patch >= 4.4.0>= 4.4.1

Testing

This is an automated dependency update. No functional changes are expected.

Impact


Release Notes

hashicorp/terraform-provider-external (external)

v2.4.2

Compare Source

NOTES:

  • Upgrade the Go toolchain to 1.26.8. (#​512)
  • Update golang.org/x/crypto to v0.56.0, golang.org/x/mod to v0.40.0, and google.golang.org/protobuf to v1.36.12. (#​513)
hashicorp/terraform-provider-random (random)

v3.9.1

Compare Source

NOTES:

  • Upgrade the Go toolchain to 1.26.8. (#​812)
  • Update golang.org/x/mod to v0.40.0 and google.golang.org/protobuf to v1.36.12. (#​814)
hashicorp/terraform-provider-time (time)

v0.14.2

Compare Source

NOTES:

  • Upgrade the Go toolchain to 1.26.8. (#​511)
  • Update golang.org/x/crypto to v0.56.0, golang.org/x/mod to v0.40.0, and google.golang.org/protobuf to v1.36.12. (#​512)
hashicorp/terraform-provider-tls (tls)

v4.4.1

Compare Source

NOTES:

  • Upgrade the Go toolchain to 1.26.8. (#​813)

Additional Information


  • If you want to rebase/retry this PR, check this box

This PR was generated automatically by Renovate.

@github-actions

github-actions Bot commented Sep 10, 2026

Copy link
Copy Markdown

🔍 Vulnerabilities of dockerhubaneo/armonik_control_metrics:0.41.1-renovateterraformprovider.1.sha.5124859c

📦 Image Reference dockerhubaneo/armonik_control_metrics:0.41.1-renovateterraformprovider.1.sha.5124859c
digestsha256:a152631577985065fe7079d3268cfed51cf31b53a76dc27c7d759ce9d5e33546
vulnerabilitiescritical: 0 high: 0 medium: 1 low: 0
platformlinux/amd64
size67 MB
packages63
critical: 0 high: 0 medium: 1 low: 0 SharpCompress 0.30.1 (nuget)

pkg:nuget/SharpCompress@0.30.1

# Dockerfile (159:159)
COPY --from=build /app/publish/metrics .

medium 5.9: CVE--2026--44788 Improper Limitation of a Pathname to a Restricted Directory ('Path Traversal')

Affected range<0.48.0
Fixed version0.48.0
CVSS Score5.9
CVSS VectorCVSS:3.1/AV:N/AC:H/PR:N/UI:R/S:U/C:N/I:H/A:L
EPSS Score0.313%
EPSS Percentile24th percentile
Description

Summary

A path traversal vulnerability in IArchive.WriteToDirectory() allows a malicious archive to create directories outside the intended extraction root. For TAR archives, this can be escalated to arbitrary file writes by chaining with a symlink entry, giving a full write primitive on the target filesystem subject to the permissions of the running process.

Details

The vulnerable code is in the directory-entry branch of WriteToDirectoryInternal (sync, IArchiveExtensions.cs:48–61) and WriteToDirectoryAsyncInternal (async, IAsyncArchiveExtensions.cs:70–84):

var dirPath = Path.Combine(destinationDirectory, entry.Key);
Directory.CreateDirectory(Path.GetDirectoryName(dirPath + "/"));

No Path.GetFullPath() normalisation and no bounds check are applied before the Directory.CreateDirectory call. Two .NET Path.Combine behaviours make this exploitable:

  • Relative traversal: Path.Combine("/safe/extract", "../../evil") → the OS resolves .. segments on the raw path, placing the directory outside the extraction root.
  • Absolute path override: Path.Combine("/safe/extract", "/tmp/evil") → returns "/tmp/evil" — the base is discarded entirely for rooted paths.

File entries are not directly affected — they route through ExtractionMethods.WriteEntryToDirectory which applies the correct guard (GetFullPath + StartsWith, see ExtractionMethods.cs:54–65). The directory-entry branch is a separate fast-path that was added without that guard.

Affected archive formats: ZIP and TAR (non-solid). Solid archives and 7-Zip use the reader path which calls the secure method.

Escalation to arbitrary file writes (TAR only)

Path.GetFullPath on .NET does not resolve symlinks — it only normalises . and .. segments. This means the file-entry guard in ExtractionMethods.WriteEntryToDirectory can be bypassed via symlink chaining in TAR archives when the caller supplies a SymbolicLinkHandler:

archive.WriteToDirectory("/safe/extract", new ExtractionOptions
{
    ExtractFullPath = true,
    SymbolicLinkHandler = (linkPath, linkTarget) =>
        File.CreateSymbolicLink(linkPath, linkTarget)  // naive — no validation of linkTarget
});

Attack sequence in a single TAR archive:

  1. Symlink entrylink../evil_outside/
    The SymbolicLinkHandler creates /safe/extract/link pointing outside the extraction root.

  2. File entrylink/secret.txt
    ExtractionMethods.WriteEntryToDirectory computes:

    • destdir = Path.GetFullPath("/safe/extract/link")"/safe/extract/link" — textually inside root, check passes ✓
    • File.Open("/safe/extract/link/secret.txt") — OS follows symlink, file is written to /evil_outside/secret.txt

The library does not validate linkTarget before passing it to the caller's handler, and the XML docs do not warn that it may be a traversal path. The idiomatic handler implementation above is therefore silently exploitable.

ZIP does not support symlinks in SharpCompress (ZipEntry.LinkTarget always returns null), so this escalation is TAR-only.

Attack ZIP TAR
Directory traversal (escape extraction root) Yes Yes
Escalate to arbitrary file writes via symlink chain No Yes (if caller provides SymbolicLinkHandler)

Recommended fix — apply the same pattern from ExtractionMethods.WriteEntryToDirectory to both affected files:

var fullDestDir = Path.GetFullPath(destinationDirectory);
if (!fullDestDir.EndsWith(Path.DirectorySeparatorChar))
    fullDestDir += Path.DirectorySeparatorChar;

var dirPath = Path.GetFullPath(Path.Combine(fullDestDir, entry.Key));
if (!dirPath.StartsWith(fullDestDir, PathComparison))
    throw new ExtractionException(
        "Entry is trying to create a directory outside of the destination directory.");

Directory.CreateDirectory(dirPath);

Additionally, the library should validate LinkTarget before invoking the caller's SymbolicLinkHandler, or document clearly that callers must validate it themselves.

PoC

A self-contained .NET console app is available at:
https://github.com/svenclaesson/poc-sharpcompress-traversal

git clone https://github.com/svenclaesson/poc-sharpcompress-traversal
cd poc-sharpcompress-traversal
dotnet run

The PoC crafts a ZIP with three directory entries (../../escaped_relative/, /tmp/escaped_absolute/, safe_subdir/) using System.IO.Compression (stdlib), then extracts with SharpCompress. Output shows [ESCAPED] for the two malicious entries and [ok] for the legitimate one, on both sync and async APIs.

Tested against SharpCompress 0.47.4 (latest NuGet).

Impact

This is a path traversal / zip slip vulnerability (CWE-22). Any application that calls archive.WriteToDirectory() on an untrusted archive is affected — which covers the primary documented extraction API.

For ZIP archives the impact is limited to arbitrary directory creation, which can be used to stage privilege escalation (e.g. cron drop-ins, XDG config paths, service spool directories) or shadow expected paths to alter application behaviour.

For TAR archives, callers that implement a SymbolicLinkHandler — which is the only way to faithfully restore a TAR — are exposed to a full arbitrary file write primitive via the symlink chaining described above.

@renovate
renovate Bot force-pushed the renovate/terraform-provider branch from fc591b2 to df800a5 Compare September 11, 2026 17:18
@renovate renovate Bot changed the title chore(deps): update terraform tls to >= 4.4.1 chore(deps): update terraform modules Sep 11, 2026
@renovate
renovate Bot force-pushed the renovate/terraform-provider branch from df800a5 to 5124859 Compare September 11, 2026 22:55
@sonarqubecloud

Copy link
Copy Markdown

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.

0 participants