Skip to content

vms-097: SET validates qualifiers per sub-verb (IVQUAL) - #1798

Merged
baron-3dl merged 4 commits into
mainfrom
lane/dcl-set-quals
Oct 19, 2026
Merged

baron-3dl merged 4 commits into
mainfrom
lane/dcl-set-quals

Conversation

@baron-3dl

Copy link
Copy Markdown
Contributor

SET was accept-all: SET/ZQ DEFAULT x and SET FILE/ZQ f dropped the qualifier and ran. SET's qualifiers belong to the sub-verb, so cmd_set validates against a per-sub-verb table of exactly what each handler reads (DEFAULT PROMPT VERIFY NOON ON MESSAGE CONTROL PROCESS FILE UIC WORKING_SET TIME HOST ENTRY QUEUE). Self-validating sub-verbs (TERMINAL ACL PROTECTION PASSWORD ACCOUNTING VOLUME SYMBOL) and SET AUDIT (always honest-unsupported) are unchanged.

Also: SET FILE/TRUNCATE (used by the tier-4 MMK corpus) is now a real truncate-to-EOF so it is not caught by the new rejection; SET MESSAGE /NOFACILITY etc. were read as never-matching names and are now read via the negated flag.

Known gaps found, not fixed here: SET MESSAGE flags are stored but never consumed by dcl_error; SET FILE/VERSION_LIMIT is acknowledged without effect.

Tests: tests/dcl/test_set_qualifier_validation.sh, tests/qemu/test_syssvc_dcl_set_quals.c (exit 77 w/o executive). Negctl: dcl-set-quals-accept-all.

🤖 Generated with Claude Code

@baron-3dl
baron-3dl enabled auto-merge October 18, 2026 20:22
@baron-3dl
baron-3dl added this pull request to the merge queue Oct 18, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 18, 2026
@baron-3dl
baron-3dl added this pull request to the merge queue Oct 18, 2026
@baron-3dl
baron-3dl removed this pull request from the merge queue due to a manual request Oct 18, 2026
@baron-3dl
baron-3dl added this pull request to the merge queue Oct 18, 2026
@baron-3dl
baron-3dl removed this pull request from the merge queue due to a manual request Oct 18, 2026
@baron-3dl
baron-3dl added this pull request to the merge queue Oct 18, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to invalid changes in the merge commit Oct 19, 2026
@baron-3dl
baron-3dl added this pull request to the merge queue Oct 19, 2026
baron-3dl and others added 4 commits October 19, 2026 02:32
…T MESSAGE negation read

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…TE refused)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@baron-3dl
baron-3dl removed this pull request from the merge queue due to a manual request Oct 19, 2026
@baron-3dl
baron-3dl added this pull request to the merge queue Oct 19, 2026
Merged via the queue into main with commit f872f38 Oct 19, 2026
5 checks passed
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