Skip to content

Skip module loading for isolated processes#127

Merged
JingMatrix merged 1 commit into
masterfrom
isolated_services
Apr 12, 2026
Merged

Skip module loading for isolated processes#127
JingMatrix merged 1 commit into
masterfrom
isolated_services

Conversation

@JingMatrix
Copy link
Copy Markdown
Owner

We implement a connection check to bypass process flag retrieval and module execution for isolated processes when the zygisk daemon is unreachable.

We implement a connection check to bypass process flag retrieval and module execution for isolated processes when the zygisk daemon is unreachable.
@JingMatrix JingMatrix merged commit de38c62 into master Apr 12, 2026
1 check passed
maxsteeel referenced this pull request in maxsteeel/NeoZygisk-fork Apr 12, 2026
We implement a connection check to bypass process flag retrieval and module execution for isolated processes when the zygisk daemon is unreachable.
larsmartens pushed a commit to larsmartens/NeoZygisk that referenced this pull request Apr 12, 2026
We implement a connection check to bypass process flag retrieval and module execution for isolated processes when the zygisk daemon is unreachable.

(cherry picked from commit de38c62)
JingMatrix added a commit that referenced this pull request Apr 29, 2026
Add a connectivity check for isolated processes to avoid the overhead of attempting flag retrieval and module execution when zygiskd is unreachable.

This optimization does not change the functional logic, as both GetProcessFlags and run_modules_pre already depend on a successful connection to the daemon to perform any meaningful work. If the daemon is unreachable, these steps would eventually fail or be skipped anyway; performing the check upfront simply avoids redundant IPC attempts.

Note: The connection check must use a scoped handle (UniqueFd) to ensure the socket is closed immediately. Failure to do so will result in a file descriptor leak, leading to resource exhaustion and significant app launch delays.
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