Skip to content

add checksum_cmd directive to command#252

Open
kindermax wants to merge 1 commit intomasterfrom
add-checksum_cmd-directive
Open

add checksum_cmd directive to command#252
kindermax wants to merge 1 commit intomasterfrom
add-checksum_cmd-directive

Conversation

@kindermax
Copy link
Collaborator

No description provided.

@kindermax kindermax force-pushed the add-checksum_cmd-directive branch from eacfb16 to c844058 Compare March 19, 2026 07:38
Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

New security issues found

@kindermax kindermax force-pushed the add-checksum_cmd-directive branch from c844058 to eaa0769 Compare March 19, 2026 07:40
Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

New security issues found

}

func CalculateChecksumFromCmd(shell string, workDir string, script string) (string, error) {
cmd := exec.Command(shell, "-c", script)
Copy link

Choose a reason for hiding this comment

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

security (go.lang.security.audit.dangerous-exec-command): Detected non-static command inside Command. Audit the input to 'exec.Command'. If unverified user data can reach this call site, this is a code injection vulnerability. A malicious actor can inject a malicious script to execute arbitrary code.

Source: opengrep

@kindermax kindermax force-pushed the add-checksum_cmd-directive branch from eaa0769 to 25a5d7a Compare March 19, 2026 07:46
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.

1 participant