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

UpdaterThread: Work around lying Build.VERSION.SECURITY_PATCH #52

Merged
merged 1 commit into from
Apr 6, 2024

Conversation

chenxiaolong
Copy link
Owner

CalyxOS modifies Build.VERSION.SECURITY_PATCH so that it queries ro.build.version.real_security_patch instead of
ro.build.version.security_patch. However, despite the name, the real_security_patch value is fake. The security patch level in the OTA metadata's postconditions matches the normal security_patch.

Fixes: #51

CalyxOS modifies `Build.VERSION.SECURITY_PATCH` so that it queries
`ro.build.version.real_security_patch` instead of
`ro.build.version.security_patch`. However, despite the name, the
`real_security_patch` value is fake. The security patch level in the OTA
metadata's postconditions matches the normal `security_patch`.

Fixes: #51

Signed-off-by: Andrew Gunnerson <[email protected]>
@chenxiaolong chenxiaolong self-assigned this Apr 4, 2024
chenxiaolong added a commit that referenced this pull request Apr 6, 2024
Signed-off-by: Andrew Gunnerson <[email protected]>
@chenxiaolong chenxiaolong merged commit c43e3a6 into master Apr 6, 2024
4 checks passed
@chenxiaolong chenxiaolong deleted the calyx branch April 6, 2024 17:33
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.

Weird problem with most recent CalyxOS OTA (March) - claims attempt to downgrade security patch and abort
1 participant