Skip to content

Debug zstd bundle extraction errors#2766

Closed
cklin wants to merge 2 commits into
mainfrom
cklin/zstd-extract-debugging
Closed

Debug zstd bundle extraction errors#2766
cklin wants to merge 2 commits into
mainfrom
cklin/zstd-extract-debugging

Conversation

@cklin

@cklin cklin commented Feb 14, 2025

Copy link
Copy Markdown
Contributor

Merge / deployment checklist

  • Confirm this change is backwards compatible with existing workflows.
  • Confirm the readme has been updated if necessary.
  • Confirm the changelog has been updated if necessary.

@cklin cklin force-pushed the cklin/zstd-extract-debugging branch from fd7c063 to 05549f5 Compare February 14, 2025 18:07
Comment thread src/tools-download.ts Outdated
let stderr = "";
try {
const zstdTestRunner = new toolrunner.ToolRunner(
"zstd",

Check failure

Code scanning / CodeQL

Exec call vulnerable to binary planting

This exec call might be vulnerable to Windows binary planting vulnerabilities.
@cklin cklin force-pushed the cklin/zstd-extract-debugging branch from 05549f5 to f33ae7c Compare February 14, 2025 18:49
Comment thread src/tools-download.ts
let stdout = "";
let stderr = "";
try {
const dfRunner = new toolrunner.ToolRunner("df", ["-h"], {

Check failure

Code scanning / CodeQL

Exec call vulnerable to binary planting

This exec call might be vulnerable to Windows binary planting vulnerabilities.
@cklin cklin force-pushed the cklin/zstd-extract-debugging branch from e2ac4ba to cd90a5e Compare February 14, 2025 19:38
@cklin cklin closed this Feb 14, 2025
@cklin cklin deleted the cklin/zstd-extract-debugging branch February 14, 2025 22:50
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