Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Weird behaviour of sepolicy patcher (for modules) #1724

Closed
2 of 3 tasks
MDNSSKNGHT opened this issue May 11, 2024 · 5 comments
Closed
2 of 3 tasks

Weird behaviour of sepolicy patcher (for modules) #1724

MDNSSKNGHT opened this issue May 11, 2024 · 5 comments

Comments

@MDNSSKNGHT
Copy link

Please check before submitting an issue

  • I have searched the issues and haven't found anything relevant
  • I will upload bugreport file in KernelSU Manager - Settings - Report log
  • I know how to reproduce the issue which may not be specific to my device

Describe the bug

I made a module for Magisk that makes use of the sepolicy rules patcher to hijack camera process without needing a permissive kernel. Now I'm using KernelSU and my module completely breaks the camera server (which is why the sepolicy patches were needed). The module works if I set kernel to permissive. (using setenforce 0)

Weirdly enough ksud did not complain when applying the custom sepolicy rules.

To Reproduce

No response

Expected behavior

The module should work fine without setting the kernel selinux state to permissive.

Screenshots

No response

Logs

No response

Device info

  • Device: Samsung Galaxy S21FE (SM-G990E)
  • OS Version: Android 14
  • KernelSU Version: commit 1f42bba
  • Kernel Version: 5.4.242

Additional context

No response

@tiann
Copy link
Owner

tiann commented May 11, 2024

@tiann tiann closed this as not planned Won't fix, can't repro, duplicate, stale May 11, 2024
@tiann
Copy link
Owner

tiann commented May 11, 2024

And also, we only accept kernel 5.10+ issues now. #1705

@MDNSSKNGHT
Copy link
Author

@tiann Hello. Thanks for responding, yes I'm using sepolicy.rule file. And is there a way for anyone to assist me on this?

@tiann
Copy link
Owner

tiann commented May 11, 2024

I don't know, we developers haven't any non gki devices now.

@MDNSSKNGHT
Copy link
Author

MDNSSKNGHT commented May 11, 2024

I see. No problem, thanks. This issue was just a matter of file permissions and ownership.

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

No branches or pull requests

2 participants