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

Sanitize the invalid input so that lockhammer will stop if it detects wrong parameter #53

Open
zoybai opened this issue Nov 12, 2018 · 1 comment

Comments

@zoybai
Copy link
Contributor

zoybai commented Nov 12, 2018

Sanitize the invalid input so that lockhammer will stop if it detects wrong parameter

"lh_swap_mutex -c [0, 1, 2, 3]" succeeds in shell, which should be stopped because the -c parameter is invalid.

@lucasclucasdo
Copy link
Contributor

Please be careful that the sanitization implementation doesn't break the support for test-specific arguments. Perhaps each subtest which defines its own arguments should provide a helper which checks any arguments that the main code doesn't understand.

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

No branches or pull requests

2 participants