Skip to content

fix(track-a): accept XRes returned client resource base - #461

Merged
blakinio merged 4 commits into
mainfrom
fix/OTC-20260816-track-a-canonical-runtime-xres-client-base
Aug 17, 2026
Merged

fix(track-a): accept XRes returned client resource base#461
blakinio merged 4 commits into
mainfrom
fix/OTC-20260816-track-a-canonical-runtime-xres-client-base

Conversation

@blakinio

Copy link
Copy Markdown
Owner

Track A canonical RUNTIME — hosted XRes client-base helper correction

Task: OTC-20260816-track-a-canonical-runtime-e2e
Base: trusted main@c55e3523e6e9d50df511e65dce9145a8f951a5f5 after physical identity #457 and archive #459.

This is a GitHub-hosted-only follow-up for XRES-V2-AUD-001. It performs no Synology/X11/official-client/canonical-state access and authorizes no physical retry.

The retained v2 physical reply proved that QueryClientIds for resource 0x00c00011 may return owning client resource-base 0x00c00000 with LocalClientPid 13648. The persistent helper incorrectly required the returned client identifier to echo the exact queried resource XID.

Correction:

  • keep one-spec exactly one record fail-closed cardinality;
  • require a nonzero returned client identifier;
  • require exact LocalClientPid mask;
  • require exactly one positive CARD32 PID;
  • do not require exact resource-XID echo;
  • add the retained physical v2 raw reply as a deterministic regression fixture;
  • keep all existing malformed/truncated/oversized/wrong-mask/multi-record negatives.

Expected deterministic suite after this change: 37 tests. Dedicated helper validation must remain transport-free. After protected promotion, downstream P0 work still requires its own fresh RUNTIME admission and may proceed only if a legal current IN_GAME lifecycle exists; no session bootstrap solely for P0 is authorized.

@blakinio
blakinio marked this pull request as ready for review August 17, 2026 09:50
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@blakinio
blakinio enabled auto-merge (squash) August 17, 2026 09:50
@blakinio
blakinio merged commit 1eb4a8e into main Aug 17, 2026
22 checks passed
@blakinio
blakinio deleted the fix/OTC-20260816-track-a-canonical-runtime-xres-client-base branch August 17, 2026 09:52
blakinio added a commit that referenced this pull request Aug 17, 2026
Consume the promoted #457 XID-to-PID proof and #461 client-base semantics in the canonical worker window identity path. Add fail-closed raw-XRes ownership resolution, exact-anchor worker adaptation, deterministic regression coverage, and hosted validation.
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