diff --git a/sabnzbd/cfg.py b/sabnzbd/cfg.py index 9287ad42e..15faa96ba 100644 --- a/sabnzbd/cfg.py +++ b/sabnzbd/cfg.py @@ -73,7 +73,7 @@ class ErrorCatchingArgumentParser(argparse.ArgumentParser): def clean_nice_ionice_parameters(value: str) -> ValidateResult: """Verify that the passed parameters are not exploits""" if value: - parser = ErrorCatchingArgumentParser() + parser = ErrorCatchingArgumentParser(add_help=False) # Nice parameters parser.add_argument("-n", "--adjustment", type=int) diff --git a/tests/test_cfg.py b/tests/test_cfg.py index f6203c7ed..a4e78bf23 100644 --- a/tests/test_cfg.py +++ b/tests/test_cfg.py @@ -88,6 +88,10 @@ class TestValidators: assert_blocked("echo 'how;now;brown;cow'") assert_blocked("-c'echo'") assert_blocked("--classdata=;/bin/echo") + assert_blocked("-h") + assert_blocked("--help") + assert_blocked("-h -c1") + assert_blocked("-c1 --help") @pytest.mark.parametrize( "setting, is_correct_win, is_correct_unix",