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

Dont use user supplied progname to remove CWE134 warning #49

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

vmcj
Copy link
Owner

@vmcj vmcj commented Jan 24, 2022

No description provided.

vmcj added 10 commits January 24, 2022 13:51
Documentation needs 400 for all inputs, so we loose currently a more
specific message. Alternative is duplication of code.
We explicit only check our own API and remove all other json files.

This does need an account with 42Crunch as explained in the github
action.
@vmcj vmcj force-pushed the judge_evict_CWE134 branch 4 times, most recently from 02ae652 to a72df1e Compare January 24, 2022 13:37
@vmcj vmcj closed this Jan 24, 2022
@vmcj vmcj force-pushed the judge_evict_CWE134 branch from a72df1e to 9b11244 Compare January 24, 2022 13:47
@vmcj vmcj reopened this Jan 24, 2022
@vmcj vmcj force-pushed the judge_evict_CWE134 branch 2 times, most recently from 79bdef2 to 7d63349 Compare January 24, 2022 14:44
vmcj added 2 commits January 24, 2022 16:35
Replace everything to just 1 long string of chars
@vmcj vmcj force-pushed the judge_evict_CWE134 branch from a1dd318 to 1e27882 Compare January 24, 2022 15:37
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