research(track-a): revalidate current-build auth session structure - #556
Closed
blakinio wants to merge 7 commits into
Closed
research(track-a): revalidate current-build auth session structure#556blakinio wants to merge 7 commits into
blakinio wants to merge 7 commits into
Conversation
This was referenced Aug 19, 2026
blakinio
commented
Aug 19, 2026
blakinio
left a comment
Owner
Author
There was a problem hiding this comment.
Coordinator decision: ACCEPT_WITH_EDITS.
Independent strict current-build audit reproduced the exact package and required a full QMeta switch control-flow chain (cmp edx,max -> RIP table LEA -> movsxd [table+rdx*4] -> add base -> indirect jmp). It confirmed TGameClient::onRequestLoginWithCredentials index 17 -> 0xd196f0 and connectClientToGameserverWithExistingCredentials index 11 -> 0xd19500.
Required promotion edits:
TLoginRequestUploader::loginSuccessfulindex 0 / target0xd10200is a QMeta SIGNAL dispatch target (SIGNALS=8), not a call-safe business implementation.TCharacterSelectionController::requestCharacterLoginindex 0 / target0xd52050is likewise a QMeta SIGNAL dispatch target (SIGNALS=10).- Source lifecycle text saying #555 remains an independent Draft is stale. Current trusted fence provenance is merged #555 (
2e572789a2bc4b64c5e906c4515c15c625f6bc9e) plus #561 closeout.
No live/login/credential/non-QMeta semantics are promoted. Open material findings after these bounded edits: 0.
This was referenced Aug 19, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Terminal source disposition — superseded by coordinator promotion #569
Status: CLOSED UNMERGED AS SUPERSEDED.
Task:
OTC-20260819-track-a-auth-session-current-build-staticSource head:
411d0e287d08406c682ef063fb3f3f61341d9295.Coordinator review
4970802493classified this source ACCEPT_WITH_EDITS with zero open material findings after repair.Independent coordinator strict audit #568 re-fetched the exact current package and proved the complete QMeta invoke-switch control flow rather than accepting an executable-looking table alone:
Accepted exact-build facts:
The two SIGNAL targets are not promoted as call-safe business implementations. Non-QMeta state-machine targets, field/password provenance, live object/thread provenance, helper call safety, login success,
IN_GAME, and restart/relogin/session stability remain unresolved.Source lifecycle text that described #555 as an unmerged Draft was corrected: current trusted fence authority is #555 merge
2e572789a2bc4b64c5e906c4515c15c625f6bc9eplus #561 closeout.Clean promotion #569 was based on current main, remained
behind_by=0, changed six documentation/evidence/archive paths, passed exact-head CI32239950338, final review4970837047, and squash-merged as:Temporary audit #568 was closed unmerged as audit-only/consumed. No official-client execution, Synology/KasmVNC observation, credentials, login, process-memory access, GUI input or gameplay occurred during coordinator audit/promotion.