Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Secure mode - part 2 #1614

Merged
merged 1 commit into from
Feb 1, 2025
Merged

Secure mode - part 2 #1614

merged 1 commit into from
Feb 1, 2025

Conversation

prabhu
Copy link
Collaborator

@prabhu prabhu commented Feb 1, 2025

No child process tests. Safely handle missing read and mkdir permissions.
Fixes #1328

@prabhu prabhu added the security label Feb 1, 2025
Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>
@prabhu prabhu force-pushed the feature/secure-mode-docs-imp branch from 80898c0 to d8d7fb5 Compare February 1, 2025 16:01
@prabhu prabhu marked this pull request as ready for review February 1, 2025 16:03
@prabhu prabhu merged commit b704676 into master Feb 1, 2025
33 checks passed
@prabhu prabhu deleted the feature/secure-mode-docs-imp branch February 1, 2025 16:51
emcfins pushed a commit to emcfins/cdxgen that referenced this pull request Mar 12, 2025
Secure mode no child

Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>
Signed-off-by: emcfins <erin.mcgill@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Security] Code execution risk when running cdxgen against untrusted repos
1 participant