Skip to content

Conversation

@rsuntk
Copy link
Owner

@rsuntk rsuntk commented Aug 16, 2025

  1. Wrapper for creds:
  • Some older kernel does not have {.val}, so, for nicer compatibility support and clean code,
    make some wrapper for credential use.
  • After this change, do not use current_uid().val, instead, use ksu_current_uid(). For more
    info, check kernel/include/ksu_creds.h.
  1. Refine disable_seccomp (need to add k6.11+ support)
    Reset seccomp filter count when escaping to root tiann/KernelSU#2708
    Seccomp_filters showed 1 in /proc/$ESCALATED_PROC/status tiann/KernelSU#2706

  2. Revert "Handle unmount for isolated process correctly"
    Reason: Handle unmount for isolated process correctly tiann/KernelSU#2696 (comment)

Co-authored-by: Wang Han [email protected]

rsuntk added 6 commits August 15, 2025 23:28
* Some older kernel does not have {.val}, so, for nicer compatibility support and clean code,
make some wrapper for credential use.

* After this change, do not use current_uid().val, instead, use ksu_current_uid(). For more
info, check kernel/include/ksu_creds.h.

Signed-off-by: rsuntk <[email protected]>
Signed-off-by: rsuntk <[email protected]>
* Also rename the cred to newcreds

Signed-off-by: rsuntk <[email protected]>
@rsuntk rsuntk merged commit 5cd5b54 into main Aug 16, 2025
@rsuntk rsuntk deleted the wrap branch August 16, 2025 04:19
luyanci pushed a commit to luyanci/KernelSU that referenced this pull request Aug 26, 2025
…anges (rsuntk#131)

1. Wrapper for creds:
* Some older kernel does not have {.val}, so, for nicer compatibility support and clean code,
make some wrapper for credential use.
* After this change, do not use current_uid().val, instead, use ksu_current_uid(). For more
info, check kernel/include/ksu_creds.h.

2. Refine disable_seccomp (need to add k6.11+ support)
tiann#2708
tiann#2706

3. Revert "Handle unmount for isolated process correctly"
Reason: tiann#2696 (comment)

Co-authored-by: Wang Han <[email protected]>
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