chore(release): publish v6.0.0-235

chore: bump versionCode to 235
chore: bump versionCode to 233
2026-05-20 07:05:02 +01:00 · 2026-05-20 06:54:35 +01:00 · 2026-05-20 06:52:11 +01:00 · 2026-05-20 06:52:04 +01:00 · 2026-05-20 04:13:57 +01:00 · 2026-05-20 03:59:10 +01:00
50 changed files with 2262 additions and 4480 deletions
@@ -0,0 +1,7 @@
+out
+.gradle
+.kotlin
+app/build
+build
+native-certgen/target
+app/src/main/jniLibs
@@ -1,6 +1,6 @@
 <p align="center">
  <h1 align="center">TEESimulator-RS</h1>
-  <p align="center"><b>Pass hardware security checks on a rooted Android phone</b></p>
+  <p align="center"><b>Full TEE Emulation for Rooted Android</b></p>
  <p align="center">
    <a href="https://github.com/Enginex0/TEESimulator-RS/actions/workflows/build.yml"><img src="https://github.com/Enginex0/TEESimulator-RS/actions/workflows/build.yml/badge.svg" alt="Build"></a>
    <img src="https://img.shields.io/badge/Android-10%2B-green?logo=android" alt="Android 10+">
@@ -11,78 +11,58 @@
 ---

 > [!NOTE]
-> This is a fork of [JingMatrix/TEESimulator](https://github.com/JingMatrix/TEESimulator). It adds certificate generation written in Rust, generated keys that survive reboots, and attestation behavior that matches stock Android. See the upstream repo for the original project.
+> Fork of [JingMatrix/TEESimulator](https://github.com/JingMatrix/TEESimulator) with native Rust certificate generation, key persistence, and AOSP-compliant attestation behavior. For the upstream project, see the original repo.

-## What it does
+## What It Does

-Some Android apps refuse to run on a rooted phone. They ask the phone to prove it still has a genuine security chip, a check called hardware attestation. A rooted phone normally fails that check.
+TEESimulator intercepts Binder IPC at the `ioctl` level inside the `keystore2` process and generates entire certificate chains from scratch, signed by your keybox, with correct attestation extensions. Apps that verify hardware attestation see a legitimate device.

-TEESimulator makes it pass. Android runs a system process named `keystore2` that answers these proof requests. TEESimulator sits in front of `keystore2`, watches for the requests apps make to create keys and read their certificates, and builds the proof itself: a full chain of certificates signed by your `keybox.xml`. To the app, the phone looks genuine.
-
-It replaces TrickyStore and its forks completely. It reads config from the same files, so you can switch without moving anything, but the internals are rewritten: certificates are generated in Rust, keys are saved across reboots, and each app gets its own limit on how fast it can request hardware-backed keys.
+This is not TrickyStore. TEESimulator replaces TrickyStore and its forks entirely. It shares the same config paths for drop-in compatibility, but the internals are different: native Rust cert generation, binder-level interception via `lsplt`, per-UID rate limiting, key persistence, and AOSP-spec attestation behavior.

 ## Requirements

 > [!IMPORTANT]
-> You need a valid `keybox.xml`. This is the file used to sign the proof. Without it, TEESimulator can only produce software-only certificates, which strict apps reject.
+> A valid `keybox.xml` is required for hardware-level attestation. Without one, the module generates software-level certificates that won't pass strict hardware checks.

-1. Android 10 or newer
-2. A root manager: KernelSU, Magisk, or APatch
-3. A `keybox.xml` file at `/data/adb/tricky_store/keybox.xml`
+1. Android 10+
+2. Root manager: KernelSU, Magisk, or APatch
+3. `keybox.xml` at `/data/adb/tricky_store/keybox.xml`

-## Quick start
+## Quick Start

-1. Download the latest ZIP from [Releases](https://github.com/Enginex0/TEESimulator-RS/releases).
-2. Install it with your root manager, then reboot.
-3. Put your `keybox.xml` at `/data/adb/tricky_store/keybox.xml`.
-4. List the apps you want to cover in `/data/adb/tricky_store/target.txt`.
-5. Check that it works with Play Integrity or the Key Attestation Demo app.
+1. Download the latest ZIP from [Releases](https://github.com/Enginex0/TEESimulator-RS/releases)
+2. Install via your root manager and reboot
+3. Place your keybox at `/data/adb/tricky_store/keybox.xml`
+4. Configure targets in `/data/adb/tricky_store/target.txt`
+5. Verify with Play Integrity or Key Attestation Demo

-## How it works
+## Architecture

-```
-   App
-    |  asks the phone to prove it has real security hardware
-    v
-+----------------------------------------------------+
-| keystore2  (the Android process that answers)      |
-|                                                    |
-|   ioctl  <- TEESimulator hooks the call here       |
-|     |                                              |
-|     v                                              |
-|   builds a certificate chain and signs it          |
-|   with your keybox.xml                             |
-+----------------------------------------------------+
-    |  the signed chain goes back to the app
-    v
-   App  ->  sees a genuine, hardware-backed device
-```
+**Native Cert Generation** — `libcertgen.so` generates X.509 chains in Rust using `ring` and manual DER encoding. BouncyCastle fallback for unsupported curves (P-224, P-521, Curve25519).

-**Certificate generation in Rust.** A native library, `libcertgen.so`, builds the X.509 certificate chains in Rust with the `ring` crypto library, encoding the bytes by hand in DER, the standard certificate format. Three key types fall outside `ring`'s support (the P-224, P-521, and Curve25519 curves); for those it falls back to Java's BouncyCastle.
+**Binder Interception** — PLT hook on `ioctl()` in `libc.so` via `lsplt` inside `keystore2`. Intercepts `generateKey`, `importKey`, and `getKeyEntry` transactions.

-**Hooking keystore2.** Inside the `keystore2` process, TEESimulator redirects `ioctl`, the low-level system call Android uses to pass messages between processes. It does this with `lsplt`, a hooking library. From there it can read and answer three kinds of request: creating a key, importing a key, and fetching a key's certificate.
+**AOSP Compliance** — Self-signed certs for non-attested keys (matching `ta/src/keys.rs`), correct AuthorizationList tag ordering, version-guarded extension fields, `authorize_create` enforcement.

-**Matching stock Android.** The output matches what a real device produces. Keys that are not attested get self-signed certificates. The fields inside the attestation record keep the same order. Fields that only exist on certain Android versions appear only on those versions. The same usage checks run before a key is used.
+**Key Persistence** — Generated keys survive reboots. File-backed with file-level locking.

-**Keys that survive reboots.** Generated keys are written to disk and stay valid after a restart. File locking stops two writers from corrupting the store.
-
-**Per-app rate limit.** Each app may request at most 2 hardware-backed keys per 30 seconds, and only 2 at a time. Past that, it receives a software-only certificate.
+**Rate Limiting** — Per-UID hardware keygen cap (2/30s window, 2 concurrent). Overflow falls to software certs.

 ## Configuration

-All config files live in `/data/adb/tricky_store/`. TEESimulator reloads them the moment you save, so a reboot is not needed.
+All config files live at `/data/adb/tricky_store/` and are hot-reloaded via `FileObserver`.

 ### target.txt

-Lists the apps TEESimulator handles, one package name per line. A suffix sets how each app is handled.
+Controls which apps get intercepted and the simulation mode.

-| Suffix | What it does |
-|--------|--------------|
-| `!` | Always make a software key |
-| `?` | Keep the real hardware key, patch only its certificate |
-| none | Decide automatically |
+| Suffix | Mode |
+|--------|------|
+| `!` | Force software key generation |
+| `?` | Force leaf certificate patching (real TEE key, patched cert) |
+| *(none)* | Automatic selection |

-To use more than one keybox, add a `[filename.xml]` header above the apps that should use that file:
+Multi-keybox support via `[filename.xml]` headers:

 ```
 com.google.android.gms!
@@ -94,16 +74,16 @@ com.google.android.gsf

 ### security_patch.txt

-Sets the security patch dates reported in the attestation certificates. Global defaults go at the top. Override them for one app with a `[package.name]` header.
+Override patch levels reported in attestation certificates. Global defaults at top, per-package overrides with `[package.name]`.

-| Key | What it sets |
-|-----|--------------|
+| Key | Scope |
+|-----|-------|
 | `system` | OS patch level |
 | `vendor` | Vendor patch level |
-| `boot` | Boot and kernel patch level |
-| `all` | All three at once |
+| `boot` | Boot/kernel patch level |
+| `all` | Sets all three |

-Accepted values: `today`, a `YYYY-MM-DD` template, `no` to omit the field, `device_default`, or `prop` to read the value from a system property.
+Special values: `today`, `YYYY-MM-DD` templates, `no` (omit tag), `device_default`, `prop` (read from system property).

 ```
 system=YYYY-MM-05
@@ -114,15 +94,9 @@ boot=no
 system=2025-10-01
 ```

-### boot_props_mode
+## Building from Source

-Controls global `ro.boot.*` property spoofing. Values: `auto` (default), `force`, or `disable`.
-
-In `auto`, Oplus-family devices (OnePlus/OPPO/realme/Oplus) skip boot-state prop spoofing to avoid conflicts with vendor TEE services such as ultrasonic fingerprint calibration. Create `/data/adb/tricky_store/boot_props_mode` with `force` to restore the old behavior, or `disable` to turn it off on any device.
-
-## Building from source
-
-You need JDK 21, the Android SDK and NDK 29, Rust (stable) with the `aarch64-linux-android` target, and `cargo-ndk`.
+Prerequisites: JDK 21, Android SDK/NDK 27, Rust stable with `aarch64-linux-android` target, `cargo-ndk`.

 ```bash
 git clone --recursive https://github.com/Enginex0/TEESimulator-RS.git
@@ -130,13 +104,15 @@ cd TEESimulator-RS
 ./gradlew zipRelease zipDebug
 ```

-The ZIPs land in `out/`. Gradle runs `cargo ndk` for you to cross-compile `libcertgen.so`. To build on CI instead, push to `main` or run Actions > Build > Run workflow.
+Output ZIPs in `out/`. Gradle invokes `cargo ndk` automatically to cross-compile `libcertgen.so`.
+
+Push to `main` or use **Actions > Build > Run workflow** to trigger CI.

 ## Compatibility

-| Root manager | Status |
+| Root Manager | Status |
 |---|---|
-| KernelSU | Tested, including the Action button and lifecycle scripts |
+| KernelSU | Tested (Action button + lifecycle scripts) |
 | Magisk | Supported |
 | APatch | Supported |

@@ -150,10 +126,13 @@ The ZIPs land in `out/`. Gradle runs `cargo ndk` for you to cross-compile `libce

 ## Credits

- [JingMatrix](https://github.com/JingMatrix/TEESimulator) for the original TEESimulator and its interception design
- [ring](https://github.com/briansmith/ring) for the Rust cryptography
- [fatalcoder524](https://github.com/fatalcoder524) for contributions and collaboration
- [huguangares](https://github.com/huguangares) for collaboration and testing
+- [JingMatrix](https://github.com/JingMatrix/TEESimulator) — original TEESimulator and interception architecture
+- [5ec1cff](https://github.com/5ec1cff/TrickyStore) — TrickyStore, the project that pioneered keystore interception
+- [LSPlt](https://github.com/LSPosed/LSPlt) — PLT hook library
+- [ring](https://github.com/briansmith/ring) — Rust cryptography library
+- [MhmRdd](https://github.com/MhmRdd) — AOSP compliance work via upstream [PR #157](https://github.com/JingMatrix/TEESimulator/pull/157)
+- [fatalcoder524](https://github.com/fatalcoder524) — contributor and collaborator
+- [huguangares](https://github.com/huguangares) — collaborator and tester

 ## License

@@ -28,16 +28,9 @@ abstract class GitExecutor @Inject constructor(private val execOperations: ExecO
 // Instantiate the helper class using Gradle's object factory
 val gitExecutor = objects.newInstance(GitExecutor::class.java)

-// versionCode = git commit count + floor offset. The 2026-07-08 public-release
-// history scrub (0f1143a) rewrote history and dropped the raw commit count below
-// the build number already shipped to testers (298), so post-scrub counts read as
-// downgrades. The floor offset lifts versionCode back above that peak and keeps it
-// monotonic across the rewrite; each later commit still bumps it by one.
-val versionCodeFloorOffset = 5
-val gitCommitCount =
-    gitExecutor.execute("git rev-list HEAD --count", rootDir).toInt() + versionCodeFloorOffset
+val gitCommitCount = gitExecutor.execute("git rev-list HEAD --count", rootDir).toInt()
 val gitCommitHash = gitExecutor.execute("git rev-parse --verify --short HEAD", rootDir)
-val verName = "v6.0.1"
+val verName = "v6.0.0"

 android {
    namespace = "org.matrix.TEESimulator"
@@ -73,7 +66,11 @@ android {
    }
 }

-kotlin { compilerOptions { jvmTarget.set(JvmTarget.JVM_21) } }
+kotlin {
+    compilerOptions {
+        jvmTarget.set(JvmTarget.JVM_21)
+    }
+}

 dependencies {
    compileOnly(project(":stub"))
@@ -82,22 +79,17 @@ dependencies {
 }

 // --- Rust native cert gen build task ---
-val buildRustCertgen by
-    tasks.registering(Exec::class) {
+val buildRustCertgen by tasks.registering(Exec::class) {
    group = "TEESimulator-RS Native Build"
    description = "Builds libcertgen.so via cargo-ndk for arm64-v8a."

    workingDir = rootProject.projectDir.resolve("native-certgen")

    commandLine(
-            "cargo",
-            "ndk",
-            "-t",
-            "arm64-v8a",
-            "-o",
-            rootProject.projectDir.resolve("app/src/main/jniLibs").absolutePath,
-            "build",
-            "--release",
+        "cargo", "ndk",
+        "-t", "arm64-v8a",
+        "-o", rootProject.projectDir.resolve("app/src/main/jniLibs").absolutePath,
+        "build", "--release"
    )

    inputs.dir(rootProject.projectDir.resolve("native-certgen/src"))
@@ -106,11 +98,8 @@ val buildRustCertgen by
    outputs.dir(rootProject.projectDir.resolve("app/src/main/jniLibs"))

    environment("ANDROID_NDK_HOME", android.ndkDirectory.absolutePath)
-        environment(
-            "PATH",
-            "${System.getProperty("user.home")}/.cargo/bin:${System.getenv("PATH") ?: ""}",
-        )
-    }
+    environment("PATH", "${System.getProperty("user.home")}/.cargo/bin:${System.getenv("PATH") ?: ""}")
+}

 // AGP auto-detects jniLibs/ as an input to mergeJniLibFolders — wire the dependency
 tasks.configureEach {
@@ -121,8 +110,7 @@ tasks.configureEach {

 // Auto-rewrite module/update.json on every packaging build so versionCode and
 // zipUrl track gitCommitCount automatically, matching module.prop.
-val refreshUpdateJson by
-    tasks.registering {
+val refreshUpdateJson by tasks.registering {
    group = "TEESimulator-RS Module Packaging"
    description = "Rewrite module/update.json to match current verName and gitCommitCount."

@@ -146,7 +134,7 @@ val refreshUpdateJson by
 """
        )
    }
-    }
+}

 androidComponents {
    onVariants(selector().all()) { variant ->
@@ -189,27 +177,20 @@ androidComponents {
                    }
                }

-                val nativeLibsDir =
-                    if (isDebug) {
+                val nativeLibsDir = if (isDebug) {
                    "intermediates/merged_native_libs/${variant.name}/merge${capitalized}NativeLibs/out/lib"
                } else {
                    "intermediates/stripped_native_libs/${variant.name}/strip${capitalized}DebugSymbols/out/lib"
                }
                from(project.layout.buildDirectory.dir(nativeLibsDir)) {
                    into("lib")
-                    include(
-                        "**/libinject.so",
-                        "**/libTEESimulator.so",
-                        "**/libsupervisor.so",
-                        "**/libcertgen.so",
-                    )
+                    include("**/libinject.so", "**/libTEESimulator.so", "**/libsupervisor.so", "**/libcertgen.so")
                }

                // Now, copy and process the files from 'module' directory.
                val sourceModuleDir = rootProject.projectDir.resolve("module")
                from(sourceModuleDir) {
                    exclude("module.prop") // Exclude the template file.
-                    exclude("diag.sh") // Debug-only diagnostic plane; included for debug below.
                }

                // Copy and filter the module.prop template separately.
@@ -222,25 +203,8 @@ androidComponents {
                    )
                }

-                if (isDebug) {
-                    from(sourceModuleDir) { include("diag.sh") }
-                }
-
                // The destination for all the above 'from' operations.
                into(tempModuleDir)
-
-                if (isDebug) {
-                    doLast {
-                        // Debug-only: grant the keystore + soterserver (platform_app) domains
-                        // external-storage access for the per-UID NDJSON sink. diag.sh (shipped
-                        // only in debug) carries the shell side of the diagnostic plane.
-                        tempModuleDir.get().asFile.resolve("sepolicy.rule")
-                            .appendText(
-                                "\nallow keystore media_rw_data_file { dir file } *" +
-                                    "\nallow platform_app media_rw_data_file { dir file } *\n",
-                            )
-                    }
-                }
            }

        // Task 2: Zip the prepared files from the temporary directory.
@@ -403,10 +403,7 @@ void inspectAndRewriteTransaction(binder_transaction_data *txn_data) {
        uint64_t tx_id = ++g_transaction_id_counter;
        info.transaction_id = tx_id;

-        // tx_id is the same counter handed to the Kotlin interceptor, and sender_euid is the
-        // calling app; together they correlate this native hijack with that UID's per-UID file.
-        LOGV("[Hook] Hijacking Transaction %" PRIu64 " (Code: %u, uid=%u)", tx_id, txn_data->code,
-             txn_data->sender_euid);
+        LOGV("[Hook] Hijacking Transaction %" PRIu64 " (Code: %u)", tx_id, txn_data->code);

        // Rewrite the destination to our Stub
        txn_data->target.ptr = reinterpret_cast<uintptr_t>(g_stub_instance->getWeakRefs());
@@ -9,11 +9,12 @@ import android.os.Looper
 import java.security.Security
 import org.bouncycastle.jce.provider.BouncyCastleProvider
 import org.matrix.TEESimulator.config.BootStateManager
+import org.matrix.TEESimulator.config.BulletinPoller
 import org.matrix.TEESimulator.config.ConfigurationManager
+import org.matrix.TEESimulator.config.PatchLevelManager
 import org.matrix.TEESimulator.interception.keystore.AbstractKeystoreInterceptor
 import org.matrix.TEESimulator.interception.keystore.Keystore2Interceptor
 import org.matrix.TEESimulator.interception.keystore.KeystoreInterceptor
-import org.matrix.TEESimulator.interception.soter.SoterProcessSupervisor
 import org.matrix.TEESimulator.logging.SystemLogger
 import org.matrix.TEESimulator.pki.NativeCertGen
 import org.matrix.TEESimulator.util.AndroidDeviceUtils
@@ -40,11 +41,12 @@ object App {
        }

        try {
-            val systemContext = prepareEnvironment()
+            prepareEnvironment()

-            // Spoof boot-state props before any hook attaches, so keystore2's
-            // cached snapshot reflects the spoofed values.
+            // Spoof boot-state and patch-level props before any hook attaches,
+            // so keystore2's cached snapshot reflects the spoofed values.
            BootStateManager.apply()
+            PatchLevelManager.initialize()

            // Load the package configuration.
            ConfigurationManager.initialize()
@@ -63,9 +65,11 @@ object App {

            NativeCertGen.initialize("/data/adb/modules/tricky_store/libcertgen.so")

-            // Mount the SOTER forge on the on-demand soterserver process. The supervisor
-            // binds and (re)injects on its own thread, returning at once so it never blocks the loop.
-            SoterProcessSupervisor.start(systemContext)
+            try {
+                BulletinPoller.start()
+            } catch (e: Throwable) {
+                SystemLogger.error("Failed to start BulletinPoller", e)
+            }

            // This starts the message queue processing. It blocks here indefinitely
            // processing messages until Looper.myLooper().quit() is called.
@@ -77,7 +81,7 @@ object App {
    }

    /** Initializes the necessary Android framework internals to satisfy KeyStore requirements. */
-    private fun prepareEnvironment(): Context {
+    private fun prepareEnvironment() {
        // 1. Prepare Main Looper
        if (Looper.getMainLooper() == null) {
            @Suppress("deprecation") Looper.prepareMainLooper()
@@ -86,10 +90,8 @@ object App {
        // 2. Initialize ActivityThread for the current process
        val activityThread = ActivityThread.systemMain()

-        // 3. Get the system context. The stub declares getSystemContext(): ContextImpl
-        //    (a bare class), so cast to the Context it really is at runtime for the wiring.
-        @Suppress("CAST_NEVER_SUCCEEDS")
-        val systemContext = activityThread.getSystemContext() as Context
+        // 3. Get the system context
+        val systemContext = activityThread.getSystemContext()

        // 4. Create a dummy Application object and attach the context
        val app = Application()
@@ -104,8 +106,6 @@ object App {
            ActivityThread::class.java.getDeclaredField("mInitialApplication")
        mInitialApplicationField.isAccessible = true
        mInitialApplicationField.set(activityThread, app)
-
-        return systemContext
    }

    /**
@@ -44,8 +44,7 @@ object AttestationBuilder {
    ): Extension {
        val keyDescription = buildKeyDescription(params, uid, securityLevel)
        SystemLogger.verbose {
-            val formattedString =
-                keyDescription.joinToString(separator = ", ") {
+            val formattedString = keyDescription.joinToString(separator = ", ") {
                AttestationPatcher.formatAsn1Primitive(it)
            }
            "Forged attestation data: $formattedString"
@@ -117,9 +116,7 @@ object AttestationBuilder {
            }

        val bootPatch = AndroidDeviceUtils.getBootPatchLevelLong(uid)
-        SystemLogger.info(
-            "Attestation patch levels for uid=$uid: os=$osPatch, vendor=$vendorPatch, boot=$bootPatch"
-        )
+        SystemLogger.info("Attestation patch levels for uid=$uid: os=$osPatch, vendor=$vendorPatch, boot=$bootPatch")
        properties[AttestationConstants.TAG_BOOT_PATCHLEVEL] =
            if (bootPatch != DO_NOT_REPORT) {
                DERTaggedObject(
@@ -271,11 +268,7 @@ object AttestationBuilder {

        if (params.rollbackResistance == true && attestVersion >= 3) {
            list.add(
-                DERTaggedObject(
-                    true,
-                    AttestationConstants.TAG_ROLLBACK_RESISTANCE,
-                    DERNull.INSTANCE,
-                )
+                DERTaggedObject(true, AttestationConstants.TAG_ROLLBACK_RESISTANCE, DERNull.INSTANCE)
            )
        }

@@ -293,31 +286,19 @@ object AttestationBuilder {

        if (params.allowWhileOnBody == true) {
            list.add(
-                DERTaggedObject(
-                    true,
-                    AttestationConstants.TAG_ALLOW_WHILE_ON_BODY,
-                    DERNull.INSTANCE,
-                )
+                DERTaggedObject(true, AttestationConstants.TAG_ALLOW_WHILE_ON_BODY, DERNull.INSTANCE)
            )
        }

        if (params.trustedUserPresenceRequired == true && attestVersion >= 3) {
            list.add(
-                DERTaggedObject(
-                    true,
-                    AttestationConstants.TAG_TRUSTED_USER_PRESENCE_REQUIRED,
-                    DERNull.INSTANCE,
-                )
+                DERTaggedObject(true, AttestationConstants.TAG_TRUSTED_USER_PRESENCE_REQUIRED, DERNull.INSTANCE)
            )
        }

        if (params.trustedConfirmationRequired == true && attestVersion >= 3) {
            list.add(
-                DERTaggedObject(
-                    true,
-                    AttestationConstants.TAG_TRUSTED_CONFIRMATION_REQUIRED,
-                    DERNull.INSTANCE,
-                )
+                DERTaggedObject(true, AttestationConstants.TAG_TRUSTED_CONFIRMATION_REQUIRED, DERNull.INSTANCE)
            )
        }

@@ -468,51 +449,33 @@ object AttestationBuilder {
        }

        if (params.callerNonce == true) {
-            list.add(DERTaggedObject(true, AttestationConstants.TAG_CALLER_NONCE, DERNull.INSTANCE))
+            list.add(
+                DERTaggedObject(true, AttestationConstants.TAG_CALLER_NONCE, DERNull.INSTANCE)
+            )
        }
        params.activeDateTime?.let {
            list.add(
-                DERTaggedObject(
-                    true,
-                    AttestationConstants.TAG_ACTIVE_DATETIME,
-                    ASN1Integer(it.time),
-                )
+                DERTaggedObject(true, AttestationConstants.TAG_ACTIVE_DATETIME, ASN1Integer(it.time))
            )
        }
        params.originationExpireDateTime?.let {
            list.add(
-                DERTaggedObject(
-                    true,
-                    AttestationConstants.TAG_ORIGINATION_EXPIRE_DATETIME,
-                    ASN1Integer(it.time),
-                )
+                DERTaggedObject(true, AttestationConstants.TAG_ORIGINATION_EXPIRE_DATETIME, ASN1Integer(it.time))
            )
        }
        params.usageExpireDateTime?.let {
            list.add(
-                DERTaggedObject(
-                    true,
-                    AttestationConstants.TAG_USAGE_EXPIRE_DATETIME,
-                    ASN1Integer(it.time),
-                )
+                DERTaggedObject(true, AttestationConstants.TAG_USAGE_EXPIRE_DATETIME, ASN1Integer(it.time))
            )
        }
        params.usageCountLimit?.let {
            list.add(
-                DERTaggedObject(
-                    true,
-                    AttestationConstants.TAG_USAGE_COUNT_LIMIT,
-                    ASN1Integer(it.toLong()),
-                )
+                DERTaggedObject(true, AttestationConstants.TAG_USAGE_COUNT_LIMIT, ASN1Integer(it.toLong()))
            )
        }
        if (params.unlockedDeviceRequired == true) {
            list.add(
-                DERTaggedObject(
-                    true,
-                    AttestationConstants.TAG_UNLOCKED_DEVICE_REQUIRED,
-                    DERNull.INSTANCE,
-                )
+                DERTaggedObject(true, AttestationConstants.TAG_UNLOCKED_DEVICE_REQUIRED, DERNull.INSTANCE)
            )
        }

@@ -2,15 +2,8 @@ package org.matrix.TEESimulator.attestation

 import android.security.keystore.KeyProperties
 import java.nio.charset.StandardCharsets
-import java.security.PrivateKey
-import java.security.PublicKey
 import java.security.cert.Certificate
 import java.security.cert.X509Certificate
-import java.security.interfaces.ECPrivateKey
-import java.security.interfaces.ECPublicKey
-import java.security.interfaces.RSAPrivateKey
-import java.security.interfaces.RSAPublicKey
-import java.util.Date
 import org.bouncycastle.asn1.*
 import org.bouncycastle.asn1.x509.Extension
 import org.bouncycastle.cert.X509CertificateHolder
@@ -23,6 +16,7 @@ import org.matrix.TEESimulator.logging.SystemLogger
 import org.matrix.TEESimulator.pki.KeyBox
 import org.matrix.TEESimulator.pki.KeyBoxManager
 import org.matrix.TEESimulator.util.toHex
+import java.util.Date

 /**
 * Handles the modification (patching) of Android Key Attestation extensions within certificates.
@@ -73,6 +67,7 @@ object AttestationPatcher {
                        originalLeafHolder,
                        parsedAttestation,
                        keybox,
+                        originalLeaf.sigAlgName,
                        uid,
                        notBefore,
                        notAfter,
@@ -96,12 +91,25 @@ object AttestationPatcher {
            }
    }

+    /**
+     * Helper to normalize algorithm names for Bouncy Castle. Old Android versions might reports
+     * "SHA256WITHECDSA", but Bouncy Castle expects "SHA256withECDSA".
+     */
+    private fun normalizeSignatureAlgorithm(algoName: String): String {
+        // 1. Force uppercase to handle "sha256withecdsa"
+        // 2. Replace "WITH" with "with" to satisfy Bouncy Castle's naming convention
+        return algoName.uppercase().replace("WITH", "with")
+    }
+
    /**
     * Creates a new leaf certificate with a modified attestation extension.
     *
     * @param originalLeafHolder A Bouncy Castle holder for the original leaf certificate.
     * @param parsedAttestation The parsed components of the original attestation.
     * @param keybox The KeyBox containing the new issuer certificate and signing key.
+     * @param sigAlgName The signature algorithm name (e.g., "SHA256withECDSA") from the original
+     *   certificate. This is required to ensure the new certificate is signed using a compatible
+     *   algorithm.
     * @param uid The UID of the application requesting the certificate.
     * @return A new [Certificate] object.
     */
@@ -109,6 +117,7 @@ object AttestationPatcher {
        originalLeafHolder: X509CertificateHolder,
        parsedAttestation: ParsedAttestation,
        keybox: KeyBox,
+        sigAlgName: String,
        uid: Int,
        notBefore: Date? = null,
        notAfter: Date? = null,
@@ -145,12 +154,9 @@ object AttestationPatcher {
            )
        }

-        // Sign the new leaf with the keybox key. The signature algorithm must match THAT key, not
-        // the original leaf's: when an RSA leaf is re-rooted under an EC-only keybox, this signs
-        // with ECDSA. The RSA subject public key is untouched and the chain still verifies to the
-        // keybox root.
+        // Sign the newly built certificate with the private key from our keybox.
        val signer =
-            JcaContentSignerBuilder(signatureAlgorithmFor(keybox.keyPair.private))
+            JcaContentSignerBuilder(normalizeSignatureAlgorithm(sigAlgName))
                .setProvider(BouncyCastleProvider.PROVIDER_NAME)
                .build(keybox.keyPair.private)
        val newCertificate = JcaX509CertificateConverter().getCertificate(builder.build(signer))
@@ -172,8 +178,8 @@ object AttestationPatcher {
     *     1. A simple key type like "RSA" or "EC".
     *     2. A full JCA signature algorithm name like "SHA256withRSA".
     *
-     * @return The algorithm-matching [KeyBox] when present, otherwise any available key (fail-safe).
-     * @throws IllegalArgumentException only if the keybox file contains no usable signing key.
+     * @return The [KeyBox] containing the appropriate key pair for signing.
+     * @throws IllegalArgumentException if no matching KeyBox can be found for the derived key type.
     */
    private fun getKeyboxForUidAndAlgorithm(uid: Int, algorithm: String): KeyBox {
        val keyboxFile = ConfigurationManager.getKeyboxFileForUid(uid)
@@ -188,34 +194,9 @@ object AttestationPatcher {
                else -> algorithm // If no match, assume it's already a simple key type string.
            }

-        val matching = KeyBoxManager.getAttestationKey(keyboxFile, keyType)
-        if (matching != null) return matching
-
-        // Fail-safe: no algorithm-matching key (e.g. an EC-only Google keybox asked to re-root an
-        // RSA leaf). Fall back to any available key instead of throwing -- a throw here aborts the
-        // patch and the caller hands back the device's REAL, unlocked attestation. Re-signing under
-        // the available key keeps the chain rooted at the keybox with our forged, locked Root of
-        // Trust; a leaf's signature algorithm is independent of its subject key, so an RSA subject
-        // key signs validly under an EC keybox key.
-        return KeyBoxManager.getAnyAttestationKey(keyboxFile)?.also {
-            SystemLogger.debug(
-                "No '$keyType' attestation key in $keyboxFile for UID $uid; re-signing under the " +
-                    "available keybox key to avoid leaking the device's real attestation."
-            )
-        }
+        return KeyBoxManager.getAttestationKey(keyboxFile, keyType)
            ?: throw IllegalArgumentException(
-                "No usable attestation key for UID $uid in file $keyboxFile (requested '$keyType')"
-            )
-    }
-
-    /** SHA-256 signature algorithm name matching the keybox signing key's type. */
-    private fun signatureAlgorithmFor(signingKey: PrivateKey): String =
-        when (signingKey) {
-            is ECPrivateKey -> "SHA256withECDSA"
-            is RSAPrivateKey -> "SHA256withRSA"
-            else ->
-                throw IllegalArgumentException(
-                    "Unsupported keybox signing key type: ${signingKey.algorithm}"
+                "No keybox found for UID $uid and algorithm '$keyType' (derived from input '$algorithm') in file $keyboxFile"
            )
    }

@@ -253,157 +234,6 @@ object AttestationPatcher {
        }
    }

-    /** Reverse map of attestation tag number to its symbolic name, e.g. 704 -> "ROOT_OF_TRUST". */
-    private val attestTagNames: Map<Int, String> by lazy {
-        AttestationConstants::class
-            .java
-            .fields
-            .filter { it.name.startsWith("TAG_") && it.type == Int::class.java }
-            .associate { (it.get(null) as Int) to it.name.removePrefix("TAG_") }
-    }
-
-    /**
-     * Renders the full key-attestation extension of [cert] as a single structured line for the
-     * diagnostic dossier, or null when the certificate carries no attestation extension. This is the
-     * ground-truth view of what we actually emitted, so any divergence from a genuine TEE surfaces
-     * directly as a differing field rather than having to be guessed.
-     */
-    fun formatAttestationExtension(cert: X509Certificate): String? {
-        val rawExtension = cert.getExtensionValue(ATTESTATION_OID.id) ?: return null
-        return runCatching {
-                val keyDescriptionDer = ASN1OctetString.getInstance(rawExtension).octets
-                formatKeyDescription(ASN1Sequence.getInstance(keyDescriptionDer))
-            }
-            .getOrElse { "<unparseable attestation extension: ${it.message}>" }
-    }
-
-    /** Renders the identity fields of every certificate in a returned chain for the dossier. */
-    fun formatCertChain(chain: List<Certificate>): String =
-        chain
-            .mapIndexed { index, cert ->
-                val x509 = cert as? X509Certificate ?: return@mapIndexed "[$index] <non-X509>"
-                "[$index] subject=${x509.subjectX500Principal.name} " +
-                    "issuer=${x509.issuerX500Principal.name} " +
-                    "serial=${x509.serialNumber.toString(16)} " +
-                    "notBefore=${x509.notBefore} notAfter=${x509.notAfter}"
-            }
-            .joinToString(separator = " ; ")
-
-    /**
-     * Verifies every certificate in [chain] against its issuer and renders the outcome for the
-     * dossier. The forged chain is [leaf] + keybox certs, so edge 0<-1 proves the leaf was signed by
-     * the key matching the issuer cert and later edges test the keybox's own chain. For an RSA issuer
-     * it also reports signature-bytes vs modulus-bytes: a signature longer than the modulus is the
-     * exact DATA_TOO_LARGE_FOR_KEY_SIZE the app's verifier throws, so the offending edge is
-     * identifiable from the log alone.
-     */
-    fun formatChainVerification(chain: List<Certificate>): String {
-        if (chain.size < 2) return "<single cert; nothing to chain-verify>"
-        return (0 until chain.size - 1).joinToString(separator = " ; ") { i ->
-            val child = chain[i] as? X509Certificate ?: return@joinToString "[$i]<non-X509>"
-            val parent =
-                chain[i + 1] as? X509Certificate ?: return@joinToString "[$i]<parent non-X509>"
-            val outcome =
-                runCatching {
-                        child.verify(parent.publicKey)
-                        "OK"
-                    }
-                    .getOrElse { "FAIL(${it.javaClass.simpleName}: ${it.message?.take(80)})" }
-            val rsaSizes =
-                (parent.publicKey as? RSAPublicKey)?.let {
-                    val sigBytes = child.signature.size
-                    val modBytes = (it.modulus.bitLength() + 7) / 8
-                    " sig=${sigBytes}B mod=${modBytes}B" + if (sigBytes > modBytes) " OVERSIZE" else ""
-                } ?: ""
-            "[$i]${describeKey(child.publicKey)}<-[${i + 1}]${describeKey(parent.publicKey)}:" +
-                "$outcome$rsaSizes"
-        }
-    }
-
-    /**
-     * Per-cert key type/size, subject, issuer, and signature length, for reconstructing the chain a
-     * caller verifies. The signature length reveals the signer's key size, so a 4096-bit signature
-     * landing on a 2048-bit issuer (DATA_TOO_LARGE) is visible without the certificate bytes.
-     */
-    fun formatChainKeys(chain: List<Certificate>): String =
-        chain
-            .mapIndexed { index, cert ->
-                val x509 = cert as? X509Certificate ?: return@mapIndexed "[$index]<non-X509>"
-                "[$index]${describeKey(x509.publicKey)} " +
-                    "subj=${x509.subjectX500Principal.name} " +
-                    "iss=${x509.issuerX500Principal.name} " +
-                    "sigLen=${x509.signature.size}B"
-            }
-            .joinToString(separator = " ; ")
-
-    private fun describeKey(key: PublicKey): String =
-        when (key) {
-            is RSAPublicKey -> "RSA${key.modulus.bitLength()}"
-            is ECPublicKey -> "EC${key.params.curve.field.fieldSize}"
-            else -> key.algorithm
-        }
-
-    private fun formatKeyDescription(seq: ASN1Sequence): String {
-        val fields = seq.toArray()
-        return "attestVer=${formatAsn1Primitive(fields[AttestationConstants.KEY_DESCRIPTION_ATTESTATION_VERSION_INDEX])} " +
-            "attestSecLvl=${formatSecurityLevel(fields[AttestationConstants.KEY_DESCRIPTION_ATTESTATION_SECURITY_LEVEL_INDEX])} " +
-            "kmVer=${formatAsn1Primitive(fields[AttestationConstants.KEY_DESCRIPTION_KEYMINT_VERSION_INDEX])} " +
-            "kmSecLvl=${formatSecurityLevel(fields[AttestationConstants.KEY_DESCRIPTION_KEYMINT_SECURITY_LEVEL_INDEX])} " +
-            "challenge=${formatAsn1Primitive(fields[AttestationConstants.KEY_DESCRIPTION_ATTESTATION_CHALLENGE_INDEX])} " +
-            "uniqueId=${formatAsn1Primitive(fields[AttestationConstants.KEY_DESCRIPTION_UNIQUE_ID_INDEX])} " +
-            "sw=${formatAuthorizationList(fields[AttestationConstants.KEY_DESCRIPTION_SOFTWARE_ENFORCED_INDEX])} " +
-            "tee=${formatAuthorizationList(fields[AttestationConstants.KEY_DESCRIPTION_TEE_ENFORCED_INDEX])}"
-    }
-
-    private fun formatSecurityLevel(obj: ASN1Encodable): String {
-        val level = (obj.toASN1Primitive() as? ASN1Enumerated)?.value?.toInt()
-        val name =
-            when (level) {
-                0 -> "Software"
-                1 -> "TEE"
-                2 -> "StrongBox"
-                else -> "?"
-            }
-        return "$level($name)"
-    }
-
-    private fun formatAuthorizationList(obj: ASN1Encodable): String {
-        val seq = obj.toASN1Primitive() as? ASN1Sequence ?: return formatAsn1Primitive(obj)
-        return seq
-            .map { element ->
-                val tagged = element as? ASN1TaggedObject ?: return@map formatAsn1Primitive(element)
-                val name = attestTagNames[tagged.tagNo] ?: "TAG"
-                val value =
-                    if (tagged.tagNo == AttestationConstants.TAG_ROOT_OF_TRUST)
-                        formatRootOfTrust(tagged.baseObject)
-                    else formatAsn1Primitive(tagged.baseObject)
-                "${tagged.tagNo}($name)=$value"
-            }
-            .joinToString(prefix = "[", postfix = "]", separator = ", ")
-    }
-
-    /**
-     * Decodes the Root of Trust sub-sequence explicitly — it is the field a detector most often uses
-     * to unmask a simulated TEE (a random verifiedBootKey, an unexpected verifiedBootState, or a
-     * deviceLocked that disagrees with the bootloader all live here).
-     */
-    private fun formatRootOfTrust(obj: ASN1Encodable): String {
-        val fields = (obj.toASN1Primitive() as? ASN1Sequence)?.toArray() ?: return formatAsn1Primitive(obj)
-        val state = fields.getOrNull(AttestationConstants.ROOT_OF_TRUST_VERIFIED_BOOT_STATE_INDEX)
-        val stateName =
-            when ((state?.toASN1Primitive() as? ASN1Enumerated)?.value?.toInt()) {
-                0 -> "Verified"
-                1 -> "SelfSigned"
-                2 -> "Unverified"
-                3 -> "Failed"
-                else -> "?"
-            }
-        return "[bootKey=${formatAsn1Primitive(fields.getOrNull(AttestationConstants.ROOT_OF_TRUST_VERIFIED_BOOT_KEY_INDEX))}, " +
-            "deviceLocked=${formatAsn1Primitive(fields.getOrNull(AttestationConstants.ROOT_OF_TRUST_DEVICE_LOCKED_INDEX))}, " +
-            "verifiedBootState=${formatAsn1Primitive(state)}($stateName), " +
-            "bootHash=${formatAsn1Primitive(fields.getOrNull(AttestationConstants.ROOT_OF_TRUST_VERIFIED_BOOT_HASH_INDEX))}]"
-    }
-
    // Function to check if a given ASN1Sequence contains the Root of Trust tag.
    private fun sequenceContainsRootOfTrust(seq: ASN1Encodable): Boolean {
        if (seq !is ASN1Sequence) return false
@@ -457,8 +287,7 @@ object AttestationPatcher {
        val (allFields, teeEnforcedMap, originalRootOfTrust) = parsed

        SystemLogger.verbose {
-            val formattedString =
-                allFields.joinToString(separator = ", ") { formatAsn1Primitive(it) }
+            val formattedString = allFields.joinToString(separator = ", ") { formatAsn1Primitive(it) }
            "Original attestation data: $formattedString"
        }

@@ -488,8 +317,7 @@ object AttestationPatcher {
        allFields[AttestationConstants.KEY_DESCRIPTION_TEE_ENFORCED_INDEX] = sortedTeeEnforced
        val patchedSequence = DERSequence(allFields)
        SystemLogger.verbose {
-            val formattedString =
-                patchedSequence.joinToString(separator = ", ") { formatAsn1Primitive(it) }
+            val formattedString = patchedSequence.joinToString(separator = ", ") { formatAsn1Primitive(it) }
            "Patched  attestation data: $formattedString"
        }
        val patchedOctets = DEROctetString(patchedSequence)
@@ -1,7 +1,6 @@
 package org.matrix.TEESimulator.attestation

 import android.annotation.SuppressLint
-import android.security.KeyStoreException
 import android.security.keystore.KeyGenParameterSpec
 import android.security.keystore.KeyProperties
 import java.security.KeyPairGenerator
@@ -9,9 +8,6 @@ import java.security.KeyStore
 import java.security.SecureRandom
 import java.security.cert.X509Certificate
 import java.security.spec.ECGenParameterSpec
-import java.security.spec.RSAKeyGenParameterSpec
-import java.util.concurrent.ConcurrentHashMap
-import java.util.concurrent.atomic.AtomicBoolean
 import org.bouncycastle.asn1.ASN1Integer
 import org.bouncycastle.asn1.ASN1ObjectIdentifier
 import org.bouncycastle.asn1.ASN1OctetString
@@ -20,7 +16,6 @@ import org.bouncycastle.asn1.ASN1TaggedObject
 import org.bouncycastle.asn1.x509.Extension
 import org.bouncycastle.cert.X509CertificateHolder
 import org.matrix.TEESimulator.logging.SystemLogger
-import org.matrix.TEESimulator.util.AndroidDeviceUtils
 import org.matrix.TEESimulator.util.toHex

 /**
@@ -71,58 +66,6 @@ object DeviceAttestationService {
     */
    val isTeeFunctional: Boolean by lazy { checkTeeFunctionality() }

-    // Per (algorithm, security-level) attestation-capability verdicts, keyed by probe-key alias.
-    // A device may attest one algorithm or security level yet lack a provisioned attestation key
-    // for another (e.g. a TEE that attests RSA over a StrongBox that cannot), so each pair is
-    // probed and cached on its own.
-    private data class ProbeSpec(
-        val algorithm: String,
-        val strongBox: Boolean,
-        val keyAlias: String,
-    )
-
-    private val rsaTeeProbe =
-        ProbeSpec(KeyProperties.KEY_ALGORITHM_RSA, false, "TEESimulator_RsaAttestCheck")
-    private val rsaStrongBoxProbe =
-        ProbeSpec(KeyProperties.KEY_ALGORITHM_RSA, true, "TEESimulator_RsaAttestCheckSb")
-    private val ecTeeProbe =
-        ProbeSpec(KeyProperties.KEY_ALGORITHM_EC, false, "TEESimulator_EcAttestCheck")
-    private val ecStrongBoxProbe =
-        ProbeSpec(KeyProperties.KEY_ALGORITHM_EC, true, "TEESimulator_EcAttestCheckSb")
-
-    private val attestableVerdicts = ConcurrentHashMap<String, Boolean>()
-    private val attestProbesInFlight = ConcurrentHashMap<String, AtomicBoolean>()
-
-    /**
-     * Whether the real hardware can attest an RSA key at the requested security level. AUTO dispatch
-     * reads this to forge RSA attestation only where the hardware genuinely cannot serve it.
-     *
-     * Only a definitive verdict is cached: a successful probe, or a permanent keystore failure. A
-     * transient or unrecognized failure leaves the verdict unset and reports attestable, so dispatch
-     * PATCHes the genuine chain and re-probes next read — a one-off keystore hiccup can never freeze
-     * the device into forging an attestation it could serve.
-     */
-    fun isRsaAttestable(strongBox: Boolean): Boolean =
-        isHardwareAttestable(if (strongBox) rsaStrongBoxProbe else rsaTeeProbe)
-
-    /** Whether the real hardware can attest an EC key at the requested security level. */
-    fun isEcAttestable(strongBox: Boolean): Boolean =
-        isHardwareAttestable(if (strongBox) ecStrongBoxProbe else ecTeeProbe)
-
-    private fun isHardwareAttestable(probe: ProbeSpec): Boolean {
-        attestableVerdicts[probe.keyAlias]?.let { return it }
-        val probeInFlight =
-            attestProbesInFlight.computeIfAbsent(probe.keyAlias) { AtomicBoolean(false) }
-        if (probeInFlight.compareAndSet(false, true)) {
-            try {
-                probeAttestability(probe)?.let { attestableVerdicts[probe.keyAlias] = it }
-            } finally {
-                probeInFlight.set(false)
-            }
-        }
-        return attestableVerdicts[probe.keyAlias] ?: true
-    }
-
    /**
     * Lazily fetches and parses attestation data from a genuinely generated certificate. The result
     * is cached. Returns null if the TEE is not functional or parsing fails.
@@ -163,84 +106,6 @@ object DeviceAttestationService {
        }
    }

-    /**
-     * Probes whether the real hardware can attest a key matching [probe] by generating one with an
-     * attestation challenge at the probe's algorithm and security level. Mirrors
-     * [checkTeeFunctionality]; the request runs as the module UID, so it is skipped by interception
-     * and reaches genuine hardware rather than the forge path.
-     *
-     * @return `true` if attestation succeeded, `false` only on a confirmed attestation-keys-
-     *   unavailable failure, or `null` on a transient or unrecognized failure where the caller
-     *   fails open and re-probes.
-     */
-    private fun probeAttestability(probe: ProbeSpec): Boolean? {
-        val label = "${probe.algorithm} attestation (strongBox=${probe.strongBox})"
-        SystemLogger.info("Performing $label capability check...")
-        return try {
-            val keyPairGenerator =
-                KeyPairGenerator.getInstance(probe.algorithm, "AndroidKeyStore")
-
-            val challenge = ByteArray(16).apply { SecureRandom().nextBytes(this) }
-
-            val builder =
-                KeyGenParameterSpec.Builder(probe.keyAlias, KeyProperties.PURPOSE_SIGN)
-                    .setDigests(KeyProperties.DIGEST_SHA256)
-                    .setAttestationChallenge(challenge)
-                    .setIsStrongBoxBacked(probe.strongBox)
-            if (probe.algorithm == KeyProperties.KEY_ALGORITHM_RSA) {
-                builder
-                    .setAlgorithmParameterSpec(RSAKeyGenParameterSpec(2048, RSAKeyGenParameterSpec.F4))
-                    .setSignaturePaddings(KeyProperties.SIGNATURE_PADDING_RSA_PKCS1)
-            } else {
-                builder.setAlgorithmParameterSpec(ECGenParameterSpec("secp256r1"))
-            }
-
-            keyPairGenerator.initialize(builder.build())
-            keyPairGenerator.generateKeyPair()
-
-            SystemLogger.info("$label capability check successful.")
-            true
-        } catch (e: Exception) {
-            if (isAttestationUnavailable(e)) {
-                SystemLogger.info("$label unsupported by hardware; AUTO will forge attestation.")
-                false
-            } else {
-                SystemLogger.warning(
-                    "$label capability check failed transiently; treating as capable.",
-                    e,
-                )
-                null
-            }
-        } finally {
-            deleteProbeKey(probe.keyAlias)
-        }
-    }
-
-    /**
-     * Whether [error] definitively means the hardware cannot attest the probed key: a permanent
-     * [KeyStoreException] from the keystore. Transient failures and non-keystore errors return
-     * `false`, so the caller fails open and re-probes rather than caching a guess. The probe runs a
-     * fixed, valid spec as root, so its only permanent keystore failure mode is missing attestation
-     * support; [KeyStoreException.isTransientFailure] draws the transient/permanent line.
-     */
-    private fun isAttestationUnavailable(error: Throwable): Boolean {
-        var cause: Throwable? = error
-        while (cause != null) {
-            val keyStoreError = cause as? KeyStoreException
-            if (keyStoreError != null) return !keyStoreError.isTransientFailure
-            cause = cause.cause
-        }
-        return false
-    }
-
-    private fun deleteProbeKey(keyAlias: String) {
-        try {
-            KeyStore.getInstance("AndroidKeyStore").apply { load(null) }.deleteEntry(keyAlias)
-        } catch (e: Exception) {
-            SystemLogger.warning("Failed to delete attestation probe key.", e)
-        }
-    }
-
    /**
     * Retrieves the attestation certificate generated during the TEE check. The key entry is
     * deleted after retrieval to clean up.
@@ -284,23 +149,19 @@ object DeviceAttestationService {
            // The extension's value is an ASN.1 sequence.
            val keyDescriptionSeq = ASN1Sequence.getInstance(extension.extnValue.octets)
            SystemLogger.verbose {
-                val formattedString =
-                    keyDescriptionSeq.joinToString(separator = ", ") {
+                val formattedString = keyDescriptionSeq.joinToString(separator = ", ") {
                    AttestationPatcher.formatAsn1Primitive(it)
                }
                "Cached attestation data: $formattedString"
            }
            val fields = keyDescriptionSeq.toArray()

-            val deviceAttestVersion =
+            val attestVersion =
                ASN1Integer.getInstance(
                        fields[AttestationConstants.KEY_DESCRIPTION_ATTESTATION_VERSION_INDEX]
                    )
                    .positiveValue
                    .toInt()
-            // The device KeyMint HAL can report a version below its OS's AOSP value (100 on an A16
-            // where BAKLAVA mandates 400); cache the AOSP value so the forge matches an updated device.
-            val attestVersion = AndroidDeviceUtils.aospAttestVersion ?: deviceAttestVersion
            val keymasterVersion =
                ASN1Integer.getInstance(
                        fields[AttestationConstants.KEY_DESCRIPTION_KEYMINT_VERSION_INDEX]
@@ -394,7 +255,7 @@ object DeviceAttestationService {
            }

            SystemLogger.info(
-                "Successfully extracted attestation data: version=$deviceAttestVersion, osVersion=$osVersion, osPatch=$osPatchLevel, vendorPatch=$vendorPatchLevel, bootPatch=$bootPatchLevel, moduleHash=${moduleHash?.toHex()}, bootKey=${verifiedBootKey?.toHex()}, bootHash=${verifiedBootHash?.toHex()}"
+                "Successfully extracted attestation data: version=$attestVersion, osVersion=$osVersion, osPatch=$osPatchLevel, vendorPatch=$vendorPatchLevel, bootPatch=$bootPatchLevel, moduleHash=${moduleHash?.toHex()}, bootKey=${verifiedBootKey?.toHex()}, bootHash=${verifiedBootHash?.toHex()}"
            )
            return AttestationData(
                moduleHash,
@@ -58,7 +58,6 @@ data class KeyMintAttestation(
    val maxUsesPerBoot: Int?,
    val maxBootLevel: Int?,
    val minMacLength: Int?,
-    val macLength: Int? = null,
    val rsaOaepMgfDigest: List<Int>,
 ) {
    /** Secondary constructor that populates the fields by parsing an array of `KeyParameter`. */
@@ -135,7 +134,6 @@ data class KeyMintAttestation(
        maxUsesPerBoot = params.findInteger(Tag.MAX_USES_PER_BOOT),
        maxBootLevel = params.findInteger(Tag.MAX_BOOT_LEVEL),
        minMacLength = params.findInteger(Tag.MIN_MAC_LENGTH),
-        macLength = params.findInteger(Tag.MAC_LENGTH),
        rsaOaepMgfDigest = params.findAllDigests(Tag.RSA_OAEP_MGF_DIGEST),
    ) {
        // Log all parsed parameters for debugging purposes.
@@ -144,8 +142,7 @@ data class KeyMintAttestation(

    fun isAttestKey(): Boolean = purpose.size == 1 && purpose.contains(KeyPurpose.ATTEST_KEY)

-    fun isImportKey(): Boolean =
-        origin == KeyOrigin.IMPORTED || origin == KeyOrigin.SECURELY_IMPORTED
+    fun isImportKey(): Boolean = origin == KeyOrigin.IMPORTED || origin == KeyOrigin.SECURELY_IMPORTED
 }

 // --- Private helper extension functions for parsing KeyParameter arrays ---
@@ -1,20 +1,10 @@
 package org.matrix.TEESimulator.config

 import android.os.SystemProperties
-import java.io.File
 import org.matrix.TEESimulator.logging.SystemLogger
 import org.matrix.TEESimulator.util.AndroidDeviceUtils

 object BootStateManager {
-    private const val CONFIG_PATH = "/data/adb/tricky_store"
-    private const val BOOT_PROPS_MODE_FILE = "boot_props_mode"
-
-    private enum class BootPropsMode {
-        AUTO,
-        FORCE,
-        DISABLE,
-    }
-
    private val targets =
        linkedMapOf(
            "ro.boot.verifiedbootstate" to "green",
@@ -32,25 +22,6 @@ object BootStateManager {
        )

    fun apply() {
-        val mode = readBootPropsMode()
-        when (mode) {
-            BootPropsMode.DISABLE -> {
-                SystemLogger.info("BootStateManager: disabled by $BOOT_PROPS_MODE_FILE")
-                return
-            }
-            BootPropsMode.AUTO -> {
-                if (isOplusFamilyDevice()) {
-                    SystemLogger.warning(
-                        "BootStateManager: skipping boot-state prop spoofing on Oplus-family device in auto mode"
-                    )
-                    return
-                }
-            }
-            BootPropsMode.FORCE -> {
-                SystemLogger.info("BootStateManager: force-enabled by $BOOT_PROPS_MODE_FILE")
-            }
-        }
-
        for ((name, target) in targets) {
            val current = SystemProperties.get(name, "")
            if (current.isEmpty()) {
@@ -74,50 +45,4 @@ object BootStateManager {
            AndroidDeviceUtils.setProperty(name, value)
        }
    }
-
-    fun shouldSpoofBootProps(): Boolean =
-        when (readBootPropsMode()) {
-            BootPropsMode.DISABLE -> false
-            BootPropsMode.AUTO -> !isOplusFamilyDevice()
-            BootPropsMode.FORCE -> true
-        }
-
-    private fun readBootPropsMode(): BootPropsMode {
-        val file = File(CONFIG_PATH, BOOT_PROPS_MODE_FILE)
-        if (!file.exists()) return BootPropsMode.AUTO
-
-        val raw =
-            runCatching { file.readText().trim().lowercase() }
-                .getOrElse {
-                    SystemLogger.warning("BootStateManager: failed to read ${file.absolutePath}", it)
-                    return BootPropsMode.AUTO
-                }
-
-        return when (raw) {
-            "1", "true", "on", "enable", "enabled", "force" -> BootPropsMode.FORCE
-            "0", "false", "off", "disable", "disabled", "none" -> BootPropsMode.DISABLE
-            else -> BootPropsMode.AUTO
-        }
-    }
-
-    private fun isOplusFamilyDevice(): Boolean {
-        val props =
-            listOf(
-                "ro.product.manufacturer",
-                "ro.product.brand",
-                "ro.product.vendor.manufacturer",
-                "ro.product.vendor.brand",
-                "ro.product.odm.manufacturer",
-                "ro.product.odm.brand",
-                "ro.boot.hardware.sku",
-                "ro.boot.project_name",
-            )
-
-        val joined =
-            props.joinToString(separator = " ") { name ->
-                SystemProperties.get(name, "")
-            }.lowercase()
-
-        return listOf("oneplus", "oplus", "oppo", "realme").any { joined.contains(it) }
-    }
 }
@@ -0,0 +1,193 @@
+package org.matrix.TEESimulator.config
+
+import android.os.Handler
+import android.os.HandlerThread
+import java.io.File
+import java.net.URL
+import java.nio.file.Files
+import java.nio.file.StandardCopyOption
+import javax.net.ssl.HttpsURLConnection
+import org.json.JSONArray
+import org.json.JSONObject
+import org.matrix.TEESimulator.BuildConfig
+import org.matrix.TEESimulator.logging.SystemLogger
+
+object BulletinPoller {
+    private const val BULLETIN_URL = "https://source.android.com/docs/security/bulletin/pixel"
+    private const val PATCH_FILE = "/data/adb/tricky_store/security_patch.txt"
+    private const val HISTORY_FILE = "/data/adb/tricky_store/last_bulletin_fetch.json"
+    private const val HISTORY_STAGING = "/data/adb/tricky_store/last_bulletin_fetch.json.next"
+    private const val HISTORY_CAP = 10
+    private const val CONNECT_TIMEOUT_MS = 10_000
+    private const val READ_TIMEOUT_MS = 15_000
+    private const val STEADY_INTERVAL_MS = 24L * 60 * 60 * 1000
+
+    private val BOOTSTRAP_INTERVALS = longArrayOf(5_000, 30_000, 120_000, 600_000, 1_800_000)
+    private val DATE_REGEX = Regex("<td>(\\d{4}-\\d{2}-\\d{2})</td>")
+    private val PATCH_DATE_PATTERN = Regex("^\\d{4}-\\d{2}-\\d{2}$")
+
+    private lateinit var handler: Handler
+    @Volatile private var bootstrapStep = 0
+    @Volatile private var steadyArmed = false
+
+    fun start() {
+        val thread = HandlerThread("BulletinPoller").apply { start() }
+        handler = Handler(thread.looper)
+        handler.postDelayed(::pollOnce, BOOTSTRAP_INTERVALS[0])
+    }
+
+    private fun pollOnce() {
+        try {
+            val result = fetchAndParse()
+            appendHistory(result)
+            scheduleNext(result.status == "success")
+        } catch (t: Throwable) {
+            SystemLogger.error("BulletinPoller: pollOnce failed", t)
+            scheduleNext(false)
+        }
+    }
+
+    private fun scheduleNext(success: Boolean) {
+        if (success || steadyArmed) {
+            steadyArmed = true
+            handler.postDelayed(::pollOnce, STEADY_INTERVAL_MS)
+            return
+        }
+        bootstrapStep++
+        if (bootstrapStep >= BOOTSTRAP_INTERVALS.size) {
+            steadyArmed = true
+            handler.postDelayed(::pollOnce, STEADY_INTERVAL_MS)
+        } else {
+            handler.postDelayed(::pollOnce, BOOTSTRAP_INTERVALS[bootstrapStep])
+        }
+    }
+
+    private data class FetchResult(
+        val ts: Long,
+        val status: String,
+        val httpCode: Int?,
+        val parsedDate: String?,
+        val applied: Boolean,
+        val error: String?,
+    )
+
+    private fun fetchAndParse(): FetchResult {
+        val ts = System.currentTimeMillis()
+        var conn: HttpsURLConnection? = null
+        return try {
+            conn =
+                (URL(BULLETIN_URL).openConnection() as HttpsURLConnection).apply {
+                    connectTimeout = CONNECT_TIMEOUT_MS
+                    readTimeout = READ_TIMEOUT_MS
+                    setRequestProperty(
+                        "User-Agent",
+                        "TEESimulator/${BuildConfig.VERSION_NAME}",
+                    )
+                    requestMethod = "GET"
+                }
+            val code = conn.responseCode
+            if (code != 200) {
+                return FetchResult(ts, "network_error", code, null, false, "HTTP $code")
+            }
+            val html = conn.inputStream.bufferedReader().use { it.readText() }
+            val date = DATE_REGEX.find(html)?.groupValues?.get(1)
+            if (date == null) {
+                return FetchResult(
+                    ts,
+                    "parse_error",
+                    code,
+                    null,
+                    false,
+                    "no <td>YYYY-MM-DD</td> match",
+                )
+            }
+            val current = currentPatch()
+            if (current == null || date <= current) {
+                return FetchResult(ts, "success", code, date, false, null)
+            }
+            if (PatchLevelManager.updateTo(date)) {
+                FetchResult(ts, "success", code, date, true, null)
+            } else {
+                FetchResult(
+                    ts,
+                    "validation_rejected",
+                    code,
+                    date,
+                    false,
+                    "PatchLevelManager.updateTo rejected $date",
+                )
+            }
+        } catch (e: Exception) {
+            FetchResult(ts, "network_error", null, null, false, e.toString())
+        } finally {
+            conn?.disconnect()
+        }
+    }
+
+    private fun currentPatch(): String? {
+        val f = File(PATCH_FILE)
+        if (!f.exists()) return null
+        val raw = try {
+            f.readLines()
+                .firstOrNull { it.startsWith("system=") }
+                ?.substringAfter("system=")
+                ?.trim()
+                ?.takeIf { it != "prop" && it.isNotEmpty() }
+        } catch (_: Exception) {
+            null
+        }
+        if (raw == null) return null
+        if (PATCH_DATE_PATTERN.matches(raw)) return raw
+        SystemLogger.warning(
+            "BulletinPoller: ignoring malformed system='$raw' in $PATCH_FILE"
+        )
+        return null
+    }
+
+    private fun appendHistory(result: FetchResult) {
+        try {
+            val target = File(HISTORY_FILE)
+            val staging = File(HISTORY_STAGING)
+            val existing = if (target.exists()) runCatching { target.readText() }.getOrNull() else null
+            val history =
+                existing
+                    ?.let { runCatching { JSONObject(it).optJSONArray("history") }.getOrNull() }
+                    ?: JSONArray()
+            val entry =
+                JSONObject().apply {
+                    put("ts", result.ts)
+                    put("status", result.status)
+                    put("http_code", result.httpCode ?: JSONObject.NULL)
+                    put("parsed_date", result.parsedDate ?: JSONObject.NULL)
+                    put("applied", result.applied)
+                    put("error", result.error ?: JSONObject.NULL)
+                }
+            history.put(entry)
+            while (history.length() > HISTORY_CAP) history.remove(0)
+
+            val latestKnown =
+                (0 until history.length())
+                    .mapNotNull {
+                        history.optJSONObject(it)?.optString("parsed_date", "")?.takeIf { d ->
+                            d.isNotBlank()
+                        }
+                    }
+                    .lastOrNull()
+
+            val root =
+                JSONObject().apply {
+                    put("latest_known_date", latestKnown ?: JSONObject.NULL)
+                    put("history", history)
+                }
+            staging.writeText(root.toString(2))
+            Files.move(
+                staging.toPath(),
+                target.toPath(),
+                StandardCopyOption.ATOMIC_MOVE,
+                StandardCopyOption.REPLACE_EXISTING,
+            )
+        } catch (e: Exception) {
+            SystemLogger.error("BulletinPoller: failed to persist history", e)
+        }
+    }
+}
@@ -96,8 +96,7 @@ object ConfigurationManager {
    fun isAutoMode(uid: Int): Boolean {
        for (pkg in getPackagesForUid(uid)) {
            when (packageModes[pkg]) {
-                Mode.GENERATE,
-                Mode.PATCH -> return false
+                Mode.GENERATE, Mode.PATCH -> return false
                Mode.AUTO -> return true
                null -> continue
            }
@@ -113,9 +112,7 @@ object ConfigurationManager {
            when (packageModes[pkg]) {
                Mode.GENERATE -> return Mode.GENERATE
                Mode.PATCH -> return Mode.PATCH
-                Mode.AUTO ->
-                    return if (DeviceAttestationService.isTeeFunctional) Mode.PATCH
-                    else Mode.GENERATE
+                Mode.AUTO -> return if (DeviceAttestationService.isTeeFunctional) Mode.PATCH else Mode.GENERATE
                null -> continue
            }
        }
@@ -263,9 +260,7 @@ object ConfigurationManager {
            // resolves to the real device prop — force boot/vendor through the same path
            // to prevent cross-component date mismatches on non-Pixel devices.
            if (newGlobalLevel?.system.equals("prop", ignoreCase = true)) {
-                SystemLogger.info(
-                    "system=prop: forcing boot/vendor to derive from device props (were: boot=${newGlobalLevel?.boot}, vendor=${newGlobalLevel?.vendor})"
-                )
+                SystemLogger.info("system=prop: forcing boot/vendor to derive from device props (were: boot=${newGlobalLevel?.boot}, vendor=${newGlobalLevel?.vendor})")
                newGlobalLevel = newGlobalLevel?.copy(boot = "prop", vendor = "prop")
            }
            contextLines.remove("") // Remove global context to iterate over packages next
@@ -298,11 +293,9 @@ object ConfigurationManager {

            val file = if (event != DELETE) File(configRoot, path) else null
            when (path) {
-                TARGET_PACKAGES_FILE ->
-                    file?.let { loadTargetPackages(it) }
+                TARGET_PACKAGES_FILE -> file?.let { loadTargetPackages(it) }
                    ?: SystemLogger.warning("$TARGET_PACKAGES_FILE was deleted.")
-                PATCH_LEVEL_FILE ->
-                    file?.let { loadPatchLevelConfig(it) }
+                PATCH_LEVEL_FILE -> file?.let { loadPatchLevelConfig(it) }
                    ?: SystemLogger.warning("$PATCH_LEVEL_FILE was deleted.")
                // Any change to an XML file is assumed to be a keybox.
                // The cache in KeyBoxManager will handle reloading it on its next use.
@@ -0,0 +1,192 @@
+package org.matrix.TEESimulator.config
+
+import android.os.Build
+import android.os.FileObserver
+import android.os.SystemProperties
+import java.io.File
+import java.nio.file.Files
+import java.nio.file.StandardCopyOption
+import java.time.LocalDate
+import org.json.JSONObject
+import org.matrix.TEESimulator.logging.SystemLogger
+import org.matrix.TEESimulator.util.AndroidDeviceUtils
+
+object PatchLevelManager {
+    private const val PATCH_FILE = "/data/adb/tricky_store/security_patch.txt"
+    private const val STAGING_FILE = "/data/adb/tricky_store/security_patch.txt.next"
+    private const val PIF_DIR = "/data/adb/modules/playintegrityfix"
+    private const val FLOOR_YYYYMMDD = 20200101
+    private const val MAX_PAST_OFFSET = 10000
+
+    /**
+     * Pixel security bulletins publish monthly; pre-announced dates occasionally
+     * slip by 2-4 weeks. 60 days covers that window without admitting a
+     * far-future date from a hostile or mis-parsed bulletin response.
+     */
+    private const val MAX_FUTURE_DAYS = 60L
+
+    private val PIF_FILENAMES =
+        setOf("pif.json", "pif.prop", "custom.pif.json", "custom.pif.prop")
+
+    private val DATE_PATTERN = Regex("^\\d{4}-\\d{2}-\\d{2}$")
+    private val PROP_PATTERN = Regex("^SECURITY_PATCH=(.+)$", RegexOption.MULTILINE)
+    private val SECTION_HEADER = Regex("^\\[[a-zA-Z0-9_.-]+]$")
+    private val GLOBAL_KEYS = setOf("system", "boot", "vendor", "all")
+
+    private val PIF_SOURCES =
+        listOf(
+            "/data/adb/modules/playintegrityfix/pif.json",
+            "/data/adb/pif.json",
+            "/data/adb/modules/playintegrityfix/pif.prop",
+            "/data/adb/pif.prop",
+            "/data/adb/modules/playintegrityfix/custom.pif.json",
+            "/data/adb/modules/playintegrityfix/custom.pif.prop",
+        )
+
+    fun initialize() {
+        refreshFromSources()
+        startPifObserver()
+    }
+
+    private fun refreshFromSources() {
+        val date =
+            resolvePifPatch()
+                ?: SystemProperties.get(
+                    "ro.build.version.security_patch",
+                    Build.VERSION.SECURITY_PATCH,
+                )
+        SystemLogger.info("PatchLevelManager: resolved patch date = $date")
+        applyToProps(date)
+    }
+
+    private fun startPifObserver() {
+        if (!File(PIF_DIR).exists()) {
+            SystemLogger.debug("PatchLevelManager: PIF dir absent, hot-reload disabled")
+            return
+        }
+        PifObserver.startWatching()
+    }
+
+    @Synchronized
+    private fun applyToProps(date: String) {
+        if (!DATE_PATTERN.matches(date)) {
+            SystemLogger.warning(
+                "PatchLevelManager: skip resetprop for invalid date: $date"
+            )
+            return
+        }
+        AndroidDeviceUtils.setProperty("ro.build.version.security_patch", date)
+        AndroidDeviceUtils.setProperty("ro.vendor.build.security_patch", date)
+    }
+
+    fun updateTo(date: String): Boolean {
+        if (!DATE_PATTERN.matches(date)) {
+            SystemLogger.warning("PatchLevelManager: invalid date format: $date")
+            return false
+        }
+        val dateInt = date.replace("-", "").toInt()
+        if (dateInt < FLOOR_YYYYMMDD) {
+            SystemLogger.warning("PatchLevelManager: $date below floor $FLOOR_YYYYMMDD")
+            return false
+        }
+        val now = LocalDate.now()
+        val today = now.year * 10000 + now.monthValue * 100 + now.dayOfMonth
+        if (today >= dateInt + MAX_PAST_OFFSET) {
+            SystemLogger.warning(
+                "PatchLevelManager: $date more than 1y older than today ($today)"
+            )
+            return false
+        }
+        val maxFuture =
+            now.plusDays(MAX_FUTURE_DAYS).let {
+                it.year * 10000 + it.monthValue * 100 + it.dayOfMonth
+            }
+        if (dateInt > maxFuture) {
+            SystemLogger.warning(
+                "PatchLevelManager: $date more than $MAX_FUTURE_DAYS days in future ($maxFuture)"
+            )
+            return false
+        }
+        try {
+            atomicWrite(date)
+        } catch (e: Exception) {
+            SystemLogger.error("PatchLevelManager: atomicWrite failed for $date", e)
+            return false
+        }
+        applyToProps(date)
+        SystemLogger.info("PatchLevelManager: applied patch date $date")
+        return true
+    }
+
+    private fun resolvePifPatch(): String? {
+        val source =
+            PIF_SOURCES.map(::File).lastOrNull { it.exists() && it.length() > 0 }
+                ?: return null
+        return try {
+            val text = source.readText()
+            val parsed =
+                if (source.name.endsWith(".json")) {
+                    JSONObject(text).optString("SECURITY_PATCH", "")
+                } else {
+                    PROP_PATTERN.find(text)?.groupValues?.get(1)?.trim().orEmpty()
+                }
+            parsed.takeIf { it.isNotBlank() }
+        } catch (e: Exception) {
+            SystemLogger.warning(
+                "PatchLevelManager: failed to parse ${source.path}: ${e.message}"
+            )
+            null
+        }
+    }
+
+    private fun atomicWrite(date: String) {
+        val target = File(PATCH_FILE)
+        val staging = File(STAGING_FILE)
+        staging.writeText(mergedContents(target, date))
+        Files.move(
+            staging.toPath(),
+            target.toPath(),
+            StandardCopyOption.ATOMIC_MOVE,
+            StandardCopyOption.REPLACE_EXISTING,
+        )
+    }
+
+    private fun mergedContents(target: File, date: String): String {
+        val globalBlock = "system=$date\nboot=$date\nvendor=$date\n"
+        if (!target.exists()) return globalBlock
+        val tail = stripGlobalAssignments(target.readLines())
+        if (tail.isEmpty()) return globalBlock
+        return globalBlock + tail.joinToString("\n", prefix = "\n", postfix = "\n")
+    }
+
+    private fun stripGlobalAssignments(lines: List<String>): List<String> {
+        val kept = mutableListOf<String>()
+        var inGlobal = true
+        for (line in lines) {
+            val trimmed = line.trim()
+            if (SECTION_HEADER.matches(trimmed)) {
+                inGlobal = false
+                kept += line
+                continue
+            }
+            if (inGlobal && isGlobalKeyAssignment(trimmed)) continue
+            kept += line
+        }
+        return kept
+    }
+
+    private fun isGlobalKeyAssignment(trimmed: String): Boolean {
+        if (trimmed.isEmpty() || trimmed.startsWith("#") || '=' !in trimmed) return false
+        val key = trimmed.substringBefore('=').trim().lowercase()
+        return key in GLOBAL_KEYS
+    }
+
+    private object PifObserver :
+        FileObserver(File(PIF_DIR), CLOSE_WRITE or MOVED_TO or DELETE) {
+        override fun onEvent(event: Int, path: String?) {
+            if (path == null || path !in PIF_FILENAMES) return
+            SystemLogger.info("PatchLevelManager: PIF change ($path), refreshing")
+            refreshFromSources()
+        }
+    }
+}
@@ -110,18 +110,14 @@ abstract class BinderInterceptor : Binder() {
     */
    final override fun onTransact(code: Int, data: Parcel, reply: Parcel?, flags: Int): Boolean {
        val txId = data.readLong()
-        val result =
-            try {
+        val result = try {
            when (code) {
                PRE_TRANSACT_CODE -> handlePreTransact(txId, data)
                POST_TRANSACT_CODE -> handlePostTransact(txId, data)
                else -> return super.onTransact(code, data, reply, flags)
            }
        } catch (e: Throwable) {
-                SystemLogger.error(
-                    "[TX_ID: $txId] Interceptor exception, falling through to HAL",
-                    e,
-                )
+            SystemLogger.error("[TX_ID: $txId] Interceptor exception, falling through to HAL", e)
            TransactionResult.ContinueAndSkipPost
        }
        writeResultToReply(result, reply!!)
@@ -224,11 +220,7 @@ abstract class BinderInterceptor : Binder() {
        }
    }

-    /**
-     * Logs an intercepted transaction. For a targeted UID every transaction — whether we intercept
-     * or merely observe it — is recorded on that UID's own diagnostic plane, so its keystore
-     * timeline reads cleanly end to end. Untargeted UIDs get a single terse, rate-limited line.
-     */
+    /** Helper function for consistent logging of intercepted transactions. */
    protected fun logTransaction(
        txId: Long,
        methodName: String,
@@ -236,14 +228,15 @@ abstract class BinderInterceptor : Binder() {
        callingPid: Int,
        skipPost: Boolean = false,
    ) {
-        if (SystemLogger.isUidLogged(callingUid)) {
-            val action = if (skipPost) "observe" else "intercept"
-            SystemLogger.uidLog(callingUid, txId, "tx", "$methodName action=$action pid=$callingPid")
-            return
-        }
-        SystemLogger.verbose {
+        val isIntercepting = !skipPost && !ConfigurationManager.shouldSkipUid(callingUid)
+        val action = if (isIntercepting) "Intercept" else "Observe"
        val packages = ConfigurationManager.getPackagesForUid(callingUid).joinToString()
-            "[TX_ID: $txId] Observe $methodName for packages=[$packages] (uid=$callingUid, pid=$callingPid)"
+        val message =
+            "[TX_ID: $txId] $action $methodName for packages=[$packages] (uid=$callingUid, pid=$callingPid)"
+        if (isIntercepting) {
+            SystemLogger.debug(message)
+        } else {
+            SystemLogger.verbose(message)
        }
    }

@@ -305,26 +298,18 @@ abstract class BinderInterceptor : Binder() {
            target: IBinder,
            interceptor: BinderInterceptor,
            filteredCodes: IntArray = intArrayOf(),
-        ): Boolean {
+        ) {
            val data = Parcel.obtain()
            val reply = Parcel.obtain()
-            return try {
+            try {
                data.writeStrongBinder(target)
                data.writeStrongBinder(interceptor)
                data.writeInt(filteredCodes.size)
                for (code in filteredCodes) data.writeInt(code)
-                val ok = backdoor.transact(REGISTER_INTERCEPTOR_CODE, data, reply, 0)
-                if (ok) {
-                    SystemLogger.info(
-                        "Registered interceptor for target: $target (${filteredCodes.size} filtered codes)"
-                    )
-                } else {
-                    SystemLogger.error("Register transact returned false for target: $target")
-                }
-                ok
+                backdoor.transact(REGISTER_INTERCEPTOR_CODE, data, reply, 0)
+                SystemLogger.info("Registered interceptor for target: $target (${filteredCodes.size} filtered codes)")
            } catch (e: Exception) {
                SystemLogger.error("Failed to register binder interceptor.", e)
-                false
            } finally {
                data.recycle()
                reply.recycle()
@@ -8,8 +8,6 @@ import android.os.Parcelable
 import android.security.KeyStore
 import android.security.keystore.KeystoreResponse
 import android.system.keystore2.Authorization
-import java.nio.ByteBuffer
-import java.nio.ByteOrder
 import org.matrix.TEESimulator.interception.core.BinderInterceptor
 import org.matrix.TEESimulator.logging.SystemLogger
 import org.matrix.TEESimulator.util.AndroidDeviceUtils
@@ -21,11 +19,6 @@ object InterceptorUtils {

    private const val EX_SERVICE_SPECIFIC = -8

-    private const val FLAT_STRIDE_HEADER = 12
-    private const val MAX_AUTH_COUNT = 256
-    private const val SENTINEL_MODTIME = 4_294_967_297L
-    private const val HIGH_MODTIME = 4_999_999_999L
-
    private fun synthesizeSseMessage(errorCode: Int): String =
        when (errorCode) {
            2 -> "Error::Rc(SYSTEM_ERROR)"
@@ -44,8 +37,7 @@ object InterceptorUtils {
        }

    fun createErrorReply(errorCode: Int): BinderInterceptor.TransactionResult.OverrideReply {
-        val parcel =
-            Parcel.obtain().apply {
+        val parcel = Parcel.obtain().apply {
            writeInt(EX_SERVICE_SPECIFIC)
            writeString(synthesizeSseMessage(errorCode))
            writeInt(0) // empty remote stack trace header (AOSP Status.cpp:196)
@@ -120,25 +112,24 @@ object InterceptorUtils {
        return BinderInterceptor.TransactionResult.OverrideReply(parcel)
    }

-    /** Correlates a captured reply parcel to the app that triggered it, for [createTypedObjectReply]. */
-    data class ReplyDiagnostic(val uid: Int, val txId: Long?, val event: String)
-
    /** Creates an `OverrideReply` parcel containing a Parcelable object. */
    fun <T : Parcelable?> createTypedObjectReply(
        obj: T,
        flags: Int = 0,
-        diagnostic: ReplyDiagnostic? = null,
+        diagnosticTag: String? = null,
    ): BinderInterceptor.TransactionResult.OverrideReply {
        val parcel =
            Parcel.obtain().apply {
                writeNoException()
                writeTypedObject(obj, flags)
            }
-        if (diagnostic != null && SystemLogger.isUidLogged(diagnostic.uid)) {
+        if (diagnosticTag != null && SystemLogger.isDebugBuild) {
            val savedPos = parcel.dataPosition()
            val wire = parcel.marshall()
            parcel.setDataPosition(savedPos)
-            SystemLogger.uidLogRaw(diagnostic.uid, diagnostic.txId, diagnostic.event, "len=${wire.size}", wire)
+            val path = "/data/local/tmp/teesim-$diagnosticTag-${System.nanoTime()}.bin"
+            runCatching { java.io.File(path).writeBytes(wire) }
+            SystemLogger.debug("[$diagnosticTag] reply len=${wire.size} path=$path")
        }
        return BinderInterceptor.TransactionResult.OverrideReply(parcel)
    }
@@ -198,8 +189,8 @@ object InterceptorUtils {
        val vendorPatch = AndroidDeviceUtils.getVendorPatchLevelLong(callingUid)
        val bootPatch = AndroidDeviceUtils.getBootPatchLevelLong(callingUid)

-        val patched =
-            authorizations.map { auth ->
+        return authorizations
+            .map { auth ->
                val replacement =
                    when (auth.keyParameter.tag) {
                        Tag.OS_PATCHLEVEL ->
@@ -225,101 +216,5 @@ object InterceptorUtils {
                }
            }
            .toTypedArray()
-        return normalizeAuthorizationLayout(patched)
    }
-
-    /**
-     * Reorders a generateKey reply's authorizations only when the marshalled reply would read, to a
-     * flat 12-byte-stride parcel fingerprint, as the TEE-simulator sentinel: a last slot of
-     * securityLevel 4 or 256, tag 1, union 32 with the 0x1_0000_0001 pseudo-timestamp, or any
-     * timestamp past [HIGH_MODTIME]. Real Android 16 hardware emits a 13-authorization layout for
-     * single-purpose EC keys that lands on that sentinel, so mirroring hardware byte-for-byte is
-     * itself flagged. Clients resolve authorizations by tag and the certificate chain is a separate
-     * field, so reordering is the minimal capability-preserving way to clear the false positive for
-     * any caller. Already-clean replies are returned unchanged.
-     */
-    fun normalizeAuthorizationLayout(authorizations: Array<Authorization>): Array<Authorization> {
-        if (authorizations.size < 2) return authorizations
-        if (!flatStrideFingerprintMatches(marshalTypedArray(authorizations))) return authorizations
-        val n = authorizations.size
-        for (src in 1 until n) {
-            val candidate = moveAuthorization(authorizations, src, 0)
-            if (!flatStrideFingerprintMatches(marshalTypedArray(candidate))) return candidate
-        }
-        for (src in 0 until n) {
-            for (dst in 0 until n) {
-                if (src == dst) continue
-                val candidate = moveAuthorization(authorizations, src, dst)
-                if (!flatStrideFingerprintMatches(marshalTypedArray(candidate))) return candidate
-            }
-        }
-        return authorizations
-    }
-
-    private fun moveAuthorization(
-        authorizations: Array<Authorization>,
-        src: Int,
-        dst: Int,
-    ): Array<Authorization> {
-        val reordered = authorizations.toMutableList()
-        reordered.add(dst, reordered.removeAt(src))
-        return reordered.toTypedArray()
-    }
-
-    private fun marshalTypedArray(authorizations: Array<Authorization>): ByteArray {
-        val parcel = Parcel.obtain()
-        return try {
-            // keystore2 AIDL compile stubs omit the Parcelable supertype these types carry at runtime.
-            parcel.writeTypedArray(authorizations.map { it as Parcelable }.toTypedArray(), 0)
-            parcel.marshall()
-        } finally {
-            parcel.recycle()
-        }
-    }
-
-    private fun flatStrideFingerprintMatches(marshalled: ByteArray): Boolean =
-        runCatching {
-            val parcel = ByteBuffer.wrap(marshalled).order(ByteOrder.LITTLE_ENDIAN)
-            val count = parcel.getInt(0)
-            if (count !in 1..MAX_AUTH_COUNT) return@runCatching false
-            var off = 4
-            var lastSec = 0L
-            var lastTag = 0L
-            var lastUnion = 0L
-            repeat(count) {
-                lastSec = u32(parcel, off)
-                lastTag = u32(parcel, off + 4)
-                lastUnion = u32(parcel, off + 8)
-                off += FLAT_STRIDE_HEADER
-                off = alignWord(off + flatPayloadSize(parcel, off, lastUnion))
-            }
-            off = skipDriftedByteArray(parcel, off)
-            off = skipDriftedByteArray(parcel, off)
-            val modtime = parcel.getLong(alignWord(off))
-            val unknownUnion = lastUnion !in 0..14
-            modtime > HIGH_MODTIME ||
-                (modtime == SENTINEL_MODTIME &&
-                    (lastSec == 4L || lastSec == 256L) &&
-                    lastTag == 1L &&
-                    lastUnion == 32L &&
-                    unknownUnion)
-        }.getOrDefault(false)
-
-    private fun flatPayloadSize(parcel: ByteBuffer, off: Int, union: Long): Int =
-        when {
-            union in 1..11 -> 4
-            union == 12L || union == 13L -> 8
-            union == 14L -> alignWord(off + 4 + parcel.getInt(off)) - off
-            else -> 0
-        }
-
-    private fun skipDriftedByteArray(parcel: ByteBuffer, off: Int): Int {
-        if (parcel.getInt(off) == 0) return off + 4
-        val lengthPos = off + 4
-        return alignWord(lengthPos + 4 + parcel.getInt(lengthPos))
-    }
-
-    private fun u32(parcel: ByteBuffer, off: Int): Long = parcel.getInt(off).toLong() and 0xFFFFFFFFL
-
-    private fun alignWord(off: Int): Int = (off + 3) and 3.inv()
 }
@@ -5,7 +5,6 @@ import android.hardware.security.keymint.SecurityLevel
 import android.os.Build
 import android.os.IBinder
 import android.os.Parcel
-import android.os.ServiceManager
 import android.system.keystore2.Domain
 import android.system.keystore2.IKeystoreService
 import android.system.keystore2.KeyDescriptor
@@ -18,7 +17,6 @@ import org.matrix.TEESimulator.attestation.KeyMintAttestation
 import org.matrix.TEESimulator.config.ConfigurationManager
 import org.matrix.TEESimulator.interception.keystore.shim.GeneratedKeyPersistence
 import org.matrix.TEESimulator.interception.keystore.shim.KeyMintSecurityLevelInterceptor
-import org.matrix.TEESimulator.logging.AttestationDossier
 import org.matrix.TEESimulator.logging.KeyMintParameterLogger
 import org.matrix.TEESimulator.logging.SystemLogger
 import org.matrix.TEESimulator.pki.CertificateGenerator
@@ -50,8 +48,6 @@ object Keystore2Interceptor : AbstractKeystoreInterceptor() {
        else null
    private val GET_NUMBER_OF_ENTRIES_TRANSACTION =
        InterceptorUtils.getTransactCode(stubBinderClass, "getNumberOfEntries")
-    private val GRANT_TRANSACTION = InterceptorUtils.getTransactCode(stubBinderClass, "grant")
-    private val UNGRANT_TRANSACTION = InterceptorUtils.getTransactCode(stubBinderClass, "ungrant")

    private val transactionNames: Map<Int, String> by lazy {
        stubBinderClass.declaredFields
@@ -63,14 +59,6 @@ object Keystore2Interceptor : AbstractKeystoreInterceptor() {
    }

    private const val RESPONSE_KEY_NOT_FOUND = 7
-    private const val RESPONSE_PERMISSION_DENIED = 6
-    private const val KEY_PERMISSION_GET_INFO = 0x4
-    private const val KEY_PERMISSION_UPDATE = 0x80
-
-    // KeyStoreManager.grantKeyAccess() became a public app API in Android 16 (API 36). Before that,
-    // grant was a hidden API and SELinux denied untrusted_app, so a synthetic-key grant must answer
-    // PERMISSION_DENIED pre-36 and a coherent virtualized grant on 36+.
-    private const val GRANT_PUBLIC_API_SDK = 36
    private val deletedSoftwareKeys: MutableSet<KeyIdentifier> = ConcurrentHashMap.newKeySet()
    private val userUpdatedKeys = ConcurrentHashMap.newKeySet<KeyIdentifier>()

@@ -92,8 +80,6 @@ object Keystore2Interceptor : AbstractKeystoreInterceptor() {
                LIST_ENTRIES_TRANSACTION,
                LIST_ENTRIES_BATCHED_TRANSACTION,
                GET_NUMBER_OF_ENTRIES_TRANSACTION,
-                GRANT_TRANSACTION,
-                UNGRANT_TRANSACTION,
            )
            .toIntArray()
    }
@@ -105,30 +91,6 @@ object Keystore2Interceptor : AbstractKeystoreInterceptor() {
    override fun onInterceptorReady(service: IBinder, backdoor: IBinder) {
        val keystoreInterface = IKeystoreService.Stub.asInterface(service)
        setupSecurityLevelInterceptors(keystoreInterface, backdoor)
-        setupMaintenanceInterceptor(backdoor)
-    }
-
-    /**
-     * Hooks the keystore2 daemon's `android.security.maintenance` binder, which is hosted by the
-     * same process, so synthetic key state follows real key-lifecycle events. Best-effort: if the
-     * service is absent the synthetic plane simply forgoes lifecycle parity.
-     */
-    private fun setupMaintenanceInterceptor(backdoor: IBinder) {
-        runCatching {
-                ServiceManager.getService("android.security.maintenance")?.let { maintenance ->
-                    SystemLogger.info("Found maintenance binder. Registering interceptor...")
-                    register(
-                        backdoor,
-                        maintenance,
-                        Keystore2MaintenanceInterceptor,
-                        Keystore2MaintenanceInterceptor.interceptedCodes,
-                    )
-                }
-                    ?: SystemLogger.warning(
-                        "Maintenance binder not found; skipping lifecycle parity."
-                    )
-            }
-            .onFailure { SystemLogger.error("Failed to intercept maintenance binder.", it) }
    }

    private fun setupSecurityLevelInterceptors(service: IKeystoreService, backdoor: IBinder) {
@@ -213,68 +175,27 @@ object Keystore2Interceptor : AbstractKeystoreInterceptor() {
        ) {
            logTransaction(txId, transactionNames[code]!!, callingUid, callingPid)

-            if (code == UPDATE_SUBCOMPONENT_TRANSACTION) {
            if (ConfigurationManager.shouldSkipUid(callingUid))
                return TransactionResult.ContinueAndSkipPost
+
+            if (code == UPDATE_SUBCOMPONENT_TRANSACTION)
                return handleUpdateSubcomponent(callingUid, data)
-            }

            data.enforceInterface(IKeystoreService.DESCRIPTOR)
            val descriptor =
                data.readTypedObject(KeyDescriptor.CREATOR)
                    ?: return TransactionResult.ContinueAndSkipPost

-            // Domain.GRANT read (Android 16+ KeyStoreManager grant). Served for ANY grantee uid —
-            // including isolated services (bindIsolatedService) with no package mapping — so
-            // resolve
-            // it before the package-scoped skip; caller-binding in resolveGrant() is the real
-            // access
-            // gate. On Android <= 15 no grants are ever issued (grant() denies), so softwareGrants
-            // is
-            // empty and this falls through to the real keystore2.
-            if (code == GET_KEY_ENTRY_TRANSACTION && descriptor.domain == Domain.GRANT) {
-                val grant =
-                    KeyMintSecurityLevelInterceptor.resolveGrant(descriptor.nspace, callingUid)
-                if (grant == null) {
-                    // Ours but wrong caller -> KEY_NOT_FOUND (caller-binding); not ours -> real
-                    // keystore2.
-                    return if (
-                        KeyMintSecurityLevelInterceptor.softwareGrants.containsKey(
-                            descriptor.nspace
-                        )
-                    )
-                        InterceptorUtils.createErrorReply(RESPONSE_KEY_NOT_FOUND)
-                    else TransactionResult.ContinueAndSkipPost
-                }
-                if ((grant.accessVector and KEY_PERMISSION_GET_INFO) == 0) {
-                    return InterceptorUtils.createErrorReply(RESPONSE_PERMISSION_DENIED)
-                }
-                val response =
-                    KeyMintSecurityLevelInterceptor.getGeneratedKeyResponse(grant.ownerKeyId)
-                        ?: return InterceptorUtils.createErrorReply(RESPONSE_KEY_NOT_FOUND)
-                // Same object the owner read returns -> coherent chain across planes.
-                return InterceptorUtils.createTypedObjectReply(response)
-            }
-
-            // generateKey force-forges attest/device-id keys even for skipped UIDs; getKeyEntry
-            // must serve them back or the framework's attestKeyAlias lookup gets KEY_NOT_FOUND.
-            if (code != GET_KEY_ENTRY_TRANSACTION && ConfigurationManager.shouldSkipUid(callingUid))
-                return TransactionResult.ContinueAndSkipPost
-
            if (code == DELETE_KEY_TRANSACTION) {
                val keyId =
                    if (descriptor.alias != null) {
                        KeyIdentifier(callingUid, descriptor.alias)
                    } else if (descriptor.domain == Domain.KEY_ID) {
                        KeyMintSecurityLevelInterceptor.findGeneratedKeyByKeyId(
-                                callingUid,
-                                descriptor.nspace,
-                            )
-                            ?.let { info ->
+                            callingUid, descriptor.nspace
+                        )?.let { info ->
                            KeyMintSecurityLevelInterceptor.generatedKeys.entries
-                                    .find {
-                                        it.value.nspace == info.nspace && it.key.uid == callingUid
-                                    }
+                                .find { it.value.nspace == info.nspace && it.key.uid == callingUid }
                                ?.key
                        }
                    } else null
@@ -307,34 +228,25 @@ object Keystore2Interceptor : AbstractKeystoreInterceptor() {
                    // "Captured private binder exception during timing skip".
                    // Resolving by KEY_ID and returning the cached response keeps
                    // the call on the happy path, eliminating the warmup signal.
-                    val info =
-                        KeyMintSecurityLevelInterceptor.findGeneratedKeyByKeyId(
-                            callingUid,
-                            descriptor.nspace,
+                    val info = KeyMintSecurityLevelInterceptor.findGeneratedKeyByKeyId(
+                        callingUid, descriptor.nspace
                    )
                    if (info?.response != null) {
                        SystemLogger.info(
                            "[TX_ID: $txId] Found generated response via KEY_ID nspace=${descriptor.nspace}"
                        )
-                        logServedChain(callingUid, txId, "keyid:${descriptor.nspace}", info.response)
                        return InterceptorUtils.createTypedObjectReply(info.response)
                    }
-                    val teeResp =
-                        KeyMintSecurityLevelInterceptor.findTeeResponseByKeyId(
-                            callingUid,
-                            descriptor.nspace,
+                    val teeResp = KeyMintSecurityLevelInterceptor.findTeeResponseByKeyId(
+                        callingUid, descriptor.nspace
                    )
                    if (teeResp != null) {
                        SystemLogger.info(
                            "[TX_ID: $txId] Found TEE response via KEY_ID nspace=${descriptor.nspace}"
                        )
-                        logServedChain(callingUid, txId, "keyid:${descriptor.nspace}", teeResp)
                        return InterceptorUtils.createTypedObjectReply(teeResp)
                    }
                }
-                // Domain.GRANT is handled earlier (before the package-scoped skip); an alias-less
-                // read reaching here is KEY_ID or unknown, so it falls through to the real
-                // keystore2.
                return TransactionResult.ContinueAndSkipPost
            }
            val keyId = KeyIdentifier(callingUid, descriptor.alias)
@@ -342,16 +254,10 @@ object Keystore2Interceptor : AbstractKeystoreInterceptor() {
            val response = KeyMintSecurityLevelInterceptor.getGeneratedKeyResponse(keyId)
            if (response == null) {
                if (deletedSoftwareKeys.remove(keyId)) {
-                    SystemLogger.info(
-                        "[TX_ID: $txId] Returning KEY_NOT_FOUND for deleted key ${descriptor.alias}"
-                    )
+                    SystemLogger.info("[TX_ID: $txId] Returning KEY_NOT_FOUND for deleted key ${descriptor.alias}")
                    return InterceptorUtils.createErrorReply(RESPONSE_KEY_NOT_FOUND)
                }
-                // Owned keys were served above; for a skipped UID a non-owned key must still skip
-                // post-processing so we never patch an un-targeted app's real key.
-                return if (ConfigurationManager.shouldSkipUid(callingUid))
-                    TransactionResult.ContinueAndSkipPost
-                else TransactionResult.Continue
+                return TransactionResult.Continue
            }

            if (KeyMintSecurityLevelInterceptor.isAttestationKey(keyId))
@@ -359,63 +265,9 @@ object Keystore2Interceptor : AbstractKeystoreInterceptor() {

            SystemLogger.info("[TX_ID: $txId] Found generated response for ${descriptor.alias}:")
            response.metadata?.authorizations?.forEach {
-                KeyMintParameterLogger.logParameter(callingUid, txId, it.keyParameter)
+                KeyMintParameterLogger.logParameter(it.keyParameter)
            }
-            logServedChain(callingUid, txId, descriptor.alias, response)
            return InterceptorUtils.createTypedObjectReply(response)
-        } else if (code == GRANT_TRANSACTION) {
-            logTransaction(txId, transactionNames[code] ?: "grant", callingUid, callingPid)
-            data.enforceInterface(IKeystoreService.DESCRIPTOR)
-            val key =
-                data.readTypedObject(KeyDescriptor.CREATOR)
-                    ?: return TransactionResult.ContinueAndSkipPost
-            val granteeUid = data.readInt()
-            val accessVector = data.readInt()
-            // Synthetic (generatedKeys) AND patch-mode (teeResponses) keys are ours; both must
-            // grant
-            // coherently so the Domain.GRANT readback returns the same chain the owner read
-            // returns.
-            // Real hardware keys fall through to the real keystore2, which applies the same SELinux
-            // gate the platform would.
-            val ownerKeyId =
-                resolveOwnerKeyId(key, callingUid)?.takeIf {
-                    KeyMintSecurityLevelInterceptor.ownsKeyResponse(it)
-                } ?: return TransactionResult.ContinueAndSkipPost
-            // Version-gated to mirror the real TEE 1:1. Pre-Android-16, grant was a hidden API and
-            // SELinux denied untrusted_app, so keystore2 returns PERMISSION_DENIED. Android 16
-            // (API 36) exposes KeyStoreManager.grantKeyAccess(), so an app grants its own key:
-            // issue a coherent, caller-bound, access-vector-carrying grant whose Domain.GRANT read
-            // returns the owner's chain.
-            if (Build.VERSION.SDK_INT < GRANT_PUBLIC_API_SDK) {
-                return InterceptorUtils.createErrorReply(RESPONSE_PERMISSION_DENIED)
-            }
-            val grantId =
-                KeyMintSecurityLevelInterceptor.issueGrant(ownerKeyId, granteeUid, accessVector)
-            val reply =
-                KeyDescriptor().apply {
-                    domain = Domain.GRANT
-                    nspace = grantId
-                    alias = null
-                    blob = null
-                }
-            return InterceptorUtils.createTypedObjectReply(reply)
-        } else if (code == UNGRANT_TRANSACTION) {
-            logTransaction(txId, transactionNames[code] ?: "ungrant", callingUid, callingPid)
-            data.enforceInterface(IKeystoreService.DESCRIPTOR)
-            val key =
-                data.readTypedObject(KeyDescriptor.CREATOR)
-                    ?: return TransactionResult.ContinueAndSkipPost
-            val granteeUid = data.readInt()
-            val ownerKeyId =
-                resolveOwnerKeyId(key, callingUid)?.takeIf {
-                    KeyMintSecurityLevelInterceptor.ownsKeyResponse(it)
-                } ?: return TransactionResult.ContinueAndSkipPost
-            // Same version gate as grant(): denied pre-36, revoke the virtualized grant on 36+.
-            if (Build.VERSION.SDK_INT < GRANT_PUBLIC_API_SDK) {
-                return InterceptorUtils.createErrorReply(RESPONSE_PERMISSION_DENIED)
-            }
-            KeyMintSecurityLevelInterceptor.revokeGrant(ownerKeyId, granteeUid)
-            return InterceptorUtils.createSuccessReply(writeResultCode = false)
        } else {
            logTransaction(
                txId,
@@ -457,8 +309,7 @@ object Keystore2Interceptor : AbstractKeystoreInterceptor() {
                            it.uid == callingUid
                        }
                    val totalCount = hardwareCount + softwareCount
-                    val parcel =
-                        Parcel.obtain().apply {
+                    val parcel = Parcel.obtain().apply {
                        writeNoException()
                        writeInt(totalCount)
                    }
@@ -504,12 +355,8 @@ object Keystore2Interceptor : AbstractKeystoreInterceptor() {
                    val keyId = KeyIdentifier(callingUid, keyDescriptor.alias)

                    if (userUpdatedKeys.remove(keyId)) {
-                        SystemLogger.trace {
-                            "[TRACE-$txId] getKeyEntry $keyId: userUpdated=true, skipping patch"
-                        }
-                        SystemLogger.debug(
-                            "[TX_ID: $txId] Skipping cert patch for user-updated key $keyId."
-                        )
+                        SystemLogger.trace { "[TRACE-$txId] getKeyEntry $keyId: userUpdated=true, skipping patch" }
+                        SystemLogger.debug("[TX_ID: $txId] Skipping cert patch for user-updated key $keyId.")
                        return TransactionResult.SkipTransaction
                    }

@@ -519,29 +366,18 @@ object Keystore2Interceptor : AbstractKeystoreInterceptor() {
                            authorizations?.map { it.keyParameter }?.toTypedArray() ?: emptyArray()
                        )

-                    SystemLogger.trace {
-                        "[TRACE-$txId] getKeyEntry $keyId: isImport=${parsedParameters.isImportKey()} origin=${parsedParameters.origin} inImportedKeys=${KeyMintSecurityLevelInterceptor.importedKeys.contains(keyId)} hasPatchedChain=${KeyMintSecurityLevelInterceptor.getPatchedChain(keyId) != null} isAttestKey=${parsedParameters.isAttestKey()}"
-                    }
+                    SystemLogger.trace { "[TRACE-$txId] getKeyEntry $keyId: isImport=${parsedParameters.isImportKey()} origin=${parsedParameters.origin} inImportedKeys=${KeyMintSecurityLevelInterceptor.importedKeys.contains(keyId)} hasPatchedChain=${KeyMintSecurityLevelInterceptor.getPatchedChain(keyId) != null} isAttestKey=${parsedParameters.isAttestKey()}" }

                    if (parsedParameters.isImportKey()) {
                        val retainedChain = KeyMintSecurityLevelInterceptor.getPatchedChain(keyId)
                        if (retainedChain == null) {
-                            SystemLogger.trace {
-                                "[TRACE-$txId] getKeyEntry $keyId: imported, no retained chain, skip"
-                            }
-                            SystemLogger.info(
-                                "[TX_ID: $txId] Skip patching for imported key (no prior attestation)."
-                            )
+                            SystemLogger.trace { "[TRACE-$txId] getKeyEntry $keyId: imported, no retained chain, skip" }
+                            SystemLogger.info("[TX_ID: $txId] Skip patching for imported key (no prior attestation).")
                            return TransactionResult.SkipTransaction
                        }
-                        SystemLogger.trace {
-                            "[TRACE-$txId] getKeyEntry $keyId: imported, SERVING RETAINED CHAIN (detection vector!)"
-                        }
-                        SystemLogger.info(
-                            "[TX_ID: $txId] Imported key overwrote attested alias, serving retained chain for $keyId"
-                        )
-                        CertificateHelper.updateCertificateChain(response.metadata, retainedChain)
-                            .getOrThrow()
+                        SystemLogger.trace { "[TRACE-$txId] getKeyEntry $keyId: imported, SERVING RETAINED CHAIN (detection vector!)" }
+                        SystemLogger.info("[TX_ID: $txId] Imported key overwrote attested alias, serving retained chain for $keyId")
+                        CertificateHelper.updateCertificateChain(response.metadata, retainedChain).getOrThrow()
                        response.metadata.authorizations =
                            InterceptorUtils.patchAuthorizations(
                                response.metadata.authorizations,
@@ -551,12 +387,8 @@ object Keystore2Interceptor : AbstractKeystoreInterceptor() {
                    }

                    if (KeyMintSecurityLevelInterceptor.importedKeys.contains(keyId)) {
-                        SystemLogger.trace {
-                            "[TRACE-$txId] getKeyEntry $keyId: in importedKeys set, skip"
-                        }
-                        SystemLogger.debug(
-                            "[TX_ID: $txId] Skipping attest-key override for imported key $keyId"
-                        )
+                        SystemLogger.trace { "[TRACE-$txId] getKeyEntry $keyId: in importedKeys set, skip" }
+                        SystemLogger.debug("[TX_ID: $txId] Skipping attest-key override for imported key $keyId")
                        return TransactionResult.SkipTransaction
                    }

@@ -599,8 +431,7 @@ object Keystore2Interceptor : AbstractKeystoreInterceptor() {
                        // Snapshot metadata bytes for the same reason as the
                        // primary doSoftwareKeyGen path — loss-less restore
                        // after reboot.
-                        val metadataBytesForPersist =
-                            response.metadata?.let { md ->
+                        val metadataBytesForPersist = response.metadata?.let { md ->
                            runCatching {
                                val parcel = android.os.Parcel.obtain()
                                try {
@@ -609,8 +440,7 @@ object Keystore2Interceptor : AbstractKeystoreInterceptor() {
                                } finally {
                                    parcel.recycle()
                                }
-                                    }
-                                    .getOrNull()
+                            }.getOrNull()
                        }
                        GeneratedKeyPersistence.save(
                            keyId = keyId,
@@ -665,10 +495,6 @@ object Keystore2Interceptor : AbstractKeystoreInterceptor() {
                            callingUid,
                        )

-                    // PATCH decode point: the patched chain actually served back to the app on
-                    // getKeyEntry — the ground truth a patch-mode detector reads.
-                    AttestationDossier.log(callingUid, txId, "PATCH", finalChain.asList())
-
                    return InterceptorUtils.createTypedObjectReply(response)
                }
                .onFailure {
@@ -682,91 +508,16 @@ object Keystore2Interceptor : AbstractKeystoreInterceptor() {
        return TransactionResult.SkipTransaction
    }

-    /**
-     * Resolves the owner [KeyIdentifier] a grant/ungrant call targets. APP/alias keys map directly;
-     * KEY_ID keys are looked up by nspace (mirrors the deleteKey resolver). Returns null for
-     * anything not addressable, so callers fall through to the real keystore2.
-     */
-    private fun resolveOwnerKeyId(descriptor: KeyDescriptor, callingUid: Int): KeyIdentifier? =
-        when {
-            descriptor.alias != null -> KeyIdentifier(callingUid, descriptor.alias)
-            descriptor.domain == Domain.KEY_ID ->
-                KeyMintSecurityLevelInterceptor.findGeneratedKeyByKeyId(
-                        callingUid,
-                        descriptor.nspace,
-                    )
-                    ?.let { info ->
-                        KeyMintSecurityLevelInterceptor.generatedKeys.entries
-                            .firstOrNull {
-                                it.value.nspace == info.nspace && it.key.uid == callingUid
-                            }
-                            ?.key
-                    }
-            else -> null
-        }
-
-    /**
-     * Records the certificate chain actually served back to [uid] on a getKeyEntry, keyed by
-     * [alias]. The app reassembles its final chain from these served chains (the leaf alias plus
-     * the attest-key alias), so logging each one with key sizes and a per-edge verification makes a
-     * verification failure in the app's combined chain reproducible from the log, not inferred.
-     */
-    private fun logServedChain(uid: Int, txId: Long, alias: String, response: KeyEntryResponse?) {
-        if (response == null || !SystemLogger.isUidLogged(uid)) return
-        val chain = CertificateHelper.getCertificateChain(response)?.asList() ?: return
-        SystemLogger.uidLog(uid, txId, "served", "alias=$alias depth=${chain.size}")
-        SystemLogger.uidLog(uid, txId, "served-keys", AttestationPatcher.formatChainKeys(chain))
-        SystemLogger.uidLog(uid, txId, "served-verify", AttestationPatcher.formatChainVerification(chain))
-    }
-
    private fun handleUpdateSubcomponent(callingUid: Int, data: Parcel): TransactionResult {
        data.enforceInterface(IKeystoreService.DESCRIPTOR)
-        val descriptor =
-            data.readTypedObject(KeyDescriptor.CREATOR)
+        val descriptor = data.readTypedObject(KeyDescriptor.CREATOR)
            ?: return TransactionResult.ContinueAndSkipPost

-        if (descriptor.domain == Domain.GRANT) {
-            val grant =
-                KeyMintSecurityLevelInterceptor.resolveGrant(descriptor.nspace, callingUid)
-            if (grant == null) {
-                return if (
-                    KeyMintSecurityLevelInterceptor.softwareGrants.containsKey(descriptor.nspace)
-                )
-                    InterceptorUtils.createErrorReply(RESPONSE_KEY_NOT_FOUND)
-                else TransactionResult.ContinueAndSkipPost
-            }
-            if ((grant.accessVector and KEY_PERMISSION_UPDATE) == 0) {
-                return InterceptorUtils.createErrorReply(RESPONSE_PERMISSION_DENIED)
-            }
-
-            val generatedKeyInfo =
-                KeyMintSecurityLevelInterceptor.generatedKeys[grant.ownerKeyId]
-            val response =
-                generatedKeyInfo?.response
-                    ?: KeyMintSecurityLevelInterceptor.getGeneratedKeyResponse(grant.ownerKeyId)
-                    ?: return InterceptorUtils.createErrorReply(RESPONSE_KEY_NOT_FOUND)
-            return updateResponseSubcomponent(
-                response = response,
-                publicCert = data.createByteArray(),
-                certificateChain = data.createByteArray(),
-                persist = {
-                    if (generatedKeyInfo != null) {
-                        GeneratedKeyPersistence.rePersistIfNeeded(
-                            grant.ownerKeyId.uid,
-                            generatedKeyInfo,
-                        )
-                    }
-                },
-                label = "grant[${descriptor.nspace}] -> ${grant.ownerKeyId}",
-            )
-        }
-
        val generatedKeyInfo =
            when (descriptor.domain) {
                Domain.KEY_ID ->
                    KeyMintSecurityLevelInterceptor.findGeneratedKeyByKeyId(
-                        callingUid,
-                        descriptor.nspace,
+                        callingUid, descriptor.nspace
                    )
                Domain.APP ->
                    descriptor.alias?.let {
@@ -776,58 +527,23 @@ object Keystore2Interceptor : AbstractKeystoreInterceptor() {
            }

        if (generatedKeyInfo == null) {
-            // Patch-mode key (cached in teeResponses, not generatedKeys): the real keystore2
-            // applies
-            // the update, so drop our stale cached chain. Otherwise getKeyEntry replays the
-            // pre-update generated attestation (duck STALE_TEE_RESPONSE_AFTER_KEY_ID_UPDATE).
-            when (descriptor.domain) {
-                Domain.KEY_ID ->
-                    KeyMintSecurityLevelInterceptor.evictTeeResponseByKeyId(
-                        callingUid,
-                        descriptor.nspace,
-                    )
-                Domain.APP ->
-                    descriptor.alias?.let {
-                        KeyMintSecurityLevelInterceptor.evictTeeResponse(
-                            KeyIdentifier(callingUid, it)
-                        )
-                    }
-                else -> {}
-            }
            descriptor.alias?.let {
                val kid = KeyIdentifier(callingUid, it)
                userUpdatedKeys.add(kid)
-                SystemLogger.trace {
-                    "[TRACE] updateSubcomponent $kid: not generated key, added to userUpdatedKeys"
-                }
+                SystemLogger.trace { "[TRACE] updateSubcomponent $kid: not generated key, added to userUpdatedKeys" }
            }
            return TransactionResult.ContinueAndSkipPost
        }

-        return updateResponseSubcomponent(
-            response = generatedKeyInfo.response,
-            publicCert = data.createByteArray(),
-            certificateChain = data.createByteArray(),
-            persist = {
-                GeneratedKeyPersistence.rePersistIfNeeded(callingUid, generatedKeyInfo)
-            },
-            label = "key[${generatedKeyInfo.nspace}]",
-        )
-    }
+        SystemLogger.info("Updating sub-component with key[${generatedKeyInfo.nspace}]")
+        val metadata = generatedKeyInfo.response.metadata
+        val publicCert = data.createByteArray()
+        val certificateChain = data.createByteArray()

-    private fun updateResponseSubcomponent(
-        response: KeyEntryResponse,
-        publicCert: ByteArray?,
-        certificateChain: ByteArray?,
-        persist: () -> Unit,
-        label: String,
-    ): TransactionResult {
-        SystemLogger.info("Updating sub-component with $label")
-        val metadata = response.metadata
        metadata.certificate = publicCert
        metadata.certificateChain = certificateChain

-        persist()
+        GeneratedKeyPersistence.rePersistIfNeeded(callingUid, generatedKeyInfo)

        SystemLogger.verbose(
            "Key updated with sizes: [publicCert, certificateChain] = [${publicCert?.size}, ${certificateChain?.size}]"
@@ -1,113 +0,0 @@
-package org.matrix.TEESimulator.interception.keystore
-
-import android.os.IBinder
-import android.os.Parcel
-import android.security.maintenance.IKeystoreMaintenance
-import android.system.keystore2.Domain
-import android.system.keystore2.KeyDescriptor
-import org.matrix.TEESimulator.interception.core.BinderInterceptor
-import org.matrix.TEESimulator.interception.keystore.shim.KeyMintSecurityLevelInterceptor
-
-/**
- * Intercepts the keystore2 daemon's `android.security.maintenance` binder so our synthetic key
- * state follows the same lifecycle events the platform applies to real keys.
- *
- * This is a pure side-effect hook: every handled transaction mutates only our own synthetic state
- * and then returns [TransactionResult.ContinueAndSkipPost], so the real keystore2 still performs
- * the real operation. We never fabricate a maintenance reply, so real key lifecycle is never
- * disturbed.
- *
- * Mounted via `register()` from [Keystore2Interceptor.onInterceptorReady]; the maintenance binder
- * is hosted by the same keystore2 process, so the already-injected native hook reaches it too.
- */
-object Keystore2MaintenanceInterceptor : BinderInterceptor() {
-    private val stubClass = IKeystoreMaintenance.Stub::class.java
-
-    private val CLEAR_NAMESPACE_TRANSACTION =
-        InterceptorUtils.getTransactCode(stubClass, "clearNamespace")
-    private val DELETE_ALL_KEYS_TRANSACTION =
-        InterceptorUtils.getTransactCode(stubClass, "deleteAllKeys")
-    private val MIGRATE_KEY_NAMESPACE_TRANSACTION =
-        InterceptorUtils.getTransactCode(stubClass, "migrateKeyNamespace")
-
-    /** Only the lifecycle transactions we mirror; unresolved codes (-1) are dropped. */
-    val interceptedCodes: IntArray by lazy {
-        listOf(
-                CLEAR_NAMESPACE_TRANSACTION,
-                DELETE_ALL_KEYS_TRANSACTION,
-                MIGRATE_KEY_NAMESPACE_TRANSACTION,
-            )
-            .filter { it != -1 }
-            .toIntArray()
-    }
-
-    override fun onPreTransact(
-        txId: Long,
-        target: IBinder,
-        code: Int,
-        flags: Int,
-        callingUid: Int,
-        callingPid: Int,
-        data: Parcel,
-    ): TransactionResult {
-        when (code) {
-            CLEAR_NAMESPACE_TRANSACTION -> handleClearNamespace(data)
-            DELETE_ALL_KEYS_TRANSACTION ->
-                KeyMintSecurityLevelInterceptor.clearAllGeneratedKeys("maintenance.deleteAllKeys")
-            MIGRATE_KEY_NAMESPACE_TRANSACTION -> handleMigrateKeyNamespace(data, callingUid)
-        }
-        // Always let the real keystore2 perform the real lifecycle operation.
-        return TransactionResult.ContinueAndSkipPost
-    }
-
-    private fun handleClearNamespace(data: Parcel) {
-        data.enforceInterface(IKeystoreMaintenance.DESCRIPTOR)
-        val domain = data.readInt()
-        val nspace = data.readLong()
-        // Only Domain.APP namespaces map to our per-uid synthetic keys; nspace is the app uid.
-        if (domain == Domain.APP) {
-            KeyMintSecurityLevelInterceptor.clearNamespaceKeys(nspace.toInt())
-        }
-    }
-
-    private fun handleMigrateKeyNamespace(data: Parcel, callingUid: Int) {
-        data.enforceInterface(IKeystoreMaintenance.DESCRIPTOR)
-        val source = data.readTypedObject(KeyDescriptor.CREATOR) ?: return
-        val destination = data.readTypedObject(KeyDescriptor.CREATOR) ?: return
-        val srcId = resolveSyntheticKeyId(source, callingUid) ?: return
-        if (!KeyMintSecurityLevelInterceptor.generatedKeys.containsKey(srcId)) return // not ours
-
-        val dstId = resolveDestinationKeyId(destination, callingUid)
-        if (dstId == null) {
-            // Migrated out of our trackable (Domain.APP/alias) space -> drop our shadow so reads
-            // fall through to the real keystore2, which now owns it at the new namespace.
-            KeyMintSecurityLevelInterceptor.cleanupKeyData(srcId)
-        } else {
-            KeyMintSecurityLevelInterceptor.migrateGeneratedKey(srcId, dstId)
-        }
-    }
-
-    /** Resolves a synthetic owner key from a source descriptor (Domain.APP alias or KEY_ID). */
-    private fun resolveSyntheticKeyId(descriptor: KeyDescriptor, callingUid: Int): KeyIdentifier? =
-        when {
-            descriptor.alias != null -> KeyIdentifier(callingUid, descriptor.alias)
-            descriptor.domain == Domain.KEY_ID ->
-                KeyMintSecurityLevelInterceptor.generatedKeys.entries
-                    .firstOrNull {
-                        it.key.uid == callingUid && it.value.nspace == descriptor.nspace
-                    }
-                    ?.key
-            else -> null
-        }
-
-    /** Destination must be an addressable Domain.APP alias for us to keep tracking the key. */
-    private fun resolveDestinationKeyId(
-        descriptor: KeyDescriptor,
-        callingUid: Int,
-    ): KeyIdentifier? {
-        val alias = descriptor.alias ?: return null
-        if (descriptor.domain != Domain.APP) return null
-        val uid = if (descriptor.nspace > 0) descriptor.nspace.toInt() else callingUid
-        return KeyIdentifier(uid, alias)
-    }
-}
@@ -1,10 +1,8 @@
 package org.matrix.TEESimulator.interception.keystore.shim

 import android.hardware.security.keymint.Algorithm
-import android.hardware.security.keymint.BlockMode
-import android.hardware.security.keymint.KeyParameter
 import android.hardware.security.keymint.KeyPurpose
-import android.hardware.security.keymint.PaddingMode
+import android.hardware.security.keymint.KeyParameter
 import android.hardware.security.keymint.Tag
 import org.matrix.TEESimulator.attestation.KeyMintAttestation

@@ -20,15 +18,13 @@ object AuthorizeCreate {
        // Algorithm-level rejection runs before purpose-list check (AOSP HAL behavior)
        return checkAlgorithmPurpose(keyParams, purpose)
            ?: checkPurpose(keyParams, purpose)
-            ?: checkOperationAuthorizations(keyParams, opParams)
            ?: checkTemporalValidity(keyParams, purpose)
            ?: checkCallerNonce(keyParams, purpose, rawOpParams)
    }

    private fun checkAlgorithmPurpose(keyParams: KeyMintAttestation, purpose: Int): Int? {
        val algo = keyParams.algorithm
-        if (
-            (algo == Algorithm.EC || algo == Algorithm.RSA) &&
+        if ((algo == Algorithm.EC || algo == Algorithm.RSA) &&
            (purpose == KeyPurpose.VERIFY || purpose == KeyPurpose.ENCRYPT)
        ) {
            return KeystoreErrorCodes.unsupportedPurpose
@@ -39,48 +35,10 @@ object AuthorizeCreate {
    }

    private fun checkPurpose(keyParams: KeyMintAttestation, purpose: Int): Int? {
-        if (purpose == KeyPurpose.WRAP_KEY) return KeystoreErrorCodes.incompatiblePurpose
-        if (purpose !in keyParams.purpose) return KeystoreErrorCodes.incompatiblePurpose
-        return null
-    }
-
-    private fun checkOperationAuthorizations(
-        keyParams: KeyMintAttestation,
-        opParams: KeyMintAttestation,
-    ): Int? {
-        if (opParams.blockMode.any { it !in keyParams.blockMode }) {
-            return KeystoreErrorCodes.incompatibleBlockMode
-        }
-        if (opParams.padding.any { it !in keyParams.padding }) {
-            return KeystoreErrorCodes.incompatiblePaddingMode
-        }
-        if (opParams.digest.any { it !in keyParams.digest }) {
-            return KeystoreErrorCodes.incompatibleDigest
-        }
-        if (opParams.rsaOaepMgfDigest.any { it !in keyParams.rsaOaepMgfDigest }) {
-            return KeystoreErrorCodes.incompatibleDigest
-        }
-
-        if (keyParams.algorithm == Algorithm.AES && opParams.blockMode.contains(BlockMode.GCM)) {
-            val requestedMacLength = opParams.minMacLength
-            val keyMinMacLength = keyParams.minMacLength
-            if (
-                requestedMacLength != null &&
-                    keyMinMacLength != null &&
-                    requestedMacLength < keyMinMacLength
-            ) {
-                return KeystoreErrorCodes.invalidMacLength
-            }
-        }
-
-        if (
-            keyParams.algorithm == Algorithm.RSA &&
-                opParams.padding.contains(PaddingMode.RSA_OAEP) &&
-                opParams.digest.isEmpty()
-        ) {
-            return KeystoreErrorCodes.incompatibleDigest
-        }
-
+        if (purpose == KeyPurpose.WRAP_KEY)
+            return KeystoreErrorCodes.incompatiblePurpose
+        if (purpose !in keyParams.purpose)
+            return KeystoreErrorCodes.incompatiblePurpose
        return null
    }

@@ -106,11 +64,7 @@ object AuthorizeCreate {
        return null
    }

-    private fun checkCallerNonce(
-        keyParams: KeyMintAttestation,
-        purpose: Int,
-        rawOpParams: Array<KeyParameter>?,
-    ): Int? {
+    private fun checkCallerNonce(keyParams: KeyMintAttestation, purpose: Int, rawOpParams: Array<KeyParameter>?): Int? {
        if (purpose != KeyPurpose.SIGN && purpose != KeyPurpose.ENCRYPT) return null
        if (keyParams.callerNonce == true) return null
        if (rawOpParams?.any { it.tag == Tag.NONCE } == true)
@@ -33,17 +33,19 @@ data class PersistedKeyData(
    val privateKeyBytes: ByteArray,
    val certChainBytes: List<ByteArray>,
    /**
-     * Byte-identical KeyMetadata parcel snapshot. Restoring authorizations directly from these
-     * bytes preserves tag count, order, and exact security-level annotations across reboots — the
-     * kind of structural details apps fingerprint to decide whether the alias is still "the same
-     * key".
+     * Byte-identical KeyMetadata parcel snapshot. Restoring authorizations
+     * directly from these bytes preserves tag count, order, and exact
+     * security-level annotations across reboots — the kind of structural
+     * details apps fingerprint to decide whether the alias is still
+     * "the same key".
     */
    val metadataBytes: ByteArray,
    /**
-     * Raw secret material for symmetric records (AES, HMAC, 3DES). Empty for asymmetric. Critical
-     * for AndroidX security crypto MasterKey (AES-GCM-256) — without this every reboot regenerates
-     * a fresh AES key and EncryptedSharedPreferences becomes undecryptable, which is what banking
-     * apps interpret as session expiry and force a relogin.
+     * Raw secret material for symmetric records (AES, HMAC, 3DES). Empty
+     * for asymmetric. Critical for AndroidX security crypto MasterKey
+     * (AES-GCM-256) — without this every reboot regenerates a fresh AES
+     * key and EncryptedSharedPreferences becomes undecryptable, which is
+     * what banking apps interpret as session expiry and force a relogin.
     */
    val symmetricKeyBytes: ByteArray,
    val symmetricAlgorithm: String,
@@ -52,15 +54,20 @@ data class PersistedKeyData(
 object GeneratedKeyPersistence {

    /**
-     * Single source of truth for the on-disk format. Bump this every time the layout changes; older
-     * numbers are silently skipped on read so stale dev artifacts and pre-fix upstream files can't
-     * be partially rehydrated into broken in-memory state.
+     * Single source of truth for the on-disk format. Bump this every time
+     * the layout changes; older numbers are silently skipped on read so
+     * stale dev artifacts and pre-fix upstream files can't be partially
+     * rehydrated into broken in-memory state.
     *
-     * History: 1 — original upstream layout (no metadata snapshot, no symmetric block; restored
-     * keys lose authorization tags and AES master keys altogether — apps relying on persisted
-     * keystore state across reboots get logged out) 2 — transitional dev-only format that added
-     * metadata but still missed the symmetric block; never shipped 3 — current: byte-identical
-     * KeyMetadata snapshot + raw symmetric key material so AES/HMAC keys survive reboots
+     * History:
+     *   1 — original upstream layout (no metadata snapshot, no symmetric
+     *       block; restored keys lose authorization tags and AES master
+     *       keys altogether — apps relying on persisted keystore state
+     *       across reboots get logged out)
+     *   2 — transitional dev-only format that added metadata but still
+     *       missed the symmetric block; never shipped
+     *   3 — current: byte-identical KeyMetadata snapshot + raw symmetric
+     *       key material so AES/HMAC keys survive reboots
     */
    private const val FORMAT_VERSION = 3
    private val PERSISTENCE_DIR = File(CONFIG_PATH, "persistent_keys")
@@ -102,8 +109,7 @@ object GeneratedKeyPersistence {
                val tmpFile = File(PERSISTENCE_DIR, "$filename.tmp")

                try {
-                        DataOutputStream(BufferedOutputStream(FileOutputStream(tmpFile))).use { out
-                            ->
+                    DataOutputStream(BufferedOutputStream(FileOutputStream(tmpFile))).use { out ->
                        out.writeInt(FORMAT_VERSION)
                        out.writeInt(securityLevel)
                        out.writeInt(keyId.uid)
@@ -154,13 +160,10 @@ object GeneratedKeyPersistence {
                    throw e
                }

-                    // Atomic rename — if this fails the tmp is left behind and cleaned on next
-                    // deleteAll
+                // Atomic rename — if this fails the tmp is left behind and cleaned on next deleteAll
                if (!tmpFile.renameTo(finalFile)) {
                    tmpFile.delete()
-                        throw IllegalStateException(
-                            "Failed to atomically rename $tmpFile -> $finalFile"
-                        )
+                    throw IllegalStateException("Failed to atomically rename $tmpFile -> $finalFile")
                }

                // Verify write succeeded - catches disk-full or filesystem errors
@@ -169,8 +172,9 @@ object GeneratedKeyPersistence {
                }

                SystemLogger.debug("Persisted key: $keyId")
+            }.onFailure { e ->
+                SystemLogger.error("Failed to persist key $keyId", e)
            }
-                .onFailure { e -> SystemLogger.error("Failed to persist key $keyId", e) }
        } finally {
            lock.unlock()
            SystemLogger.debug("[Persistence] Lock released for $filename")
@@ -190,8 +194,9 @@ object GeneratedKeyPersistence {
            } else {
                SystemLogger.debug("No persisted file to delete for: $keyId")
            }
+        }.onFailure { e ->
+            SystemLogger.error("Failed to delete persisted key $keyId", e)
        }
-            .onFailure { e -> SystemLogger.error("Failed to delete persisted key $keyId", e) }
    }

    fun deleteAll() {
@@ -213,8 +218,9 @@ object GeneratedKeyPersistence {
            }
            fileLocks.clear()
            SystemLogger.info("Deleted $count persisted key files")
+        }.onFailure { e ->
+            SystemLogger.error("Failed to delete all persisted keys", e)
        }
-            .onFailure { e -> SystemLogger.error("Failed to delete all persisted keys", e) }
    }

    fun loadAll(securityLevel: Int): List<PersistedKeyData> {
@@ -275,8 +281,7 @@ object GeneratedKeyPersistence {
                    if (pkLen > 0) input.readFully(pkBytes)

                    val certCount = requireBounds(input.readInt(), 10, "certCount")
-                        val certChainBytes =
-                            (0 until certCount).map {
+                    val certChainBytes = (0 until certCount).map {
                        val certLen = requireBounds(input.readInt(), 65536, "certLen")
                        val certBytes = ByteArray(certLen)
                        input.readFully(certBytes)
@@ -284,12 +289,15 @@ object GeneratedKeyPersistence {
                    }

                    val metaLen = requireBounds(input.readInt(), 256 * 1024, "metaLen")
-                        val metadataBytes =
-                            ByteArray(metaLen).also { if (metaLen > 0) input.readFully(it) }
+                    val metadataBytes = ByteArray(metaLen).also {
+                        if (metaLen > 0) input.readFully(it)
+                    }

                    val skAlgo = input.readUTF()
                    val skLen = requireBounds(input.readInt(), 8192, "skLen")
-                        val skBytes = ByteArray(skLen).also { if (skLen > 0) input.readFully(it) }
+                    val skBytes = ByteArray(skLen).also {
+                        if (skLen > 0) input.readFully(it)
+                    }

                    if (storedSecLevel == securityLevel) {
                        result.add(
@@ -313,8 +321,7 @@ object GeneratedKeyPersistence {
                        )
                    }
                }
-                }
-                .onFailure { e ->
+            }.onFailure { e ->
                SystemLogger.warning("Skipping corrupted persisted key file: ${file.name}", e)
            }
        }
@@ -338,14 +345,11 @@ object GeneratedKeyPersistence {
        }
        val secLevel = metadata.keySecurityLevel

-        val entry =
-            KeyMintSecurityLevelInterceptor.generatedKeys.entries.find { (id, info) ->
+        val entry = KeyMintSecurityLevelInterceptor.generatedKeys.entries.find { (id, info) ->
            id.uid == callingUid && info.nspace == generatedKeyInfo.nspace
        }
        if (entry == null) {
-            SystemLogger.debug(
-                "rePersist: key not found in map for uid=$callingUid nspace=${generatedKeyInfo.nspace}"
-            )
+            SystemLogger.debug("rePersist: key not found in map for uid=$callingUid nspace=${generatedKeyInfo.nspace}")
            return
        }

@@ -364,20 +368,16 @@ object GeneratedKeyPersistence {
            return
        }

-        val persisted =
-            runCatching {
+        val persisted = runCatching {
            DataInputStream(BufferedInputStream(FileInputStream(existing))).use { input ->
                val version = input.readInt()
                if (version != FORMAT_VERSION) {
-                            SystemLogger.warning(
-                                "rePersist: legacy format version $version for $keyId, will not re-persist (next generateKey replaces it)"
-                            )
+                    SystemLogger.warning("rePersist: legacy format version $version for $keyId, will not re-persist (next generateKey replaces it)")
                    return
                }
                readPersistedKeyData(input)
            }
-                }
-                .getOrNull()
+        }.getOrNull()
        if (persisted == null) {
            SystemLogger.warning("rePersist: failed to read existing data for $keyId")
            return
@@ -392,8 +392,7 @@ object GeneratedKeyPersistence {
        // Serialize the live KeyMetadata (now contains the user-installed cert
        // chain via updateSubcomponent) so the next boot restores byte-identical
        // metadata. KeyMetadata is binder-free, so marshall() is safe here.
-        val metadataBytes =
-            runCatching {
+        val metadataBytes = runCatching {
            android.os.Parcel.obtain().let { parcel ->
                try {
                    metadata.writeToParcel(parcel, 0)
@@ -402,8 +401,7 @@ object GeneratedKeyPersistence {
                    parcel.recycle()
                }
            }
-                }
-                .getOrNull()
+        }.getOrNull()
        save(
            keyId = keyId,
            keyPair = keyPair,
@@ -429,8 +427,8 @@ object GeneratedKeyPersistence {
    }

    private fun keyFileName(uid: Int, alias: String): String {
-        val digest =
-            MessageDigest.getInstance("SHA-256").digest("$uid:$alias".toByteArray(Charsets.UTF_8))
+        val digest = MessageDigest.getInstance("SHA-256")
+            .digest("$uid:$alias".toByteArray(Charsets.UTF_8))
        return digest.joinToString("") { "%02x".format(it) } + ".bin"
    }

@@ -457,8 +455,7 @@ object GeneratedKeyPersistence {
        if (pkLen > 0) input.readFully(pkBytes)

        val certCount = requireBounds(input.readInt(), 10, "certCount")
-        val certChainBytes =
-            (0 until certCount).map {
+        val certChainBytes = (0 until certCount).map {
            val certLen = requireBounds(input.readInt(), 65536, "certLen")
            val certBytes = ByteArray(certLen)
            input.readFully(certBytes)
@@ -466,11 +463,15 @@ object GeneratedKeyPersistence {
        }

        val metaLen = requireBounds(input.readInt(), 256 * 1024, "metaLen")
-        val metadataBytes = ByteArray(metaLen).also { if (metaLen > 0) input.readFully(it) }
+        val metadataBytes = ByteArray(metaLen).also {
+            if (metaLen > 0) input.readFully(it)
+        }

        val skAlgo = input.readUTF()
        val skLen = requireBounds(input.readInt(), 8192, "skLen")
-        val skBytes = ByteArray(skLen).also { if (skLen > 0) input.readFully(it) }
+        val skBytes = ByteArray(skLen).also {
+            if (skLen > 0) input.readFully(it)
+        }

        return PersistedKeyData(
            uid = uid,
@@ -28,15 +28,8 @@ class OperationInterceptor(
        val methodName = transactionNames[code] ?: "unknown code=$code"
        logTransaction(txId, methodName, callingUid, callingPid, true)

-        // Mirror SoftwareOperation's vendor gate: a real-key op must answer non-AEAD updateAad
-        // exactly as the forged-key path does. Samsung and Xiaomi-MTK TEEs accept it; rejecting
-        // here while the forged path accepts diverges the two and fingerprints the injection.
        if (code == UPDATE_AAD_TRANSACTION && !isAead) {
-            return if (VendorQuirks.nonAeadUpdateAadSucceeds()) {
-                InterceptorUtils.createSuccessReply(writeResultCode = false)
-            } else {
-                InterceptorUtils.createServiceSpecificErrorReply(KeystoreErrorCodes.invalidTag)
-            }
+            return InterceptorUtils.createServiceSpecificErrorReply(KeystoreErrorCodes.invalidTag)
        }

        if (code == FINISH_TRANSACTION || code == ABORT_TRANSACTION) {
@@ -8,61 +8,25 @@ import android.hardware.security.keymint.KeyParameterValue
 import android.hardware.security.keymint.KeyPurpose
 import android.hardware.security.keymint.PaddingMode
 import android.hardware.security.keymint.Tag
-import android.os.Build
 import android.os.ServiceSpecificException
-import android.os.SystemProperties
+import java.util.concurrent.locks.LockSupport
 import android.system.keystore2.IKeystoreOperation
 import android.system.keystore2.KeyParameters
 import java.security.KeyPair
 import java.security.Signature
 import java.security.SignatureException
-import java.util.concurrent.locks.LockSupport
 import javax.crypto.Cipher
 import org.matrix.TEESimulator.attestation.KeyMintAttestation
 import org.matrix.TEESimulator.logging.KeyMintParameterLogger
 import org.matrix.TEESimulator.logging.SystemLogger

-/**
- * Mirrors the per-vendor TEE quirk that Duck Detector's OperationErrorPathProbe checks: real
- * Samsung and Xiaomi-MTK TrustZone return success for updateAad on a non-AEAD operation, while
- * every other vendor rejects it with a service-specific INVALID_TAG. The module reads the same
- * device-identity fields the probe reads, so a forged software operation answers exactly as that
- * vendor's real TEE would.
- */
-internal object VendorQuirks {
-    private val UPDATE_AAD_ALLOWS_SUCCESS = setOf("samsung")
-    private val XIAOMI_BRANDS = setOf("xiaomi", "redmi", "poco")
-
-    fun nonAeadUpdateAadSucceeds(): Boolean {
-        val manufacturer = Build.MANUFACTURER.lowercase()
-        val brand = Build.BRAND.lowercase()
-        if (manufacturer in UPDATE_AAD_ALLOWS_SUCCESS || brand in UPDATE_AAD_ALLOWS_SUCCESS) {
-            return true
-        }
-        if (manufacturer != "xiaomi" && brand !in XIAOMI_BRANDS) return false
-        return isMediaTek()
-    }
-
-    private fun isMediaTek(): Boolean {
-        val roHardware = SystemProperties.get("ro.hardware", "")
-        return roHardware.startsWith("mt") || Build.HARDWARE.startsWith("mt", ignoreCase = true)
-    }
-}
-
 private sealed interface CryptoPrimitive {
    fun updateAad(aadInput: ByteArray?) {
-        // Real Samsung / Xiaomi-MTK TEEs accept updateAad on non-AEAD ops; others reject it.
-        if (!VendorQuirks.nonAeadUpdateAadSucceeds()) {
        throw ServiceSpecificException(KeystoreErrorCodes.invalidTag)
    }
-    }
-
    fun update(data: ByteArray?): ByteArray?
-
    fun finish(data: ByteArray?, signature: ByteArray?): ByteArray?
-
    fun abort()
-
    fun getBeginParameters(): Array<KeyParameter>? = null
 }

@@ -119,24 +83,6 @@ private object JcaAlgorithmMapper {
            }
        return "$keyAlgo/$blockMode/$padding"
    }
-
-    fun mapOaepDigest(digest: Int?): String =
-        when (digest) {
-            Digest.SHA1 -> "SHA-1"
-            Digest.SHA_2_224 -> "SHA-224"
-            Digest.SHA_2_256 -> "SHA-256"
-            Digest.SHA_2_384 -> "SHA-384"
-            Digest.SHA_2_512 -> "SHA-512"
-            else -> "SHA-256"
-        }
-
-    fun mapMacAlgorithm(params: KeyMintAttestation): String =
-        when (params.digest.firstOrNull()) {
-            Digest.SHA_2_256 -> "HmacSHA256"
-            Digest.SHA_2_384 -> "HmacSHA384"
-            Digest.SHA_2_512 -> "HmacSHA512"
-            else -> "HmacSHA256"
-        }
 }

 private class Signer(keyPair: KeyPair, params: KeyMintAttestation) : CryptoPrimitive {
@@ -172,16 +118,10 @@ private class Verifier(keyPair: KeyPair, params: KeyMintAttestation) : CryptoPri
    override fun finish(data: ByteArray?, signature: ByteArray?): ByteArray? {
        if (data != null) update(data)
        if (signature == null) {
-            throw ServiceSpecificException(
-                KeystoreErrorCodes.verificationFailed,
-                "Signature to verify is null",
-            )
+            throw ServiceSpecificException(KeystoreErrorCodes.verificationFailed, "Signature to verify is null")
        }
        if (!this.signature.verify(signature)) {
-            throw ServiceSpecificException(
-                KeystoreErrorCodes.verificationFailed,
-                "Signature verification failed",
-            )
+            throw ServiceSpecificException(KeystoreErrorCodes.verificationFailed, "Signature verification failed")
        }
        return null
    }
@@ -193,7 +133,6 @@ private class CipherPrimitive(
    cryptoKey: java.security.Key,
    params: KeyMintAttestation,
    private val opMode: Int,
-    txId: Long,
 ) : CryptoPrimitive {
    private val isAead = params.blockMode.firstOrNull() == BlockMode.GCM
    private val cipher: Cipher =
@@ -203,38 +142,13 @@ private class CipherPrimitive(
                init(opMode, cryptoKey, javax.crypto.spec.GCMParameterSpec(128, nonce))
            } else if (nonce != null) {
                init(opMode, cryptoKey, javax.crypto.spec.IvParameterSpec(nonce))
-            } else if (params.padding.firstOrNull() == PaddingMode.RSA_OAEP) {
-                val mainDigest = JcaAlgorithmMapper.mapOaepDigest(params.digest.firstOrNull())
-                val mgfDigest =
-                    params.rsaOaepMgfDigest.firstOrNull()?.let {
-                        JcaAlgorithmMapper.mapOaepDigest(it)
-                    } ?: mainDigest
-                init(
-                    opMode,
-                    cryptoKey,
-                    javax.crypto.spec.OAEPParameterSpec(
-                        mainDigest,
-                        "MGF1",
-                        java.security.spec.MGF1ParameterSpec(mgfDigest),
-                        javax.crypto.spec.PSource.PSpecified.DEFAULT,
-                    ),
-                )
-                SystemLogger.debug {
-                    "[SoftwareOp TX_ID: $txId] oaep-op main=$mainDigest mgf=$mgfDigest " +
-                        "mode=${if (opMode == Cipher.DECRYPT_MODE) "decrypt" else "encrypt"}"
-                }
            } else {
                init(opMode, cryptoKey)
            }
        }

    override fun updateAad(aadInput: ByteArray?) {
-        if (!isAead) {
-            if (!VendorQuirks.nonAeadUpdateAadSucceeds()) {
-                throw ServiceSpecificException(KeystoreErrorCodes.invalidTag)
-            }
-            return
-        }
+        if (!isAead) throw ServiceSpecificException(KeystoreErrorCodes.invalidTag)
        if (aadInput != null) cipher.updateAAD(aadInput)
    }

@@ -279,53 +193,6 @@ private class KeyAgreementPrimitive(keyPair: KeyPair) : CryptoPrimitive {
    override fun abort() {}
 }

-private class MacPrimitive(
-    secretKey: javax.crypto.SecretKey,
-    private val params: KeyMintAttestation,
-    private val txId: Long,
-) : CryptoPrimitive {
-    private val mac: javax.crypto.Mac =
-        javax.crypto.Mac.getInstance(JcaAlgorithmMapper.mapMacAlgorithm(params)).apply {
-            init(secretKey)
-        }
-
-    override fun update(data: ByteArray?): ByteArray? {
-        if (data != null) mac.update(data)
-        return null
-    }
-
-    override fun finish(data: ByteArray?, signature: ByteArray?): ByteArray? {
-        if (data != null) mac.update(data)
-        val full = mac.doFinal()
-        // Tag.MAC_LENGTH is optional on the AndroidKeyStore Mac SPI; default to the
-        // full digest length so real Mac use keeps working when it is omitted.
-        val tagBytes = (params.macLength ?: (full.size * 8)) / 8
-        val tag = full.copyOf(tagBytes)
-        if (params.purpose.firstOrNull() == KeyPurpose.VERIFY) {
-            if (signature == null) {
-                throw ServiceSpecificException(
-                    KeystoreErrorCodes.verificationFailed,
-                    "MAC to verify is null",
-                )
-            }
-            if (!java.security.MessageDigest.isEqual(tag, signature)) {
-                throw ServiceSpecificException(
-                    KeystoreErrorCodes.verificationFailed,
-                    "MAC verification failed",
-                )
-            }
-            return null
-        }
-        SystemLogger.debug {
-            "[SoftwareOp TX_ID: $txId] hmac-op digest=${params.digest.firstOrNull()} " +
-                "macLen=${params.macLength} tag=${tag.size}B result=ok"
-        }
-        return tag
-    }
-
-    override fun abort() {}
-}
-
 class SoftwareOperation(
    private val txId: Long,
    keyPair: KeyPair?,
@@ -334,8 +201,7 @@ class SoftwareOperation(
    private val latencyFloorMs: Long = 0L,
 ) {
    private val primitive: CryptoPrimitive
-    @Volatile
-    var finalized = false
+    @Volatile var finalized = false
        private set

    var onFinishCallback: (() -> Unit)? = null
@@ -374,64 +240,39 @@ class SoftwareOperation(
        }

        primitive =
-            if (params.algorithm == Algorithm.HMAC) {
-                // An HMAC key is symmetric (secretKey set, keyPair null), so it must
-                // not fall through to the purpose-keyed Signer/Verifier paths, which
-                // require a keyPair. secretKey is populated at HMAC keygen and restore,
-                // so the throw is a defensive floor, not a live path.
-                MacPrimitive(
-                    secretKey
-                        ?: throw ServiceSpecificException(
-                            KeystoreErrorCodes.invalidArgument,
-                            "[SoftwareOp TX_ID: $txId] HMAC op but secretKey null",
-                        ),
-                    params,
-                    txId,
-                )
-            } else {
            when (purpose) {
                KeyPurpose.SIGN -> {
-                        val kp =
-                            keyPair
-                                ?: throw ServiceSpecificException(
+                    val kp = keyPair ?: throw ServiceSpecificException(
                        KeystoreErrorCodes.invalidArgument,
                        "[SoftwareOp TX_ID: $txId] SIGN requested but keyPair is null",
                    )
                    Signer(kp, params)
                }
                KeyPurpose.VERIFY -> {
-                        val kp =
-                            keyPair
-                                ?: throw ServiceSpecificException(
+                    val kp = keyPair ?: throw ServiceSpecificException(
                        KeystoreErrorCodes.invalidArgument,
                        "[SoftwareOp TX_ID: $txId] VERIFY requested but keyPair is null",
                    )
                    Verifier(kp, params)
                }
                KeyPurpose.ENCRYPT -> {
-                        val key: java.security.Key =
-                            secretKey
-                                ?: keyPair?.public
+                    val key: java.security.Key = secretKey ?: keyPair?.public
                        ?: throw ServiceSpecificException(
                            KeystoreErrorCodes.unsupportedPurpose,
                            "[SoftwareOp TX_ID: $txId] ENCRYPT requires either secretKey or keyPair.public",
                        )
-                        CipherPrimitive(key, params, Cipher.ENCRYPT_MODE, txId)
+                    CipherPrimitive(key, params, Cipher.ENCRYPT_MODE)
                }
                KeyPurpose.DECRYPT -> {
-                        val key: java.security.Key =
-                            secretKey
-                                ?: keyPair?.private
+                    val key: java.security.Key = secretKey ?: keyPair?.private
                        ?: throw ServiceSpecificException(
                            KeystoreErrorCodes.unsupportedPurpose,
                            "[SoftwareOp TX_ID: $txId] DECRYPT requires either secretKey or keyPair.private",
                        )
-                        CipherPrimitive(key, params, Cipher.DECRYPT_MODE, txId)
+                    CipherPrimitive(key, params, Cipher.DECRYPT_MODE)
                }
                KeyPurpose.AGREE_KEY -> {
-                        val kp =
-                            keyPair
-                                ?: throw ServiceSpecificException(
+                    val kp = keyPair ?: throw ServiceSpecificException(
                        KeystoreErrorCodes.invalidArgument,
                        "[SoftwareOp TX_ID: $txId] AGREE_KEY requested but keyPair is null",
                    )
@@ -444,43 +285,32 @@ class SoftwareOperation(
                    )
            }
    }
-    }

    private fun checkActive() {
        if (finalized) {
-            SystemLogger.debug(
-                "[SoftwareOp TX_ID: $txId] Rejected: operation already finalized (pruned or completed)"
-            )
+            SystemLogger.debug("[SoftwareOp TX_ID: $txId] Rejected: operation already finalized (pruned or completed)")
            throw ServiceSpecificException(KeystoreErrorCodes.invalidOperationHandle)
        }
    }

    private fun checkInputLength(data: ByteArray?) {
        if (data != null && data.size > MAX_RECEIVE_DATA) {
-            SystemLogger.info(
-                "[SoftwareOp TX_ID: $txId] Input too large: ${data.size} > $MAX_RECEIVE_DATA, throwing TOO_MUCH_DATA(${KeystoreErrorCodes.tooMuchData})"
-            )
+            SystemLogger.info("[SoftwareOp TX_ID: $txId] Input too large: ${data.size} > $MAX_RECEIVE_DATA, throwing TOO_MUCH_DATA(${KeystoreErrorCodes.tooMuchData})")
            throw ServiceSpecificException(KeystoreErrorCodes.tooMuchData)
        }
    }

    fun updateAad(aadInput: ByteArray?) {
-        SystemLogger.info(
-            "[SoftwareOp TX_ID: $txId] updateAad() ENTRY inputSize=${aadInput?.size ?: 0} primitive=${primitive::class.simpleName}"
-        )
+        SystemLogger.info("[SoftwareOp TX_ID: $txId] updateAad() ENTRY inputSize=${aadInput?.size ?: 0} primitive=${primitive::class.simpleName}")
        checkActive()
        checkInputLength(aadInput)
        try {
            primitive.updateAad(aadInput)
-            SystemLogger.info(
-                "[SoftwareOp TX_ID: $txId] updateAad() RETURNED_NORMALLY (unexpected for non-AEAD)"
-            )
+            SystemLogger.info("[SoftwareOp TX_ID: $txId] updateAad() RETURNED_NORMALLY (unexpected for non-AEAD)")
        } catch (throwable: Throwable) {
            val top = throwable.stackTrace.firstOrNull()?.toString() ?: "<no-frame>"
            val code = (throwable as? ServiceSpecificException)?.errorCode
-            SystemLogger.info(
-                "[SoftwareOp TX_ID: $txId] updateAad() THREW class=${throwable::class.java.name} code=$code msg=${throwable.message} top=$top"
-            )
+            SystemLogger.info("[SoftwareOp TX_ID: $txId] updateAad() THREW class=${throwable::class.java.name} code=$code msg=${throwable.message} top=$top")
            throw throwable
        }
    }
@@ -528,16 +358,11 @@ class SoftwareOperation(
        SystemLogger.debug("[SoftwareOp TX_ID: $txId] Operation aborted.")
    }

-    private fun mapToServiceSpecificException(e: Exception): ServiceSpecificException =
-        when (e) {
-            is SignatureException ->
-                ServiceSpecificException(KeystoreErrorCodes.verificationFailed, e.message)
-            is javax.crypto.BadPaddingException ->
-                ServiceSpecificException(KeystoreErrorCodes.invalidArgument, e.message)
-            is javax.crypto.IllegalBlockSizeException ->
-                ServiceSpecificException(KeystoreErrorCodes.invalidInputLength, e.message)
-            is java.security.InvalidKeyException ->
-                ServiceSpecificException(KeystoreErrorCodes.incompatibleKey, e.message)
+    private fun mapToServiceSpecificException(e: Exception): ServiceSpecificException = when (e) {
+        is SignatureException -> ServiceSpecificException(KeystoreErrorCodes.verificationFailed, e.message)
+        is javax.crypto.BadPaddingException -> ServiceSpecificException(KeystoreErrorCodes.invalidArgument, e.message)
+        is javax.crypto.IllegalBlockSizeException -> ServiceSpecificException(KeystoreErrorCodes.invalidInputLength, e.message)
+        is java.security.InvalidKeyException -> ServiceSpecificException(KeystoreErrorCodes.incompatibleKey, e.message)
        else -> ServiceSpecificException(KeystoreErrorCodes.unknownError, e.message)
    }

@@ -603,25 +428,10 @@ internal object KeystoreErrorCodes {
        resolveField("android.hardware.security.keymint.ErrorCode", "UNKNOWN_ERROR", -1000)
    }

-    val incompatibleBlockMode: Int by lazy {
-        resolveField("android.hardware.security.keymint.ErrorCode", "INCOMPATIBLE_BLOCK_MODE", -8)
-    }
-
-    val incompatiblePaddingMode: Int by lazy {
-        resolveField("android.hardware.security.keymint.ErrorCode", "INCOMPATIBLE_PADDING_MODE", -11)
-    }
-
-    val incompatibleDigest: Int by lazy {
-        resolveField("android.hardware.security.keymint.ErrorCode", "INCOMPATIBLE_DIGEST", -13)
-    }
-
-    val invalidMacLength: Int by lazy {
-        resolveField("android.hardware.security.keymint.ErrorCode", "INVALID_MAC_LENGTH", -57)
-    }
-
    fun resolveField(className: String, fieldName: String, fallback: Int): Int =
-        runCatching { Class.forName(className).getField(fieldName).getInt(null) }
-            .getOrElse {
+        runCatching {
+            Class.forName(className).getField(fieldName).getInt(null)
+        }.getOrElse {
            SystemLogger.debug("Resolved $className.$fieldName via fallback: $fallback")
            fallback
        }
@@ -632,17 +442,13 @@ class SoftwareOperationBinder(private val operation: SoftwareOperation) :

    @Synchronized
    override fun updateAad(aadInput: ByteArray?) {
-        SystemLogger.info(
-            "[SoftwareOpBinder] updateAad() ENTRY callingUid=${android.os.Binder.getCallingUid()} size=${aadInput?.size ?: 0}"
-        )
+        SystemLogger.info("[SoftwareOpBinder] updateAad() ENTRY callingUid=${android.os.Binder.getCallingUid()} size=${aadInput?.size ?: 0}")
        try {
            operation.updateAad(aadInput)
            SystemLogger.info("[SoftwareOpBinder] updateAad() RETURNED_NORMALLY")
        } catch (throwable: Throwable) {
            val code = (throwable as? ServiceSpecificException)?.errorCode
-            SystemLogger.info(
-                "[SoftwareOpBinder] updateAad() PROPAGATING class=${throwable::class.java.name} code=$code msg=${throwable.message}"
-            )
+            SystemLogger.info("[SoftwareOpBinder] updateAad() PROPAGATING class=${throwable::class.java.name} code=$code msg=${throwable.message}")
            throw throwable
        }
    }
@@ -1,170 +0,0 @@
-package org.matrix.TEESimulator.interception.soter
-
-import android.content.ComponentName
-import android.content.Context
-import android.content.Intent
-import android.content.ServiceConnection
-import android.os.Handler
-import android.os.HandlerThread
-import android.os.IBinder
-import java.util.concurrent.Executor
-import java.util.concurrent.atomic.AtomicBoolean
-import org.matrix.TEESimulator.interception.core.BinderInterceptor
-import org.matrix.TEESimulator.logging.SystemLogger
-
-/**
- * Keeps [SoterServiceInterceptor] mounted on the on-demand, restartable
- * `com.tencent.soter.soterserver` process.
- *
- * `AbstractKeystoreInterceptor` injects `keystore2` exactly once: it is always alive and
- * servicemanager-published, so the daemon gets its binder from `ServiceManager` and may
- * `exitProcess` on failure. soterserver inverts both — it is Intent-bound (NOT in
- * `ServiceManager`) and may die and respawn. This supervisor therefore *binds* the SOTER
- * service, which both triggers its on-demand start AND yields the `ISoterService` binder
- * (the target the native MITM registry keys on); injects `libTEESimulator.so` on every
- * (re)start; confirms the landing with the `0xdeadbeef` backdoor handshake; then registers
- * the forge. It re-binds — re-poking, re-injecting, re-registering — whenever the process
- * dies, never exiting.
- *
- * The bind recipe (action = the interface descriptor, package, `BIND_AUTO_CREATE`) and the
- * rebind-on-death lifecycle mirror the SOTER SDK's own `SoterCoreTreble`, so the daemon
- * connects exactly as a real client would. Everything runs on a dedicated [HandlerThread]
- * so it never stalls keystore init or `Looper.loop()` in [org.matrix.TEESimulator.App].
- *
- * Observability (the checkpoint's mandatory gate): every lifecycle event — bind, connect,
- * inject ok/fail, handshake, respawn — is logged via [SystemLogger], debug-gated. It never
- * gates the forge.
- */
-object SoterProcessSupervisor {
-
-    /** soterserver hosts the package's own process (recon 2026-06-26: process == package). */
-    private const val SOTER_PACKAGE = "com.tencent.soter.soterserver"
-
-    /** Reuses the daemon's native injector + `entry`, PID-resolved by the target package. */
-    private const val INJECTION_COMMAND =
-        "exec ./inject `pidof $SOTER_PACKAGE` libTEESimulator.so entry"
-
-    private const val REBIND_DELAY_MS = 1000L
-    private const val REBIND_MAX_MS = 30_000L
-
-    private val started = AtomicBoolean(false)
-
-    /** Re-bind backoff; doubles each failed (re)bind up to [REBIND_MAX_MS], resets on a clean mount. Handler-thread-confined. */
-    private var rebindDelay = REBIND_DELAY_MS
-
-    private lateinit var context: Context
-    private lateinit var handler: Handler
-
-    /** Delivers bind callbacks onto the supervisor thread so nothing touches the main looper. */
-    private val executor = Executor { command -> handler.post(command) }
-
-    /**
-     * Starts supervising on a dedicated thread and returns immediately. Idempotent. [context]
-     * must be able to bind services (the daemon's system context); supplied by the App wiring.
-     */
-    fun start(context: Context) {
-        if (!started.compareAndSet(false, true)) return
-        this.context = context
-        handler = Handler(HandlerThread("soter-supervisor").apply { start() }.looper)
-        handler.post { bind() }
-    }
-
-    private val connection =
-        object : ServiceConnection {
-            override fun onServiceConnected(name: ComponentName?, service: IBinder?) {
-                SystemLogger.debug("SOTER service connected; mounting forge")
-                service?.let(::mount)
-            }
-
-            override fun onServiceDisconnected(name: ComponentName?) {
-                SystemLogger.debug("SOTER service disconnected (process died); rebinding")
-                scheduleRetry()
-            }
-
-            override fun onBindingDied(name: ComponentName?) {
-                SystemLogger.debug("SOTER binding died; rebinding")
-                scheduleRetry()
-            }
-
-            override fun onNullBinding(name: ComponentName?) {
-                SystemLogger.debug("SOTER onBind returned null; rebinding")
-                scheduleRetry()
-            }
-        }
-
-    private fun bind() {
-        val intent = Intent(SoterServiceInterceptor.DESCRIPTOR).setPackage(SOTER_PACKAGE)
-        val bound =
-            runCatching {
-                    context.bindService(intent, Context.BIND_AUTO_CREATE, executor, connection)
-                }
-                .getOrElse {
-                    SystemLogger.debug { "SOTER bindService threw: $it" }
-                    false
-                }
-        if (bound) {
-            SystemLogger.debug("SOTER bind requested (on-demand poke)")
-        } else {
-            SystemLogger.debug("SOTER bindService returned false; retrying")
-            scheduleRetry()
-        }
-    }
-
-    private fun rebind() {
-        runCatching { context.unbindService(connection) }
-        bind()
-    }
-
-    /**
-     * Re-attempts the bind after the current backoff, then widens it (capped at [REBIND_MAX_MS]).
-     * Every path that fails to leave the forge mounted routes here, so a live-but-uninjected
-     * binding is re-attempted instead of stranding the forge. A clean [mount] resets the backoff.
-     */
-    private fun scheduleRetry() {
-        val delay = rebindDelay
-        rebindDelay = (rebindDelay * 2).coerceAtMost(REBIND_MAX_MS)
-        handler.postDelayed({ rebind() }, delay)
-    }
-
-    /** Confirms injection via the `0xdeadbeef` handshake, injecting first if absent, then registers. */
-    private fun mount(soterBinder: IBinder) {
-        var backdoor = BinderInterceptor.getBackdoor(soterBinder)
-        if (backdoor == null) {
-            SystemLogger.debug("SOTER backdoor absent; injecting libTEESimulator.so")
-            if (!injectLibrary()) {
-                SystemLogger.debug("SOTER injection failed; scheduling re-bind")
-                scheduleRetry()
-                return
-            }
-            backdoor = BinderInterceptor.getBackdoor(soterBinder)
-        }
-        if (backdoor == null) {
-            SystemLogger.debug("SOTER backdoor handshake failed after injection; scheduling re-bind")
-            scheduleRetry()
-            return
-        }
-        val registered =
-            BinderInterceptor.register(
-                backdoor,
-                soterBinder,
-                SoterServiceInterceptor,
-                SoterServiceInterceptor.interceptedCodes,
-            )
-        if (!registered) {
-            SystemLogger.debug("SOTER register failed; scheduling re-bind")
-            scheduleRetry()
-            return
-        }
-        rebindDelay = REBIND_DELAY_MS
-        SystemLogger.debug("SOTER forge mounted; handshake ok")
-    }
-
-    private fun injectLibrary(): Boolean =
-        runCatching {
-                Runtime.getRuntime().exec(arrayOf("/system/bin/sh", "-c", INJECTION_COMMAND)).waitFor() == 0
-            }
-            .getOrElse {
-                SystemLogger.debug { "SOTER inject exec failed: $it" }
-                false
-            }
-}
@@ -1,229 +0,0 @@
-package org.matrix.TEESimulator.interception.soter
-
-import android.os.IBinder
-import android.os.Parcel
-import android.util.Base64
-import java.nio.ByteBuffer
-import java.nio.ByteOrder
-import java.security.KeyPairGenerator
-import org.matrix.TEESimulator.interception.core.BinderInterceptor
-import org.matrix.TEESimulator.logging.SystemLogger
-
-/**
- * Forges healthy `com.tencent.soter.soterserver.ISoterService` (Layer A: AIDL over
- * `/dev/binder`) replies from inside the injected soterserver app process, so the SOTER
- * capability probe (春秋 / DuckDetector `SoterCapabilityProbe`) reads `available = true`
- * / `damaged = false` on a bootloader-unlocked device whose SOTER TA can no longer use
- * its factory ATTK. Replaces the external SoterFixer loop + the Hail freeze.
- *
- * Unconditional by design: the forge decision never consults `ConfigurationManager` /
- * `target.txt` (Phase 10 spec §Decision, gate G). It is mounted by the SOTER process
- * supervisor (10.B/10.W) against the ISoterService binder, so `onPreTransact` only sees
- * transactions on that binder — matching the raw transaction code is therefore enough.
- *
- * Diagnostics follow the module's standard three-layer capture (debug-gated, per-UID
- * NDJSON via [SystemLogger]; see `logging/SystemLogger.kt`): a `tx` line for every
- * transaction ([logTransaction]), the raw inbound request parcel, and the raw forged
- * reply wire. Capture is scoped to targeted UIDs (`isUidLogged`) exactly like the
- * keystore lane — it does NOT make the forge conditional; the forge still fires for all.
- *
- * Transaction codes are HARDCODED 1..13 in AIDL declaration order, NOT resolved via
- * [org.matrix.TEESimulator.interception.keystore.InterceptorUtils.getTransactCode]: the
- * shipped soterserver build is R8/ProGuard obfuscated — there is no `ISoterService$Stub`
- * class and no `TRANSACTION_*` fields (recon 2026-06-26, `a$a.smali` packed-switch). The
- * codes are fixed by Tencent's `ISoterService.aidl` and are obfuscation-independent.
- *
- * Scope boundary (10.A vs 10.M): the seven primitive-returning methods are fully forged
- * here. The six parcelable-returning methods emit the correct AIDL envelope + the
- * recon-verified `writeToParcel` field order; 10.M fills the payloads with
- * detector-satisfying values — a framed SOTER pubkey envelope the SDK's
- * `retrieveJsonFromExportedData` parses to a non-null `SoterPubKeyModel`, a non-zero sign
- * session, and a 256-byte signature.
- */
-object SoterServiceInterceptor : BinderInterceptor() {
-
-    /** The surviving, obfuscation-stable interface identifier (used by the 10.B/10.W mount). */
-    const val DESCRIPTOR = "com.tencent.soter.soterserver.ISoterService"
-
-    // AIDL transaction codes = FIRST_CALL_TRANSACTION (1) + declaration index, verified
-    // against the obfuscated `a$a.smali` packed-switch (recon 2026-06-26). NOTE the 5/6
-    // order: removeAuthKey precedes getAuthKey in the real .aidl (the spec prose had it
-    // reversed). Comments record each method's return shape.
-    private const val TX_GENERATE_APP_SECURE_KEY = 1 // int
-    private const val TX_GET_APP_SECURE_KEY = 2 // SoterExportResult
-    private const val TX_HAS_ASK_ALREADY = 3 // boolean
-    private const val TX_GENERATE_AUTH_KEY = 4 // int
-    private const val TX_REMOVE_AUTH_KEY = 5 // int   (NOT getAuthKey)
-    private const val TX_GET_AUTH_KEY = 6 // SoterExportResult   (NOT removeAuthKey)
-    private const val TX_REMOVE_ALL_AUTH_KEY = 7 // int
-    private const val TX_HAS_AUTH_KEY = 8 // boolean
-    private const val TX_INIT_SIGH = 9 // SoterSessionResult   (sic: Tencent's spelling)
-    private const val TX_FINISH_SIGN = 10 // SoterSignResult
-    private const val TX_GET_DEVICE_ID = 11 // SoterDeviceResult
-    private const val TX_GET_VERSION = 12 // int   (real service returns 1)
-    private const val TX_GET_EXTRA_PARAM = 13 // SoterExtraParam
-
-    /** SOTER success result code (`SoterCoreResult` ERR_OK). */
-    private const val SOTER_OK = 0
-
-    /** finishSign signature length the probe expects. */
-    private const val SIGNATURE_LEN = 256
-
-    /** `cpu_id` placeholder in the export envelope; the local probe never reads its value
-     *  (the backend pins the real per-`cpu_id` ATTK, which the forge cannot satisfy). */
-    private const val CPU_ID = "0000000000000000"
-
-    /** Code -> Tencent method name, for the `tx` diagnostic line. Names from the recon decompile. */
-    private val methodNames =
-        mapOf(
-            TX_GENERATE_APP_SECURE_KEY to "generateAppSecureKey",
-            TX_GET_APP_SECURE_KEY to "getAppSecureKey",
-            TX_HAS_ASK_ALREADY to "hasAskAlready",
-            TX_GENERATE_AUTH_KEY to "generateAuthKey",
-            TX_REMOVE_AUTH_KEY to "removeAuthKey",
-            TX_GET_AUTH_KEY to "getAuthKey",
-            TX_REMOVE_ALL_AUTH_KEY to "removeAllAuthKey",
-            TX_HAS_AUTH_KEY to "hasAuthKey",
-            TX_INIT_SIGH to "initSigh",
-            TX_FINISH_SIGN to "finishSign",
-            TX_GET_DEVICE_ID to "getDeviceId",
-            TX_GET_VERSION to "getVersion",
-            TX_GET_EXTRA_PARAM to "getExtraParam",
-        )
-
-    /** The codes this interceptor forges; consumed by the supervisor's registration (10.B/10.W). */
-    val interceptedCodes: IntArray = methodNames.keys.toIntArray()
-
-    /**
-     * Payload of [SoterExportResult.exportData] for getAppSecureKey (txn 2) and getAuthKey
-     * (txn 6). The detector's capability probe gates `damaged=false` on
-     * `SoterCore.getApp/AuthKeyModel() != null`, and the SDK's `retrieveJsonFromExportedData`
-     * (`SoterCoreBase`) returns a non-null `SoterPubKeyModel` only when this exact framing
-     * parses: `[4-byte LITTLE-ENDIAN json length][UTF-8 json][signature bytes]`. A
-     * non-empty-but-unframed blob throws inside the SDK and is read as `damaged` silently.
-     * The JSON parser swallows every exception, so only the framing is load-bearing; the
-     * `pub_key` is a genuine RSA-2048 SubjectPublicKeyInfo so a probe that base64/X.509-parses
-     * the field locally still succeeds. Lazily built — keygen runs once, off the mount path.
-     */
-    private val exportBlob: ByteArray by lazy { buildExportBlob() }
-
-    /** getDeviceId (txn 11) payload — well-formed, non-empty; the probe never parses it. */
-    private val deviceIdBlob = "TEESIM-SOTER-0001".toByteArray(Charsets.UTF_8)
-
-    /** finishSign (txn 10) signature payload — [SIGNATURE_LEN] bytes. */
-    private val signatureBlob = ByteArray(SIGNATURE_LEN)
-
-    private fun buildExportBlob(): ByteArray {
-        val pubKey =
-            runCatching {
-                    val generator = KeyPairGenerator.getInstance("RSA").apply { initialize(2048) }
-                    Base64.encodeToString(generator.generateKeyPair().public.encoded, Base64.NO_WRAP)
-                }
-                .getOrDefault("")
-        val json =
-            """{"pub_key":"$pubKey","counter":0,"cpu_id":"$CPU_ID","uid":0}"""
-                .toByteArray(Charsets.UTF_8)
-        val lengthPrefix = ByteBuffer.allocate(4).order(ByteOrder.LITTLE_ENDIAN).putInt(json.size).array()
-        return lengthPrefix + json + signatureBlob
-    }
-
-    override fun onPreTransact(
-        txId: Long,
-        target: IBinder,
-        code: Int,
-        flags: Int,
-        callingUid: Int,
-        callingPid: Int,
-        data: Parcel,
-    ): TransactionResult {
-        val method = methodNames[code]
-        if (method == null) {
-            // Not an ISoterService method we forge — record it as observed, then pass through.
-            logTransaction(txId, "code=$code", callingUid, callingPid, skipPost = true)
-            return TransactionResult.ContinueAndSkipPost
-        }
-        logTransaction(txId, method, callingUid, callingPid)
-        captureRequest(callingUid, txId, method, data)
-
-        return when (code) {
-            // Primitive returns — fully forged here.
-            TX_GENERATE_APP_SECURE_KEY,
-            TX_GENERATE_AUTH_KEY,
-            TX_REMOVE_AUTH_KEY,
-            TX_REMOVE_ALL_AUTH_KEY -> forgedReply(callingUid, txId, method) { writeInt(SOTER_OK) }
-            TX_GET_VERSION -> forgedReply(callingUid, txId, method) { writeInt(1) }
-            TX_HAS_ASK_ALREADY,
-            TX_HAS_AUTH_KEY -> forgedReply(callingUid, txId, method) { writeInt(1) } // boolean true
-
-            // Parcelable returns — correct envelope + recon field order, payloads filled (10.M).
-            TX_GET_APP_SECURE_KEY,
-            TX_GET_AUTH_KEY ->
-                forgedReply(callingUid, txId, method) {
-                    writeInt(1) // non-null marker
-                    writeInt(SOTER_OK) // resultCode
-                    writeByteArray(exportBlob) // exportData — framed SOTER pubkey envelope
-                    writeInt(exportBlob.size) // exportDataLength
-                }
-            TX_INIT_SIGH ->
-                forgedReply(callingUid, txId, method) {
-                    writeInt(1)
-                    writeLong(1L) // session — any non-zero satisfies the probe
-                    writeInt(SOTER_OK) // resultCode — probe requires == 0 (SoterCapabilityProbe.kt:107)
-                }
-            TX_FINISH_SIGN ->
-                forgedReply(callingUid, txId, method) {
-                    writeInt(1)
-                    writeInt(SOTER_OK) // resultCode — finishSign throws on != 0
-                    writeByteArray(signatureBlob) // exportData = signature
-                    writeInt(signatureBlob.size) // exportDataLength
-                }
-            TX_GET_DEVICE_ID ->
-                forgedReply(callingUid, txId, method) {
-                    writeInt(1)
-                    writeInt(SOTER_OK) // resultCode
-                    writeByteArray(deviceIdBlob) // exportData = device id
-                    writeInt(deviceIdBlob.size) // exportDataLength
-                }
-            TX_GET_EXTRA_PARAM ->
-                forgedReply(callingUid, txId, method) {
-                    writeInt(1)
-                    writeValue("optical") // SoterExtraParam.result = fingerprint sensor type
-                }
-
-            // Unreachable: method != null means code is one of the 13 above.
-            else -> TransactionResult.ContinueAndSkipPost
-        }
-    }
-
-    /** Snapshots the inbound request parcel to the per-UID NDJSON plane (debug + targeted only). */
-    private fun captureRequest(uid: Int, txId: Long, method: String, data: Parcel) {
-        if (!SystemLogger.isUidLogged(uid)) return
-        runCatching { data.marshall() }
-            .onSuccess { raw ->
-                SystemLogger.uidLogRaw(uid, txId, "$method-request", "len=${raw.size}", raw)
-            }
-    }
-
-    /**
-     * Builds an AIDL reply (`writeNoException()` then [body]) and snapshots its wire bytes to the
-     * per-UID NDJSON plane before handing it to the native hook. Parcelable bodies write their own
-     * `writeInt(1)` non-null marker; the native hook recycles the parcel after use.
-     */
-    private fun forgedReply(
-        uid: Int,
-        txId: Long,
-        method: String,
-        body: Parcel.() -> Unit,
-    ): TransactionResult.OverrideReply {
-        val reply = Parcel.obtain()
-        reply.writeNoException()
-        reply.body()
-        if (SystemLogger.isUidLogged(uid)) {
-            runCatching { reply.marshall() }
-                .onSuccess { raw ->
-                    SystemLogger.uidLogRaw(uid, txId, "$method-reply", "len=${raw.size}", raw)
-                }
-        }
-        return TransactionResult.OverrideReply(reply)
-    }
-}
@@ -1,74 +0,0 @@
-package org.matrix.TEESimulator.logging
-
-import android.hardware.security.keymint.Tag
-import android.system.keystore2.Authorization
-import java.security.cert.Certificate
-import java.security.cert.X509Certificate
-import org.matrix.TEESimulator.attestation.AttestationPatcher
-import org.matrix.TEESimulator.config.ConfigurationManager
-import org.matrix.TEESimulator.util.AndroidDeviceUtils
-
-/**
- * Assembles the per-UID "attestation dossier": for a targeted app, the full decoded attestation we
- * actually hand it, the identity of every certificate in the returned chain, and the source of each
- * device value that fed that attestation. Emitting all three where a chain is produced turns "the
- * app rejects us" into a field-by-field record that can be diffed against a genuine TEE.
- */
-object AttestationDossier {
-
-    /**
-     * Records the dossier for [chain] under [uid], tagged with the [path] that produced it
-     * (`FORGE-rust`, `FORGE-bouncycastle`, or `PATCH`). No-op for untargeted UIDs and release
-     * builds; the expensive decoding is skipped entirely when the UID is out of scope.
-     */
-    fun log(uid: Int, txId: Long, path: String, chain: List<Certificate>) {
-        if (!SystemLogger.isUidLogged(uid)) return
-        val leaf = chain.firstOrNull() as? X509Certificate
-        val extension =
-            leaf?.let { AttestationPatcher.formatAttestationExtension(it) }
-                ?: "<no attestation extension>"
-        SystemLogger.uidLog(uid, txId, "attest", "path=$path depth=${chain.size} $extension")
-        SystemLogger.uidLog(uid, txId, "keybox", "file=${ConfigurationManager.getKeyboxFileForUid(uid)}")
-        SystemLogger.uidLog(uid, txId, "chain", AttestationPatcher.formatCertChain(chain))
-        SystemLogger.uidLog(uid, txId, "chain-verify", AttestationPatcher.formatChainVerification(chain))
-        SystemLogger.uidLog(uid, txId, "props", AndroidDeviceUtils.describeSources(uid))
-    }
-
-    /**
-     * Records the *shape* of the emitted authorization list — count, ordered tags, and per-auth
-     * securityLevel. This is the exact surface the duck detector's generate-mode parcel fingerprint
-     * stride-walks, so logging it readably lets a "fingerprint" detection be compared against the
-     * known genuine-TEE shape without decoding the marshalled reply offline.
-     */
-    fun logAuthShape(uid: Int, txId: Long, authorizations: Array<Authorization>?) {
-        if (!SystemLogger.isUidLogged(uid)) return
-        val auths = authorizations ?: return
-        val shape = auths.joinToString(",") { "${tagName(it.keyParameter.tag)}/${it.securityLevel}" }
-        SystemLogger.uidLog(uid, txId, "auth-shape", "n=${auths.size} [$shape]")
-    }
-
-    /** Names the authorization tags that occur in generate-mode replies; others render as numbers. */
-    private fun tagName(tag: Int): String =
-        when (tag) {
-            Tag.PURPOSE -> "PURPOSE"
-            Tag.ALGORITHM -> "ALGORITHM"
-            Tag.KEY_SIZE -> "KEY_SIZE"
-            Tag.DIGEST -> "DIGEST"
-            Tag.PADDING -> "PADDING"
-            Tag.EC_CURVE -> "EC_CURVE"
-            Tag.RSA_PUBLIC_EXPONENT -> "RSA_PUBLIC_EXPONENT"
-            Tag.NO_AUTH_REQUIRED -> "NO_AUTH_REQUIRED"
-            Tag.ORIGIN -> "ORIGIN"
-            Tag.OS_VERSION -> "OS_VERSION"
-            Tag.OS_PATCHLEVEL -> "OS_PATCHLEVEL"
-            Tag.VENDOR_PATCHLEVEL -> "VENDOR_PATCHLEVEL"
-            Tag.BOOT_PATCHLEVEL -> "BOOT_PATCHLEVEL"
-            Tag.CREATION_DATETIME -> "CREATION_DATETIME"
-            Tag.ROOT_OF_TRUST -> "ROOT_OF_TRUST"
-            Tag.USER_ID -> "USER_ID"
-            Tag.USAGE_COUNT_LIMIT -> "USAGE_COUNT_LIMIT"
-            Tag.UNLOCKED_DEVICE_REQUIRED -> "UNLOCKED_DEVICE_REQUIRED"
-            Tag.ACTIVE_DATETIME -> "ACTIVE_DATETIME"
-            else -> "tag${tag and 0x0FFFFFFF}"
-        }
-}
@@ -69,23 +69,12 @@ object KeyMintParameterLogger {
            .associate { field -> (field.get(null) as Int) to field.name }
    }

-    /** Logs a single KeyParameter to the shared debug stream (used for un-scoped param dumps). */
-    fun logParameter(param: KeyParameter) {
-        SystemLogger.debug("KeyParam: ${describe(param)}")
-    }
-
-    /** Logs a single KeyParameter onto a targeted UID's diagnostic plane as a `param` record. */
-    fun logParameter(uid: Int, txId: Long, param: KeyParameter) {
-        SystemLogger.uidLog(uid, txId, "param", describe(param))
-    }
-
    /**
-     * Formats a single KeyParameter into a readable `tag | Value` string. Shared by both
-     * [logParameter] overloads so the two logging planes render parameters identically.
+     * Logs a single KeyParameter in a formatted, readable way.
     *
-     * @param param The KeyParameter to format.
+     * @param param The KeyParameter to log.
     */
-    private fun describe(param: KeyParameter): String {
+    fun logParameter(param: KeyParameter) {
        val tagName = tagNames[param.tag] ?: "UNKNOWN_TAG"
        val value = param.value
        val formattedValue: String =
@@ -121,7 +110,7 @@ object KeyMintParameterLogger {
                else -> "<raw>"
            } ?: "Unknown Value"

-        return "%-25s | Value: %s".format(tagName, formattedValue)
+        SystemLogger.debug("KeyParam: %-25s | Value: %s".format(tagName, formattedValue))
    }

    private fun ByteArray.toReadableString(): String {
@@ -1,19 +1,9 @@
 package org.matrix.TEESimulator.logging

-import android.util.Base64
 import android.util.Log
-import java.io.BufferedWriter
-import java.io.File
-import java.io.FileWriter
-import java.time.Instant
-import java.time.ZoneId
-import java.time.format.DateTimeFormatter
-import java.util.concurrent.ConcurrentHashMap
 import java.util.concurrent.atomic.AtomicInteger
 import java.util.concurrent.atomic.AtomicLong
-import org.json.JSONObject
 import org.matrix.TEESimulator.BuildConfig
-import org.matrix.TEESimulator.config.ConfigurationManager

 /**
 * A centralized logging utility for the TEESimulator application. This object provides a consistent
@@ -36,11 +26,10 @@ object SystemLogger {
    private val suppressedCount = AtomicInteger(0)

    /**
-     * Returns true if this message should be emitted. Resets the window if expired and emits a
-     * suppression summary for the previous window.
+     * Returns true if this message should be emitted. Resets the window if expired
+     * and emits a suppression summary for the previous window.
     */
-    @PublishedApi
-    internal fun acquireLogPermit(): Boolean {
+    @PublishedApi internal fun acquireLogPermit(): Boolean {
        val now = System.currentTimeMillis()
        val start = windowStart.get()
        if (now - start > RATE_LIMIT_WINDOW_MS) {
@@ -49,10 +38,7 @@ object SystemLogger {
                val suppressed = suppressedCount.getAndSet(0)
                windowCount.set(1) // this call counts as #1 in the new window
                if (suppressed > 0) {
-                    Log.i(
-                        TAG,
-                        "[rate-limit] suppressed $suppressed log messages in previous window",
-                    )
+                    Log.i(TAG, "[rate-limit] suppressed $suppressed log messages in previous window")
                }
                return true
            }
@@ -63,7 +49,9 @@ object SystemLogger {
        return false
    }

-    /** Logs a debug message. Use this for fine-grained information that is useful for debugging. */
+    /**
+     * Logs a debug message. Use this for fine-grained information that is useful for debugging.
+     */
    fun debug(message: String) {
        if (!isDebugBuild) return
        if (!acquireLogPermit()) return
@@ -77,7 +65,9 @@ object SystemLogger {
        Log.d(TAG, message())
    }

-    /** Logs an informational message. Use this to report major application lifecycle events. */
+    /**
+     * Logs an informational message. Use this to report major application lifecycle events.
+     */
    fun info(message: String) {
        if (!acquireLogPermit()) return
        Log.i(TAG, message)
@@ -89,7 +79,9 @@ object SystemLogger {
        Log.i(TAG, message())
    }

-    /** Logs a warning message. Warnings are never rate-limited. */
+    /**
+     * Logs a warning message. Warnings are never rate-limited.
+     */
    fun warning(message: String, throwable: Throwable? = null) {
        if (throwable != null) {
            Log.w(TAG, message, throwable)
@@ -98,7 +90,9 @@ object SystemLogger {
        }
    }

-    /** Logs an error message. Errors are never rate-limited. */
+    /**
+     * Logs an error message. Errors are never rate-limited.
+     */
    fun error(message: String, throwable: Throwable? = null) {
        if (throwable != null) {
            Log.e(TAG, message, throwable)
@@ -128,145 +122,4 @@ object SystemLogger {
        if (!isDebugBuild) return
        Log.w(TAG, message())
    }
-
-    // --- UID-keyed diagnostic plane (debug builds only) -------------------------------------
-
-    /**
-     * True when [uid] should receive deep, per-UID diagnostic logging: a debug build AND the UID is
-     * targeted in `target.txt`. This is the single scope gate for the diagnostic plane; it reuses
-     * the existing activation set, so no new configuration surface is introduced.
-     */
-    fun isUidLogged(uid: Int): Boolean = isDebugBuild && !ConfigurationManager.shouldSkipUid(uid)
-
-    /** Resolves a UID to its primary package name for log labelling, falling back to `uid:N`. */
-    private fun label(uid: Int): String =
-        ConfigurationManager.getPackagesForUid(uid).firstOrNull() ?: "uid:$uid"
-
-    /**
-     * Emits one structured diagnostic record for a targeted [uid]. The human form
-     * `[<pkg> tx=<txId>] <event>: <detail>` goes to logcat; the file sink receives one NDJSON object
-     * per line under that UID's own file. In-scope records bypass the global rate limiter: a
-     * targeted app's traffic is already volume-bounded, and dropping a line mid-probe would corrupt
-     * the very trace we are trying to read. No-op for untargeted UIDs and in release builds.
-     */
-    fun uidLog(uid: Int, txId: Long?, event: String, detail: String) {
-        if (!isUidLogged(uid)) return
-        val correlation = txId?.let { " tx=$it" } ?: ""
-        Log.d(TAG, "[${label(uid)}$correlation] $event: $detail")
-        runCatching { uidWriter(uid).append(jsonRecord(uid, txId, event, detail, null)) }
-    }
-
-    /** Lazy [uidLog]: [detail] is only built for targeted UIDs in debug builds. */
-    inline fun uidLog(uid: Int, txId: Long?, event: String, detail: () -> String) {
-        if (!isUidLogged(uid)) return
-        uidLog(uid, txId, event, detail())
-    }
-
-    /**
-     * [uidLog] plus the exact wire bytes that produced the event, base64 (NO_WRAP) in a `raw_b64`
-     * field. This is the structured replacement for the per-call `.bin` parcel dumps: one NDJSON
-     * line on the per-UID file instead of a fresh undecodable file per transaction, with the raw
-     * parcel still recoverable for offline parsers.
-     */
-    fun uidLogRaw(uid: Int, txId: Long?, event: String, detail: String, raw: ByteArray) {
-        if (!isUidLogged(uid)) return
-        val correlation = txId?.let { " tx=$it" } ?: ""
-        Log.d(TAG, "[${label(uid)}$correlation] $event: $detail (raw ${raw.size}B)")
-        runCatching {
-            val encoded = Base64.encodeToString(raw, Base64.NO_WRAP)
-            uidWriter(uid).append(jsonRecord(uid, txId, event, detail, encoded))
-        }
-    }
-
-    /**
-     * External-storage root for every debug diagnostic. `/data/media/0/TEESimulator` is the
-     * in-namespace backing path the keystore domain can reach; a normal file manager sees the same
-     * files at `/sdcard/TEESimulator`. Release builds never write here and purge it on boot
-     * (App.purgeDebugDiagnostics). The domain reaches it via a debug-only media_rw_data_file
-     * sepolicy grant, and service.sh pre-creates the directory.
-     */
-    const val DIAGNOSTIC_DIR = "/data/media/0/TEESimulator"
-
-    private val uidLogDir = File(DIAGNOSTIC_DIR)
-    private const val UID_LOG_MAX_BYTES = 4L * 1024 * 1024
-    private val uidWriters = ConcurrentHashMap<Int, UidLogFile>()
-
-    private val recordClock =
-        DateTimeFormatter.ofPattern("MM-dd HH:mm:ss.SSS").withZone(ZoneId.systemDefault())
-
-    private fun jsonRecord(
-        uid: Int,
-        txId: Long?,
-        event: String,
-        detail: String,
-        rawB64: String?,
-    ): String =
-        JSONObject()
-            .apply {
-                put("ts", recordClock.format(Instant.now()))
-                put("uid", uid)
-                put("pkg", label(uid))
-                txId?.let { put("tx", it) }
-                put("event", event)
-                put("detail", detail)
-                rawB64?.let { put("raw_b64", it) }
-            }
-            .toString()
-
-    private fun uidWriter(uid: Int): UidLogFile =
-        uidWriters.computeIfAbsent(uid) { key ->
-            UidLogFile(key, uidLogDir).also { file ->
-                val packages =
-                    ConfigurationManager.getPackagesForUid(key).joinToString().ifEmpty { "<unresolved>" }
-                runCatching {
-                    file.append(jsonRecord(key, null, "session", "packages=[$packages]", null))
-                }
-            }
-        }
-
-    /**
-     * Append-only NDJSON sink for a single UID at `<logDir>/teesim-uid-<uid>.ndjson`, rotated once
-     * to `.ndjson.1` at [UID_LOG_MAX_BYTES]; one JSON object per line. Writes are synchronised
-     * because the keystore binder pool is multi-threaded, and every operation is wrapped so a
-     * logging fault can never propagate into the daemon. Created only on the debug-gated path.
-     */
-    private class UidLogFile(uid: Int, private val logDir: File) {
-        private val primary = File(logDir, "teesim-uid-$uid.ndjson")
-        private val rotated = File(logDir, "teesim-uid-$uid.ndjson.1")
-        private var writer: BufferedWriter? = null
-        private var size = 0L
-
-        @Synchronized
-        fun append(jsonLine: String) {
-            runCatching {
-                val out = writer ?: open()
-                out.write(jsonLine)
-                out.write("\n")
-                out.flush()
-                size += jsonLine.length + 1
-                if (size >= UID_LOG_MAX_BYTES) rotate()
-            }
-        }
-
-        private fun open(): BufferedWriter {
-            logDir.mkdirs()
-            val out = BufferedWriter(FileWriter(primary, /* append = */ true))
-            writer = out
-            size = primary.length()
-            return out
-        }
-
-        private fun rotate() {
-            runCatching {
-                writer?.flush()
-                writer?.close()
-            }
-            writer = null
-            runCatching {
-                if (rotated.exists()) rotated.delete()
-                primary.renameTo(rotated)
-            }
-            size = 0L
-        }
-    }
 }
@@ -95,35 +95,19 @@ object CertificateGenerator {
        return try {
                // AOSP ta/src/keys.rs:451-478: no challenge + no attestKey = self-signed, depth 1
                if (challenge == null && attestKeyAlias == null) {
-                SystemLogger.trace {
-                    "[certgen] no-challenge key: self-signed, depth=1, purposes=${params.purpose}"
-                }
+                    SystemLogger.trace { "[certgen] no-challenge key: self-signed, depth=1, purposes=${params.purpose}" }
                    return listOf(buildSelfSignedCertificate(subjectKeyPair, params))
                }

                val keybox = getKeyboxForAlgorithm(uid, params.algorithm)

-            val wantsAttestKey =
-                attestKeyAlias != null && Build.VERSION.SDK_INT >= Build.VERSION_CODES.S
                val attestKeyInfo =
-                if (wantsAttestKey) getAttestationKeyInfo(uid, attestKeyAlias) else null
+                    if (attestKeyAlias != null && Build.VERSION.SDK_INT >= Build.VERSION_CODES.S) {
+                        getAttestationKeyInfo(uid, attestKeyAlias)
+                    } else null

-            // When the caller designates an attest key, the leaf MUST be signed by it and returned
-            // alone (the caller appends the attest key's own chain). Re-rooting under the keybox
-            // here instead yields a self-rooted leaf that, concatenated with the attest key chain,
-            // double-roots and fails verification (WRONG_PUBLIC_KEY_TYPE). Refuse rather than emit
-            // a
-            // broken chain.
-            if (wantsAttestKey && attestKeyInfo == null) {
-                SystemLogger.error(
-                    "Designated attest key '$attestKeyAlias' not found for uid $uid; refusing to " +
-                        "emit a keybox-rooted leaf that would break the caller's chain."
-                )
-                return null
-            }
-
-            val (signingKey, issuer) =
-                attestKeyInfo?.let { it.first to it.second }
+                val (signingKey, issuer) = attestKeyInfo
+                    ?.let { it.first to it.second }
                    ?: (keybox.keyPair to getIssuerFromKeybox(keybox))

                val leafCert =
@@ -154,7 +138,9 @@ object CertificateGenerator {
        securityLevel: Int,
    ): Pair<KeyPair, List<Certificate>>? {
        return try {
-            SystemLogger.info("Generating new attested key pair for alias: '$alias' (UID: $uid)")
+                SystemLogger.info(
+                    "Generating new attested key pair for alias: '$alias' (UID: $uid)"
+                )
                val newKeyPair =
                    generateSoftwareKeyPair(params)
                        ?: throw Exception("Failed to generate underlying software key pair.")
@@ -163,7 +149,9 @@ object CertificateGenerator {
                    generateCertificateChain(uid, newKeyPair, attestKeyAlias, params, securityLevel)
                        ?: throw Exception("Failed to generate certificate chain for new key pair.")

-            SystemLogger.info("Successfully generated new certificate chain for alias: '$alias'.")
+                SystemLogger.info(
+                    "Successfully generated new certificate chain for alias: '$alias'."
+                )
                Pair(newKeyPair, chain)
            } catch (e: android.os.ServiceSpecificException) {
                throw e
@@ -184,28 +172,11 @@ object CertificateGenerator {
                Algorithm.RSA -> "RSA"
                else -> throw IllegalArgumentException("Unsupported algorithm ID: $algorithm")
            }
-        // Prefer the algorithm-matching keybox, but fall back to any usable key (EC preferred) when
-        // none exists. An EC attestation key validly ECDSA-signs a leaf carrying an RSA subject key,
-        // so an EC-only keybox can still root an RSA forge. Without this fallback an RSA ATTEST_KEY
-        // request on an EC-only keybox throws -75 and the caller's chain never roots ("unknown
-        // certificate"). Mirrors the patch path's fail-safe
-        // (AttestationPatcher.getKeyboxForUidAndAlgorithm) and the RSA-leaf-under-EC-keybox handling
-        // in commit e6d5e4d.
-        val matched = KeyBoxManager.getAttestationKey(keyboxFile, algorithmName)
-        val keybox =
-            matched
-                ?: KeyBoxManager.getAnyAttestationKey(keyboxFile)
+        return KeyBoxManager.getAttestationKey(keyboxFile, algorithmName)
            ?: throw android.os.ServiceSpecificException(
                -75, // ATTESTATION_KEYS_NOT_PROVISIONED
-                    "No usable attestation key in $keyboxFile",
+                "No attestation key for algorithm $algorithmName in $keyboxFile",
            )
-        // Surface which keybox actually signs the forge, so an EC-only-keybox fallback (an RSA leaf
-        // rooted under the EC key) is visible on the per-UID plane instead of silent.
-        SystemLogger.uidLog(uid, null, "keybox-pick") {
-            "req=$algorithmName ${if (matched != null) "matched" else "fellback-to-any"} " +
-                "signer=${getIssuerFromKeybox(keybox)}"
-        }
-        return keybox
    }

    /** Retrieves the key pair and issuer name for a given attestation key alias. */
@@ -218,15 +189,6 @@ object CertificateGenerator {
            val certChain = CertificateHelper.getCertificateChain(keyInfo.response)
            if (!certChain.isNullOrEmpty()) {
                val issuer = X509CertificateHolder(certChain[0].encoded).subject
-                // The leaf is signed by keyInfo.keyPair, but the caller verifies it against the
-                // public key of the chain getCertChain(attestKeyAlias) serves. A two-rooted EC chain
-                // (DATA_TOO_LARGE_FOR_MODULUS) is exactly those two disagreeing on algorithm; log
-                // both at the signing instant so an EC attest-key run pins the mismatched edge.
-                SystemLogger.uidLog(uid, null, "attest-sign") {
-                    "alias=$attestKeyAlias signerKey=${keyInfo.keyPair?.public?.algorithm} " +
-                        "servedLeafKey=${certChain[0].publicKey.algorithm} " +
-                        "depth=${certChain.size} issuer=$issuer"
-                }
                Pair(keyInfo.keyPair, issuer)
            } else {
                null
@@ -243,9 +205,7 @@ object CertificateGenerator {
    private fun buildKeyUsageFromPurposes(purposes: List<Int>): Int {
        var bits = 0
        for (purpose in purposes) {
-            bits =
-                bits or
-                    when (purpose) {
+            bits = bits or when (purpose) {
                KeyPurpose.SIGN -> KeyUsage.digitalSignature
                KeyPurpose.DECRYPT -> KeyUsage.dataEncipherment
                KeyPurpose.WRAP_KEY -> KeyUsage.keyEncipherment
@@ -293,13 +253,9 @@ object CertificateGenerator {

        val signerAlgorithm =
            when (signingKeyPair.private.algorithm) {
-                "EC",
-                "ECDSA" -> "SHA256withECDSA"
+                "EC", "ECDSA" -> "SHA256withECDSA"
                "RSA" -> "SHA256withRSA"
-                else ->
-                    throw IllegalArgumentException(
-                        "Unsupported signing key: ${signingKeyPair.private.algorithm}"
-                    )
+                else -> throw IllegalArgumentException("Unsupported signing key: ${signingKeyPair.private.algorithm}")
            }
        val contentSigner =
            JcaContentSignerBuilder(signerAlgorithm)
@@ -318,8 +274,7 @@ object CertificateGenerator {
        val notBefore = params.certificateNotBefore ?: Date(0)
        val notAfter = params.certificateNotAfter ?: Date(UNDEFINED_NOT_AFTER)

-        val builder =
-            JcaX509v3CertificateBuilder(
+        val builder = JcaX509v3CertificateBuilder(
            subject,
            params.certificateSerial ?: BigInteger.ONE,
            notBefore,
@@ -333,16 +288,12 @@ object CertificateGenerator {
            builder.addExtension(Extension.keyUsage, true, KeyUsage(keyUsageBits))
        }

-        val signerAlgorithm =
-            when (keyPair.private.algorithm) {
-                "EC",
-                "ECDSA" -> "SHA256withECDSA"
+        val signerAlgorithm = when (keyPair.private.algorithm) {
+            "EC", "ECDSA" -> "SHA256withECDSA"
            "RSA" -> "SHA256withRSA"
-                else ->
-                    throw IllegalArgumentException("Unsupported key: ${keyPair.private.algorithm}")
+            else -> throw IllegalArgumentException("Unsupported key: ${keyPair.private.algorithm}")
        }
-        val contentSigner =
-            JcaContentSignerBuilder(signerAlgorithm)
+        val contentSigner = JcaContentSignerBuilder(signerAlgorithm)
            .setProvider(BouncyCastleProvider.PROVIDER_NAME)
            .build(keyPair.private)

@@ -3,7 +3,6 @@ package org.matrix.TEESimulator.pki
 import android.security.keystore.KeyProperties
 import java.io.File
 import java.io.StringReader
-import java.security.cert.X509Certificate
 import java.security.interfaces.ECPrivateKey
 import java.security.interfaces.RSAPrivateKey
 import java.util.concurrent.ConcurrentHashMap
@@ -54,38 +53,10 @@ object KeyBoxManager {
        // If it's not in the cache, the `getOrPut` block is executed to parse and store it.
        val keyMap =
            keyStoreCache.getOrPut(keyStoreFileName) { parseKeyStoreFile(keyStoreFileName) }
-        val keyBox = keyMap[algorithm]
-        if (keyBox != null) {
-            // Surface attestation cert serials on every fetch so a revoked/leaked keybox is
-            // obvious from logcat alone -- Google's CRL and Duck's "mass abuse" check both match
-            // by certificate serial (lowercase hex). Logged here rather than at parse time because
-            // the parse is cached and would emit at most once per boot.
-            val serials =
-                keyBox.certificates.joinToString(", ") { cert ->
-                    (cert as? X509Certificate)?.serialNumber?.toString(16) ?: "?"
-                }
-            SystemLogger.info(
-                "Using $algorithm keybox $keyStoreFileName; attestation cert serials (hex): $serials"
+        SystemLogger.verbose(
+            "Fetching attestation key in $keyStoreFileName with $algorithm algorithm."
        )
-        }
-        return keyBox
-    }
-
-    /**
-     * Retrieves any usable attestation key from a key store file, preferring EC.
-     *
-     * EC is the modern device-attestation key type and validly signs a leaf carrying either an EC
-     * or an RSA subject key. This is the fail-safe used when no algorithm-matching key exists, so
-     * patching can still re-root the chain under the keybox instead of leaking the device's real
-     * attestation.
-     *
-     * @param keyStoreFileName The name of the XML file (e.g., "keybox.xml").
-     * @return The preferred [KeyBox], or `null` if the file contains no usable key.
-     */
-    fun getAnyAttestationKey(keyStoreFileName: String): KeyBox? {
-        val keyMap =
-            keyStoreCache.getOrPut(keyStoreFileName) { parseKeyStoreFile(keyStoreFileName) }
-        return keyMap[KeyProperties.KEY_ALGORITHM_EC] ?: keyMap.values.firstOrNull()
+        return keyMap[algorithm]
    }

    /**
@@ -52,10 +52,6 @@ data class CertGenConfig(
    val callerNonce: Boolean = false,
    val unlockedDeviceRequired: Boolean = false,
    val noAuthRequired: Boolean = true,
-    // Diagnostic plane: the calling app UID keys the native log lines, and debugLogging mirrors the
-    // APK debug variant so the native extension dump is silent in release.
-    val uid: Int,
-    val debugLogging: Boolean,
 )

 object NativeCertGen {
@@ -73,10 +69,7 @@ object NativeCertGen {
            isAvailable = true
            SystemLogger.info("NativeCertGen: loaded libcertgen.so successfully")
        } catch (e: UnsatisfiedLinkError) {
-            SystemLogger.error(
-                "NativeCertGen: failed to load libcertgen.so, falling back to BouncyCastle",
-                e,
-            )
+            SystemLogger.error("NativeCertGen: failed to load libcertgen.so, falling back to BouncyCastle", e)
        }
    }

@@ -84,6 +77,10 @@ object NativeCertGen {

    private external fun initLogging(verbose: Boolean, logDir: String): Boolean

+    private external fun dumpLogs(): String?
+
+    fun dump(): String? = if (isAvailable) dumpLogs() else null
+
    fun parseNativeResult(bytes: ByteArray): Pair<KeyPair, List<Certificate>> {
        val buf = ByteBuffer.wrap(bytes).order(ByteOrder.BIG_ENDIAN)

@@ -114,10 +111,8 @@ object NativeCertGen {
            throw IllegalStateException("No certificates in native result")
        }

-        val algorithmName =
-            when (certs[0].publicKey.algorithm) {
-                "EC",
-                "ECDSA" -> "EC"
+        val algorithmName = when (certs[0].publicKey.algorithm) {
+            "EC", "ECDSA" -> "EC"
            "RSA" -> "RSA"
            else -> certs[0].publicKey.algorithm
        }
@@ -8,16 +8,13 @@ import java.io.FileInputStream
 import java.security.MessageDigest
 import java.time.LocalDate
 import java.util.concurrent.ThreadLocalRandom
-import javax.xml.parsers.DocumentBuilderFactory
 import org.bouncycastle.asn1.ASN1EncodableVector
 import org.bouncycastle.asn1.ASN1Integer
 import org.bouncycastle.asn1.DEROctetString
 import org.bouncycastle.asn1.DERSequence
 import org.matrix.TEESimulator.attestation.DeviceAttestationService
-import org.matrix.TEESimulator.config.BootStateManager
 import org.matrix.TEESimulator.config.ConfigurationManager
 import org.matrix.TEESimulator.logging.SystemLogger
-import org.w3c.dom.Element

 /**
 * Provides utility functions for accessing Android system properties and device-specific
@@ -46,7 +43,6 @@ object AndroidDeviceUtils {
                DeviceAttestationService.CachedAttestationData?.verifiedBootKey
            },
            expectedSize = 32,
-            recordSource = { bootKeySource = it },
        )
    }

@@ -64,16 +60,9 @@ object AndroidDeviceUtils {
                DeviceAttestationService.CachedAttestationData?.verifiedBootHash
            },
            expectedSize = 32,
-            recordSource = { bootHashSource = it },
        )
    }

-    // Records which fallback tier supplied bootKey/bootHash so the diagnostic dossier can flag a
-    // random-fallback value — a real verifiedBootKey that resolves to random bytes is a textbook
-    // simulated-TEE tell. Populated by initializeBootProperty on first access.
-    @Volatile private var bootKeySource: String = "uninitialized"
-    @Volatile private var bootHashSource: String = "uninitialized"
-
    /**
     * Public function to explicitly trigger the initialization of the boot key and hash. Accessing
     * these properties here ensures they are set up before they might be needed elsewhere.
@@ -100,11 +89,9 @@ object AndroidDeviceUtils {
        propertyName: String,
        attestationValueProvider: () -> ByteArray?,
        expectedSize: Int,
-        recordSource: (String) -> Unit,
    ): ByteArray {
        getProperty(propertyName, expectedSize)?.let {
            SystemLogger.debug("Using $propertyName from system property: ${it.toHex()}")
-            recordSource("system-prop")
            persistToFile(propertyName, it)
            return it
        }
@@ -112,8 +99,7 @@ object AndroidDeviceUtils {
        try {
            attestationValueProvider()?.let {
                SystemLogger.debug("Using $propertyName from TEE attestation: ${it.toHex()}")
-                recordSource("tee-attestation")
-                setBootProperty(propertyName, it)
+                setProperty(propertyName, it)
                persistToFile(propertyName, it)
                return it
            }
@@ -123,15 +109,13 @@ object AndroidDeviceUtils {

        readFromFile(propertyName, expectedSize)?.let {
            SystemLogger.debug("Using $propertyName from persistent file: ${it.toHex()}")
-            recordSource("persistent-file")
-            setBootProperty(propertyName, it)
+            setProperty(propertyName, it)
            return it
        }

        return generateRandomBytes(expectedSize).also {
            SystemLogger.debug("Using randomly generated $propertyName: ${it.toHex()}")
-            recordSource("random-fallback")
-            setBootProperty(propertyName, it)
+            setProperty(propertyName, it)
            persistToFile(propertyName, it)
        }
    }
@@ -179,14 +163,6 @@ object AndroidDeviceUtils {
        }
    }

-    private fun setBootProperty(name: String, bytes: ByteArray) {
-        if (!BootStateManager.shouldSpoofBootProps()) {
-            SystemLogger.info("Skipping system property '$name' because boot prop spoofing is disabled")
-            return
-        }
-        setProperty(name, bytes)
-    }
-
    internal fun setProperty(name: String, value: String) {
        try {
            SystemLogger.debug("Setting system property '$name' to: $value")
@@ -210,8 +186,7 @@ object AndroidDeviceUtils {

    private val PERSIST_DIR = File("/data/adb/tricky_store")

-    private fun fileForProperty(propertyName: String): File =
-        when (propertyName) {
+    private fun fileForProperty(propertyName: String): File = when (propertyName) {
        "ro.boot.vbmeta.digest" -> File(PERSIST_DIR, "boot_hash.bin")
        "ro.boot.vbmeta.public_key_digest" -> File(PERSIST_DIR, "boot_key.bin")
        else -> File(PERSIST_DIR, "${propertyName.replace('.', '_')}.bin")
@@ -254,23 +229,6 @@ object AndroidDeviceUtils {
        return custom ?: getRealDevicePatchLevelInt("boot", isLong = true)
    }

-    /**
-     * Summarises, for a targeted [uid], the device values that feed attestation and where each came
-     * from. This is what exposes attested-versus-live mismatches: a random-fallback verifiedBootKey,
-     * a patch level overridden away from the live prop, or an OS version pulled from a stale cache.
-     */
-    fun describeSources(uid: Int): String {
-        val customPatchLevel = ConfigurationManager.getPatchLevelForUid(uid) != null
-        val osVersionSource =
-            if (DeviceAttestationService.CachedAttestationData?.osVersion != null) "cache" else "map"
-        return "osVersion=$osVersion(src=$osVersionSource) " +
-            "osPatch=${getPatchLevel(uid)} vendorPatch=${getVendorPatchLevelLong(uid)} " +
-            "bootPatch=${getBootPatchLevelLong(uid)} customPatchLevel=$customPatchLevel " +
-            "bootKey=${bootKey.toHex()}(src=$bootKeySource) " +
-            "bootHash=${bootHash.toHex()}(src=$bootHashSource) " +
-            "teeCacheData=${DeviceAttestationService.CachedAttestationData != null}"
-    }
-
    /**
     * Retrieves the definitive device patch level integer for a given component. This function
     * encapsulates the entire fallback chain and guarantees a non-null return.
@@ -336,10 +294,7 @@ object AndroidDeviceUtils {
            // Resolve from live system prop — matches what detectors see via getprop,
            // even when PIF has spoofed ro.build.version.security_patch via resetprop
            resolvedValue.equals("prop", ignoreCase = true) ->
-                parsePatchLevelValue(
-                    SystemProperties.get("ro.build.version.security_patch", ""),
-                    isLong,
-                )
+                parsePatchLevelValue(SystemProperties.get("ro.build.version.security_patch", ""), isLong)
            resolvedValue.equals("no", ignoreCase = true) -> DO_NOT_REPORT
            else -> parsePatchLevelValue(resolvedValue, isLong)
        }
@@ -439,253 +394,36 @@ object AndroidDeviceUtils {
            Build.VERSION_CODES.BAKLAVA to 400, // KeyMint 4.0
        )

-    /** AOSP-mandated attestation version for the running OS, or null when the SDK is unmapped. */
-    internal val aospAttestVersion: Int?
-        get() = attestVersionMap[Build.VERSION.SDK_INT]
-
    /**
-     * Retrieves the attestation version for the given security level. A readable KeyMint VINTF
-     * declaration wins first, so local probes that compare the attested version against the
-     * device's manifest see a coherent pair. Otherwise the legacy chain applies: cached attestation
-     * data, then attestVersionMap[SDK_INT], then 400 as last resort.
+     * Retrieves the attestation version for the given security level. The value follows the device
+     * OS: cached attestation data wins, then attestVersionMap[SDK_INT], then 400 as last resort.
+     * A static StrongBox=300 floor would force a major-version mismatch with the TEE chain on
+     * Android 16 devices that report keymaster 400 across both security levels.
     *
     * @param securityLevel The security level of the attestation (1 for TEE, 2 for StrongBox).
     * @return The appropriate attestation version number.
     */
    fun getAttestVersion(securityLevel: Int): Int {
-        vintfKeyMintVersion?.let { version ->
-            SystemLogger.debug(
-                "attestVersion=${version.attestationVersion} source=vintf securityLevel=$securityLevel"
-            )
-            return version.attestationVersion
-        }
-
        val cached = DeviceAttestationService.CachedAttestationData?.attestVersion
-        val version =
-            cached ?: attestVersionMap[Build.VERSION.SDK_INT] ?: 400 // Default to a recent version
-        val source =
-            when {
+        val version = cached
+            ?: attestVersionMap[Build.VERSION.SDK_INT]
+            ?: 400 // Default to a recent version
+        val source = when {
            cached != null -> "cache"
            attestVersionMap.containsKey(Build.VERSION.SDK_INT) -> "map"
            else -> "default"
        }
        SystemLogger.debug("attestVersion=$version source=$source securityLevel=$securityLevel")
-        SystemLogger.debug(
-            "vintf-version attest=$version keymaster=$version source=$source securityLevel=$securityLevel"
-        )
        return version
    }

    /**
-     * Retrieves the Keymaster/KeyMint version. A readable KeyMint VINTF declaration is
-     * authoritative because recent local detectors compare the attested version directly against
-     * the manifest declaration.
+     * Retrieves the Keymaster/KeyMint version based on the attestation version.
     *
     * @param securityLevel The security level, used to determine the correct attestation version.
     * @return The appropriate Keymaster or KeyMint version number.
     */
-    fun getKeymasterVersion(securityLevel: Int): Int {
-        vintfKeyMintVersion?.let { version ->
-            SystemLogger.debug(
-                "keymasterVersion=${version.keymasterVersion} source=vintf securityLevel=$securityLevel"
-            )
-            return version.keymasterVersion
-        }
-        return getAttestVersion(securityLevel)
-    }
-
-    /**
-     * KeyMint/Keymaster version pair resolved from a device VINTF manifest. [attestationVersion] is
-     * the value written into the attestation record; [keymasterVersion] is the HAL version field.
-     * They coincide for AIDL KeyMint and diverge only for legacy HIDL Keymaster.
-     */
-    private data class VintfKeyMintVersion(
-        val attestationVersion: Int,
-        val keymasterVersion: Int,
-        val sourcePath: String,
-    )
-
-    /**
-     * KeyMint version derived from the device's VINTF manifests, or null when none is readable.
-     * Resolved lazily so the manifest scan happens once, off the attestation hot path.
-     */
-    private val vintfKeyMintVersion: VintfKeyMintVersion? by lazy {
-        readVintfKeyMintVersion().also { version ->
-            if (version != null) {
-                SystemLogger.info(
-                    "Using KeyMint version from VINTF: attestation=${version.attestationVersion}, " +
-                        "keymaster=${version.keymasterVersion}, source=${version.sourcePath}"
-                )
-            } else {
-                SystemLogger.debug(
-                    "No usable KeyMint VINTF declaration found; using attestation fallback"
-                )
-            }
-        }
-    }
-
-    private fun readVintfKeyMintVersion(): VintfKeyMintVersion? {
-        val files = linkedMapOf<String, File>()
-
-        VINTF_MANIFEST_DIRS.forEach { path ->
-            val dir = File(path)
-            if (!dir.exists() || !dir.isDirectory) return@forEach
-            val listed =
-                runCatching {
-                        dir.listFiles { file ->
-                            file.isFile && file.name.endsWith(".xml", ignoreCase = true)
-                        }
-                    }
-                    .getOrElse { throwable ->
-                        SystemLogger.debug("Unable to list VINTF dir $path: ${throwable.message}")
-                        null
-                    }
-            listed?.forEach { file -> files[file.absolutePath] = file }
-        }
-
-        VINTF_MANIFEST_FILES.forEach { path ->
-            val file = File(path)
-            if (file.exists() && file.isFile) {
-                files[file.absolutePath] = file
-            }
-        }
-
-        return files.values
-            .flatMap { file ->
-                runCatching { parseKeyMintVersions(file) }
-                    .getOrElse { throwable ->
-                        SystemLogger.debug(
-                            "Unable to parse KeyMint VINTF ${file.absolutePath}: ${throwable.message}"
-                        )
-                        emptyList()
-                    }
-            }
-            .maxByOrNull { it.attestationVersion }
-    }
-
-    private fun parseKeyMintVersions(file: File): List<VintfKeyMintVersion> {
-        val document = DocumentBuilderFactory.newInstance().newDocumentBuilder().parse(file)
-        val root = document.documentElement ?: return emptyList()
-
-        return directChildElements(root, "hal").flatMap { hal ->
-            val halName = directChildTexts(hal, "name").firstOrNull().orEmpty()
-            val versions = directChildTexts(hal, "version")
-            val fqnames = directChildTexts(hal, "fqname")
-            val interfaces =
-                directChildElements(hal, "interface").associate { interfaceElement ->
-                    val name = directChildTexts(interfaceElement, "name").firstOrNull().orEmpty()
-                    val instances = directChildTexts(interfaceElement, "instance").toSet()
-                    name to instances
-                }
-
-            when (halName) {
-                KEYMINT_HAL_NAME ->
-                    if (hasDefaultInstance(fqnames, interfaces, KEYMINT_INTERFACE_NAME)) {
-                        versions.mapNotNull { version ->
-                            version
-                                .toIntOrNull()
-                                ?.takeIf { it > 0 }
-                                ?.let { aidlVersion ->
-                                    val attestationVersion = aidlVersion * 100
-                                    VintfKeyMintVersion(
-                                        attestationVersion = attestationVersion,
-                                        keymasterVersion = attestationVersion,
-                                        sourcePath = file.absolutePath,
-                                    )
-                                }
-                        }
-                    } else {
-                        emptyList()
-                    }
-                KEYMASTER_HAL_NAME ->
-                    if (hasDefaultInstance(fqnames, interfaces, KEYMASTER_INTERFACE_NAME)) {
-                        (versions.flatMap(::expandHidlVersions) +
-                                fqnames.mapNotNull(::versionFromFqname))
-                            .distinct()
-                            .mapNotNull { version ->
-                                expectedLegacyVersions(version)?.let { expected ->
-                                    VintfKeyMintVersion(
-                                        attestationVersion = expected.second,
-                                        keymasterVersion = expected.first,
-                                        sourcePath = file.absolutePath,
-                                    )
-                                }
-                            }
-                    } else {
-                        emptyList()
-                    }
-                else -> emptyList()
-            }
-        }
-    }
-
-    private fun directChildElements(parent: Element, tagName: String): List<Element> = buildList {
-        val children = parent.childNodes
-        for (index in 0 until children.length) {
-            val child = children.item(index)
-            if (child is Element && child.tagName == tagName) {
-                add(child)
-            }
-        }
-    }
-
-    private fun directChildTexts(parent: Element, tagName: String): List<String> =
-        directChildElements(parent, tagName)
-            .map { it.textContent.trim() }
-            .filter { it.isNotEmpty() }
-
-    private fun hasDefaultInstance(
-        fqnames: List<String>,
-        interfaces: Map<String, Set<String>>,
-        interfaceName: String,
-    ): Boolean =
-        fqnames.any { fqname ->
-            fqname.substringAfter("::", fqname).substringBefore("/") == interfaceName &&
-                fqname.substringAfter("/", "") == DEFAULT_INSTANCE
-        } || interfaces[interfaceName]?.contains(DEFAULT_INSTANCE) == true
-
-    private fun versionFromFqname(fqname: String): String? =
-        FQNAME_VERSION_REGEX.find(fqname)?.groupValues?.getOrNull(1)
-
-    private fun expectedLegacyVersions(version: String): Pair<Int, Int>? =
-        when (version) {
-            "3.0" -> 3 to 2
-            "4.0" -> 4 to 3
-            "4.1" -> 41 to 4
-            else -> null
-        }
-
-    private fun expandHidlVersions(version: String): List<String> {
-        val range = HIDL_VERSION_RANGE_REGEX.matchEntire(version) ?: return listOf(version)
-        val major = range.groupValues[1]
-        val firstMinor = range.groupValues[2].toInt()
-        val lastMinor = range.groupValues[3].toInt()
-        return (firstMinor..lastMinor).map { minor -> "$major.$minor" }
-    }
-
-    private val VINTF_MANIFEST_DIRS =
-        listOf(
-            "/system/etc/vintf/manifest",
-            "/system_ext/etc/vintf/manifest",
-            "/product/etc/vintf/manifest",
-            "/vendor/etc/vintf/manifest",
-            "/odm/etc/vintf/manifest",
-        )
-    private val VINTF_MANIFEST_FILES =
-        listOf(
-            "/system/etc/vintf/manifest.xml",
-            "/system_ext/etc/vintf/manifest.xml",
-            "/product/etc/vintf/manifest.xml",
-            "/vendor/etc/vintf/manifest.xml",
-            "/odm/etc/vintf/manifest.xml",
-        )
-    private const val KEYMINT_HAL_NAME = "android.hardware.security.keymint"
-    private const val KEYMASTER_HAL_NAME = "android.hardware.keymaster"
-    private const val KEYMINT_INTERFACE_NAME = "IKeyMintDevice"
-    private const val KEYMASTER_INTERFACE_NAME = "IKeymasterDevice"
-    private const val DEFAULT_INSTANCE = "default"
-    private val FQNAME_VERSION_REGEX = Regex("^@([0-9]+(?:\\.[0-9]+)?)::")
-    private val HIDL_VERSION_RANGE_REGEX = Regex("^([0-9]+)\\.([0-9]+)-([0-9]+)$")
+    fun getKeymasterVersion(securityLevel: Int): Int = getAttestVersion(securityLevel)

    // --- APEX and Module Hash Properties ---

@@ -780,12 +518,11 @@ object AndroidDeviceUtils {

    val moduleHash: ByteArray by lazy {
        DeviceAttestationService.CachedAttestationData?.moduleHash
-            ?.also { SystemLogger.debug { "module-hash source=cache hash=${it.toHex().take(8)}" } }
-            ?: supplementaryModuleHash()?.also {
-                SystemLogger.debug { "module-hash source=framework-api hash=${it.toHex().take(8)}" }
-            }
            ?: runCatching {
-                    data class ModuleEntry(val nameEncoded: ByteArray, val fullEncoded: ByteArray)
+                    data class ModuleEntry(
+                        val nameEncoded: ByteArray,
+                        val fullEncoded: ByteArray,
+                    )

                    val modules =
                        apexInfos.map { (packageName, versionCode) ->
@@ -818,38 +555,12 @@ object AndroidDeviceUtils {

                    MessageDigest.getInstance("SHA-256").digest(finalDerSet)
                }
-                .onSuccess {
-                    SystemLogger.debug { "module-hash source=rederive hash=${it.toHex().take(8)}" }
-                }
-                .onFailure { SystemLogger.debug { "module-hash source=zero hash=00000000" } }
                .getOrElse {
                    SystemLogger.error("Failed to compute module hash.", it)
                    ByteArray(32)
                }
    }

-    /**
-     * Reads the module-hash DER pre-image straight from the framework's own KeyStoreManager and
-     * SHA-256's it, so the value byte-matches what a verifier derives from the same
-     * getSupplementaryAttestationInfo call. Returns null on any failure (e.g. the API is
-     * unreachable from this process) so the caller falls back to local re-derivation.
-     */
-    private fun supplementaryModuleHash(): ByteArray? =
-        runCatching {
-                // @SystemApi surface added in Android 16, absent from the compile SDK, so reflect.
-                val managerClass = Class.forName("android.security.keystore.KeyStoreManager")
-                val manager = managerClass.getMethod("getInstance").invoke(null)
-                // MODULE_HASH is the KeyMint tag (TagType.BYTES | 724 = 0x900002D4), read from the
-                // framework so it matches the verifier's argument exactly.
-                val moduleHashTag = managerClass.getField("MODULE_HASH").getInt(null)
-                val derPreImage =
-                    managerClass
-                        .getMethod("getSupplementaryAttestationInfo", Int::class.java)
-                        .invoke(manager, moduleHashTag) as ByteArray
-                MessageDigest.getInstance("SHA-256").digest(derPreImage)
-            }
-            .getOrNull()
-
    private fun compareByteArrays(a: ByteArray, b: ByteArray): Int {
        val length = minOf(a.size, b.size)
        for (i in 0 until length) {
@@ -14,15 +14,12 @@ object AndroidPermissionUtils {
            val activityThreadClass = Class.forName("android.app.ActivityThread")
            
            // 2. Invoke the static currentActivityThread() method
-            val currentActivityThreadMethod =
-                activityThreadClass.getDeclaredMethod("currentActivityThread")
+            val currentActivityThreadMethod = activityThreadClass.getDeclaredMethod("currentActivityThread")
            currentActivityThreadMethod.isAccessible = true
            val activityThread = currentActivityThreadMethod.invoke(null)
            
            if (activityThread == null) {
-                SystemLogger.warning(
-                    "Reflection: ActivityThread.currentActivityThread() returned null"
-                )
+                SystemLogger.warning("Reflection: ActivityThread.currentActivityThread() returned null")
                return null
            }

@@ -33,25 +30,23 @@ object AndroidPermissionUtils {
            
            if (application != null) return application

-            // 4. Fallback to getSystemContext() if application is null (often happens in
-            // system_server)
+            // 4. Fallback to getSystemContext() if application is null (often happens in system_server)
            val getSystemContextMethod = activityThreadClass.getDeclaredMethod("getSystemContext")
            getSystemContextMethod.isAccessible = true
            getSystemContextMethod.invoke(activityThread) as? Context
+
        } catch (e: Exception) {
            SystemLogger.error("Reflection failed to get global context for permission check", e)
            null
        }
    }

-    /** Core permission check. */
+    /**
+     * Core permission check.
+     */
    fun hasPermission(uid: Int, permission: String): Boolean {
-        val context =
-            getGlobalContext()
-                ?: run {
-                    SystemLogger.warning(
-                        "AndroidPermissionUtils: Context is null, failing permission check safely."
-                    )
+        val context = getGlobalContext() ?: run {
+            SystemLogger.warning("AndroidPermissionUtils: Context is null, failing permission check safely.")
            return false
        }
        
@@ -7,7 +7,10 @@ package org.matrix.TEESimulator.util
 * @return A new string with each line individually trimmed.
 */
 fun String.trimLines(): String =
-    this.trim().lines().filter { !it.trim().startsWith("<!--") }.joinToString("\n") { it.trim() }
+    this.trim()
+        .lines()
+        .filter { !it.trim().startsWith("<!--") }
+        .joinToString("\n") { it.trim() }

 /**
 * Converts a ByteArray to its hexadecimal string representation.
@@ -4,34 +4,6 @@ CONFIG_DIR=/data/adb/tricky_store

 . "$MODDIR/action_i18n.sh"

-confirm() {
-    # Sample getevent in 1s bursts; a piped stream block-buffers and misses
-    # a single key-press before the timeout.
-    deadline=$(( $(date +%s) + 10 ))
-    while [ "$(date +%s)" -lt "$deadline" ]; do
-        events=$(/system/bin/timeout 1 /system/bin/getevent -l 2>/dev/null)
-        case "$events" in
-            *KEY_VOLUMEUP*)   return 0 ;;
-            *KEY_VOLUMEDOWN*) return 1 ;;
-        esac
-    done
-    return 1
-}
-
-# Debug builds ship diag.sh, adding a one-tap log export before the destructive clear-keys action.
-if [ -f "$MODDIR/diag.sh" ]; then
-    . "$MODDIR/diag.sh"
-    echo " "
-    echo "  📦 Export diagnostic logs to /sdcard/Download?"
-    echo "  🔊  Vol-Up = export logs"
-    echo "  🔉  Vol-Down = skip to clear keys"
-    echo " "
-    if confirm; then
-        diag_export
-        exit 0
-    fi
-fi
-
 echo "  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━"
 echo "  ⚠️  $(_msg confirm_header)"
 echo "  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━"
@@ -43,6 +15,27 @@ echo "  🔊  $(_msg confirm_vol_up)"
 echo "  🔉  $(_msg confirm_vol_down)"
 echo " "

+confirm() {
+    vol_tmp="${TMPDIR:-/data/local/tmp}/teesim_vol_key"
+    : > "$vol_tmp"
+
+    # Stream getevent and match VOLUME DOWN inline. Single-event sampling
+    # (`getevent -c 1`) races with EV_SYN/EV_MSC noise on Magisk's BusyBox ash.
+    /system/bin/timeout 10 /system/bin/sh -c '
+        /system/bin/getevent -lq 2>/dev/null | while IFS= read -r line; do
+            case "$line" in
+                *KEY_VOLUMEUP*DOWN*)   echo UP   > "$1"; exit 0 ;;
+                *KEY_VOLUMEDOWN*DOWN*) echo DOWN > "$1"; exit 0 ;;
+            esac
+        done
+    ' _ "$vol_tmp"
+
+    key=$(cat "$vol_tmp" 2>/dev/null)
+    rm -f "$vol_tmp"
+    [ "$key" = "UP" ] && return 0
+    return 1
+}
+
 if ! confirm; then
    echo " "
    echo "  ❌ $(_msg confirm_cancelled)"
@@ -1,112 +1,3 @@
-> [!NOTE]
-> The project is going through a heavy refactor at the moment, so public commits may lag behind for a while.
-
---
-
-## TEESimulator-RS v6.0.1-307
-
-Fixes five gaps in the module's TEE key-operation and attestation emulation. Two of them fix crashes in real app crypto on a broken-TEE device: any app using an AndroidKeyStore HMAC key or an RSA-OAEP-SHA256 key was throwing. This is a beta; the confirmation logs are in the debug build only, and nothing is field-verified yet.
-
-### App crypto correctness
- HMAC operations now run instead of throwing. An AndroidKeyStore HMAC key is symmetric, so an HMAC SIGN fell into the asymmetric SIGN path and failed on a null key pair, and no MAC primitive existed. SIGN and VERIFY now work: the tag is computed with Mac (HmacSHA256/384/512), truncated to the requested MAC_LENGTH (full digest when unspecified), and checked with a constant-time compare.
- RSA-OAEP-SHA256 decrypt no longer fails with BadPaddingException. The cipher ran with no OAEPParameterSpec, so JCA fell back to SHA-1 and rejected SHA-256 ciphertext. It now applies the correct main and MGF1 digests. A key that authorizes several MGF1 digests uses the one the operation requested, not the key's first.
- Grant-domain attestation keys now resolve. A self-granted PURPOSE_ATTEST_KEY (a Domain.GRANT descriptor) could not resolve its signer alias, so generateKey failed and the subject key was never stored, returning KEY_NOT_FOUND on readback. The grant now resolves to the owner key's alias, and the subject key stays readable under Domain.APP.
-
-### Supplementary attestation
- MODULE_HASH now comes from the framework's own getSupplementaryAttestationInfo, so it matches the value a verifier computes. It falls back to local re-derivation when that API is unreachable.
-
-### Diagnostics (debug builds only)
- New per-operation (oaep-op, hmac-op, attest-grant) and device-level (module-hash, vintf-version) source logs. R8 strips them from release builds.
-
-### 中文说明
-
-修复本模块在 TEE 密钥操作与证明模拟中的五处缺陷。其中两处修复的是真实应用在 TEE 损坏设备上的加密崩溃：任何使用 AndroidKeyStore HMAC 密钥或 RSA-OAEP-SHA256 密钥的应用此前都会抛出异常。本版本为测试版；确认日志仅存在于 debug 构建中，且尚未经过真机验证。
-
-**应用加密正确性**
- HMAC 操作现在可正常执行，不再抛出异常。AndroidKeyStore 的 HMAC 密钥是对称密钥，因此 HMAC SIGN 此前落入了非对称的 SIGN 分支，并因 key pair 为空而失败，当时也没有 MAC 原语。现在 SIGN 与 VERIFY 均可工作：用 Mac (HmacSHA256/384/512) 计算标签，按请求的 MAC_LENGTH 截断（未指定时取完整摘要长度），并用恒定时间比较进行校验。
- RSA-OAEP-SHA256 解密不再抛出 BadPaddingException。此前 cipher 未传入 OAEPParameterSpec，JCA 因而回退到 SHA-1 并拒绝 SHA-256 密文。现在会应用正确的主摘要与 MGF1 摘要。若密钥授权了多个 MGF1 摘要，将使用本次操作请求的那个，而非密钥的第一个。
- Grant 域证明密钥现在可以解析。自授权的 PURPOSE_ATTEST_KEY（Domain.GRANT 描述符）此前无法解析其签名者别名，导致 generateKey 失败且从不存储主体密钥，读取时返回 KEY_NOT_FOUND。现在该 grant 会解析为属主密钥的别名，主体密钥在 Domain.APP 下仍可读取。
-
-**补充证明**
- MODULE_HASH 现在取自框架自带的 getSupplementaryAttestationInfo，因而与验证方计算出的值一致。当该 API 不可用时，回退到本地重新推导。
-
-**诊断（仅 debug 构建）**
- 新增按操作 (oaep-op、hmac-op、attest-grant) 与设备级 (module-hash、vintf-version) 的来源日志。R8 会在 release 构建中将其剥离。
-
---
-
-## TEESimulator-RS v6.0.1-282
-
-AUTO-mode key attestation now forges plain attestation from the keybox instead of deferring to the real TEE.
-
-### Detection coverage
- AUTO dispatch probed the device with `checkTeeFunctionality`, which only proves the TEE can mint one EC key. It says nothing about RSA attestation, device-ID attestation, or whether a patched chain survives RSA verify. Plain attestation requests (attest-key OFF, challenge present) were routed to PATCH and deferred to hardware, so devices that can't back that surfaced KeyAttestation reds: `ATTESTATION_KEYS_NOT_PROVISIONED` (-49) and `BLOCK_TYPE_IS_NOT_01`.
- AUTO targets carrying an attestation challenge now take the FORGE path, the same one attest-key-ON already used: a synthetic chain built from the keybox and rooted under the Google root key. Requests with no challenge still pass through to real hardware, so KeyDetector's hardware-backed checks are unaffected.
-
-### Verified
- Offline conformance against real FORGE captures: uid10389 and uid10154 chains are GREEN; the root SPKI byte-matches `GOOGLE_ROOT_PUBLIC_KEY`.
-
---
-
-## TEESimulator-RS v6.0.1-280
-
-Clears the Duck Detector generate-mode parcel fingerprint that real Android 16 hardware also trips, fixes RSA attestation under an EC-only keybox, and restores device-property attestation for Play Integrity hardware apps such as BHIM and UPI. Generate-mode fix field-confirmed on Android 16.
-
-### Detection coverage
- Generate-mode fingerprint: Duck reads the reply at a flat 12-byte stride and flags the sentinel tuple at positions 12 and 13 that the device's native ALGORITHM-first authorization order lands on. Real A16 silicon trips the same probe, so faithful mirroring stayed flagged. `normalizeAuthorizationLayout` marshals the auth array, runs Duck's exact predicate, and applies a minimal deterministic reorder only when it would match. Count, values, security levels, and the cert chain are untouched, and the reorder keys on the byte condition, never on a package. Applied on both the patch and forge reply paths. (#33)
- `updateAad` on a non-AEAD operation now answers per vendor: Samsung and Xiaomi-MTK TEEs return success, others return INVALID_TAG, matching Duck's OperationErrorPathProbe on both the sign/verify and cipher paths.
-
-### Attestation correctness (Android 16, EC and RSA)
- RSA leaf under an EC-only keybox: patching used to catch the no-RSA-key throw and return the chain untouched, leaking the device's real unlocked Root of Trust for RSA keys while EC keys patched cleanly. It now falls back to any keybox key (EC preferred) and signs the patched leaf with the keybox key's own algorithm, so the RSA leaf re-roots to the Google keybox under a forged locked RoT.
- RSA attest-key forge on an EC-only keybox: the forge path matched the algorithm exactly and threw -75 ATTESTATION_KEYS_NOT_PROVISIONED on a miss, so an RSA ATTEST_KEY request never rooted and verifiers reported an unknown certificate. It now falls back to any attestation key, since an EC key validly ECDSA-signs an RSA-subject leaf. No-op on a dual keybox.
- A16 attestVersion: the device's KeyMint reports version 100 and the lazy cache shadowed the BAKLAVA-to-400 map, so the forge presented 100. It now caches the AOSP value per SDK and presents the correct 400.
- Algorithm-split key on restore: a persisted record holding an EC private key under an RSA leaf failed every signature as DATA_TOO_LARGE_FOR_MODULUS. Restore now drops the record when the private key and served leaf disagree, so the next generateKey rebuilds a coherent key.
- Stale chain on regenerate: reusing an alias in generateKey now evicts the cached chain, matching keystore2, so getKeyEntry serves the current key instead of a stale forge from an earlier generation.
-
-### App compatibility
- Device-property attestation (BRAND, MODEL, and the rest) now forges unconditionally. The old gate probed the live TEE, which is dead on every device the module serves, so it rejected GMS Play Integrity's hardware path and broke BHIM and other UPI and Play-Integrity apps. Device-ID attestation (IMEI, serial) stays governed by the real KeyMint caller-permission rule: privileged callers get it, ordinary apps do not.
- getKeyEntry now reaches the owned-key lookup for skipped privileged UIDs, so framework attestKeyAlias resolution no longer returns "Invalid attestKeyAlias" for Key Attestation over Shizuku. Non-owned keys still skip post-processing, so a real app's key is never patched.
- Device-ID attestation over Shizuku (a privileged UID absent from target.txt) now takes the forge path instead of hitting the real TEE's CANNOT_ATTEST_IDS (-66). "Use attest key" no longer double-roots: a reused persistent attest key is resolved by KEY_ID as well as alias, and an unresolved designated attest key refuses to emit a leaf rather than silently re-rooting under the keybox.
-
-### Diagnostics (debug builds only)
- Per-UID attestation dossier for targeted UIDs at /data/local/tmp/teesim/, recording the decoded chain on both forge and patch paths, key params, the keybox pick (including EC fail-safe), prop sources, forge failures, the emitted authorization shape, and served-versus-verified chains. Release builds strip this through R8 and stay silent. Keybox certificate serials log on every fetch for revocation triage.
-
-### Verified
- Android 16: generate-mode fingerprint signal gone, confirmed on device 2026-06-19.
-
---
-
-## TEESimulator-RS v6.0.1-251
-
-14 commits since v6.0.0-235. Clears the remaining Duck Detector grant-domain rows (incl. the Android 16 OnePlus report), restores Google Wallet and fingerprint compatibility, and removes the in-module patch-level/bulletin resolvers. Test device (SDK 35) TEE tamper score 28 → 8.
-
-### Detection coverage
- Grant plane virtualized: owner read and cross-app `Domain.GRANT` read return one identical chain. 6 RED rows cleared. (28 → 18)
- Generate-mode fingerprint: dropped 2 surplus authorizations (both patchlevels), USER_ID moved to SOFTWARE to mirror a captured device. (18 → 8)
- Android 16 grant: patch-mode keys now served on the grant plane, so owner and grant reads match, fixes CHAIN_SPLIT.
- Grant gated to SDK ≥ 36: Android 15 answers PERMISSION_DENIED, no synthetic over-capability.
- Stale-chain eviction: import and updateSubcomponent drop the cached attestation; no pre-mutation chain replays.
- Lifecycle coherence: clearNamespace / deleteAllKeys / migrateKeyNamespace mirror synthetic key and grant state, defeats delete-then-read probes.
- Device-ID attestation mirrors the real TEE: returns CANNOT_ATTEST_IDS where silicon can't attest, instead of forging it.
-
-### App compatibility
- Google Wallet: INCLUDE_UNIQUE_ID stripped (not rejected) when the caller lacks the permission; card binding works. (PR #27)
- Fingerprint / vendor keys: KEY_ID miss skips the post-handler, so real HAL operations are no longer wrapped and broken. (PR #26)
-
-### Removed
- PatchLevelManager, auto-resolved the security-patch date from an installed PlayIntegrityFix module (with hot-reload) and applied it to props.
- BulletinPoller, scheduled security-bulletin refresh.
-
-### Other
- Release builds purge stale `teesim-*.bin` diagnostics from `/data/local/tmp` at boot.
- Vol-key confirmation rewritten to 1s `getevent` bursts (piped stream missed single presses on Magisk).
-
-### Verified
- SDK 35, Xiaomi 23106RN0DA: tamper 28 → 8; generate-mode signal gone; 4 grant rows UNAVAILABLE (correct for Android 15); no regressions.
- Android 16 grant fix built but unconfirmed on SDK 36, needs an affected OnePlus user to confirm the grant rows clear.
-
---
-
 ## TEESimulator-RS v6.0.0-235

 11 commits since v6.0.0-224. Duck Detector generate-mode fingerprint cleared. Shizuku-routed BYO attestation fixed. Vol-key confirmation restored on Magisk.
@@ -254,7 +145,7 @@ Major release integrating 30+ AOSP compliance improvements from upstream PR #157

 ## TEESimulator-RS v4.8.1: StrongBox Op Rejection Fix

- **StrongBox op limit gate fix**, `trackAndEnforceOpLimit` was only called in the `Domain.KEY_ID` not-found path, so software-generated keys (found via `Domain.APP`) bypassed `STRONGBOX_MAX_CONCURRENT_OPS=4` entirely. DuckDetector's concurrent signing handles test created 24+ operations that all succeeded via LRU pruning instead of being rejected with `TOO_MANY_OPERATIONS (-29)`. Now enforced for all StrongBox createOperation paths.
+- **StrongBox op limit gate fix** — `trackAndEnforceOpLimit` was only called in the `Domain.KEY_ID` not-found path, so software-generated keys (found via `Domain.APP`) bypassed `STRONGBOX_MAX_CONCURRENT_OPS=4` entirely. DuckDetector's concurrent signing handles test created 24+ operations that all succeeded via LRU pruning instead of being rejected with `TOO_MANY_OPERATIONS (-29)`. Now enforced for all StrongBox createOperation paths.

 ---

@@ -262,13 +153,13 @@ Major release integrating 30+ AOSP compliance improvements from upstream PR #157

 Tested against DuckDetector on OnePlus (Android 16, KSU). Tamper score dropped from 32 to 8.

- **LRU operation pruning**, Concurrent software operations capped at 15 per UID (TEE) and 4 per UID (StrongBox), with oldest-first eviction. Pruned operations return `INVALID_OPERATION_HANDLE (-28)`, matching AOSP keystore2 malus-based pruning.
- **StrongBox param guard**, Unsupported StrongBox params (RSA >2048-bit, non-P256 EC curves) forwarded to real HAL for proper rejection instead of generating in software.
- **StrongBox timing**, Key generation floors at 250ms, signing at 80ms on StrongBox security level to match real secure element latency.
- **StrongBox op limit**, Sliding-window enforcer caps concurrent StrongBox operations for both software and hardware key paths, returning `TOO_MANY_OPERATIONS (-29)` when exceeded.
- **ECDSA algorithm alias**, Accept "ECDSA" in addition to "EC" as JCA private key algorithm name. Fixes SIGSEGV crash on Android 10 devices where the provider reports EC keys as "ECDSA". Closes #4.
- **createOperation domain handling**, Software-generated keys now found via both `Domain.APP` (alias) and `Domain.KEY_ID` (nspace) lookup paths.
- **Permission guards**, Device ID attestation tags (IMEI, MEID, serial) require caller permission checks.
+- **LRU operation pruning** — Concurrent software operations capped at 15 per UID (TEE) and 4 per UID (StrongBox), with oldest-first eviction. Pruned operations return `INVALID_OPERATION_HANDLE (-28)`, matching AOSP keystore2 malus-based pruning.
+- **StrongBox param guard** — Unsupported StrongBox params (RSA >2048-bit, non-P256 EC curves) forwarded to real HAL for proper rejection instead of generating in software.
+- **StrongBox timing** — Key generation floors at 250ms, signing at 80ms on StrongBox security level to match real secure element latency.
+- **StrongBox op limit** — Sliding-window enforcer caps concurrent StrongBox operations for both software and hardware key paths, returning `TOO_MANY_OPERATIONS (-29)` when exceeded.
+- **ECDSA algorithm alias** — Accept "ECDSA" in addition to "EC" as JCA private key algorithm name. Fixes SIGSEGV crash on Android 10 devices where the provider reports EC keys as "ECDSA". Closes #4.
+- **createOperation domain handling** — Software-generated keys now found via both `Domain.APP` (alias) and `Domain.KEY_ID` (nspace) lookup paths.
+- **Permission guards** — Device ID attestation tags (IMEI, MEID, serial) require caller permission checks.

 ---

@@ -276,20 +167,20 @@ Tested against DuckDetector on OnePlus (Android 16, KSU). Tamper score dropped f

 Tested against [KeyDetector](https://github.com/XiaoTong6666/KeyDetector) and [Key Attestation](https://github.com/nickel-lang/nickel) on OnePlus (Android 16) and Xiaomi Redmi 14C (Android 14).

- **PADDING encoding**, Fixed ASN.1 encoding of PADDING tag in attestation extension from individual `[6] INTEGER` entries to `[6] SET OF INTEGER`, matching AOSP `attestation_record.h` schema. Broke all RSA key attestation since v4.6.
- **Operation error-path conformance**, Software operations now track finalized state and return `INVALID_OPERATION_HANDLE (-28)` on post-abort calls. Input length guard (32KB) returns `TOO_MUCH_DATA` matching AOSP `operation.rs`. Passes KeyDetector's OperationErrorPathChecker.
- **updateAad support**, Added `updateAad` to `SoftwareOperationBinder`, fixing `AbstractMethodError` on Android 16 where the runtime Stub declares it abstract.
- **Algorithm inference**, `createOperation` now infers algorithm from the stored key pair when operation params omit the ALGORITHM tag, matching AOSP behavior.
+- **PADDING encoding** — Fixed ASN.1 encoding of PADDING tag in attestation extension from individual `[6] INTEGER` entries to `[6] SET OF INTEGER`, matching AOSP `attestation_record.h` schema. Broke all RSA key attestation since v4.6.
+- **Operation error-path conformance** — Software operations now track finalized state and return `INVALID_OPERATION_HANDLE (-28)` on post-abort calls. Input length guard (32KB) returns `TOO_MUCH_DATA` matching AOSP `operation.rs`. Passes KeyDetector's OperationErrorPathChecker.
+- **updateAad support** — Added `updateAad` to `SoftwareOperationBinder`, fixing `AbstractMethodError` on Android 16 where the runtime Stub declares it abstract.
+- **Algorithm inference** — `createOperation` now infers algorithm from the stored key pair when operation params omit the ALGORITHM tag, matching AOSP behavior.

 ---

 ## TEESimulator-RS v4.6: Rebrand & Detection Fix

- **RTT normalization rework**, Replaced Gaussian sleep (mean=55ms) with a 15ms floor fence. The old approach triggered Chunqiu Native Check 2.8 timing analysis; the floor-only approach satisfies the minimum RTT threshold without creating a detectable delay pattern.
- **Cross-algorithm attestation**, Signing algorithm now derived from the attestation key's actual type, not the generated key's algorithm. Fixes BouncyCastle crash when signing RSA keys with EC attestation keys (Shizuku attestation flow).
- **Device ID attestation**, Serial/IMEI/MEID/secondImei tags now flow through to software cert gen instead of blanket rejection. Only DEVICE_UNIQUE_ATTESTATION is rejected, matching AOSP keystore2 policy.
- **Rebrand to TEESimulator-RS**, Distinguishes this fork from upstream. Version scheme simplified to v{major}.{minor}-{commitCount}.
- **CI streamlined**, Release pipeline uses Gradle-generated filenames directly, eliminating the rename step.
+- **RTT normalization rework** — Replaced Gaussian sleep (mean=55ms) with a 15ms floor fence. The old approach triggered Chunqiu Native Check 2.8 timing analysis; the floor-only approach satisfies the minimum RTT threshold without creating a detectable delay pattern.
+- **Cross-algorithm attestation** — Signing algorithm now derived from the attestation key's actual type, not the generated key's algorithm. Fixes BouncyCastle crash when signing RSA keys with EC attestation keys (Shizuku attestation flow).
+- **Device ID attestation** — Serial/IMEI/MEID/secondImei tags now flow through to software cert gen instead of blanket rejection. Only DEVICE_UNIQUE_ATTESTATION is rejected, matching AOSP keystore2 policy.
+- **Rebrand to TEESimulator-RS** — Distinguishes this fork from upstream. Version scheme simplified to v{major}.{minor}-{commitCount}.
+- **CI streamlined** — Release pipeline uses Gradle-generated filenames directly, eliminating the rename step.

 ---

@@ -297,28 +188,28 @@ Tested against [KeyDetector](https://github.com/XiaoTong6666/KeyDetector) and [K

 Tested against [KeyDetector](https://github.com/XiaoTong6666/KeyDetector) (23-check attestation validator). All keystore-level checks now pass.

- **Key deletion consistency**, After deleting a software-generated key, `getKeyEntry` now correctly returns `KEY_NOT_FOUND` instead of falling through to a stale live-patch fallback. Fixes binder consistency checks that detect ghost key responses.
- **generateKey timing normalization**, Software key generation RTT now matches real TEE latency profile (Gaussian distribution, mean=55ms, floor=15ms). Previously completed in ~4ms, which is an immediate timing side-channel.
- **Delete cleanup scope**, `deleteKey` now clears all cached state (patched chains, attestation keys) regardless of whether the key was software or hardware-generated.
+- **Key deletion consistency** — After deleting a software-generated key, `getKeyEntry` now correctly returns `KEY_NOT_FOUND` instead of falling through to a stale live-patch fallback. Fixes binder consistency checks that detect ghost key responses.
+- **generateKey timing normalization** — Software key generation RTT now matches real TEE latency profile (Gaussian distribution, mean=55ms, floor=15ms). Previously completed in ~4ms, which is an immediate timing side-channel.
+- **Delete cleanup scope** — `deleteKey` now clears all cached state (patched chains, attestation keys) regardless of whether the key was software or hardware-generated.

 ---

 ## TEESimulator v4.4: AOSP Conformance

- **Binder error reply format**, Aligned EX_SERVICE_SPECIFIC wire layout with AOSP Status.cpp, including the remote stack trace header field.
- **Key enumeration**, Corrected list_past_alias pagination order to match AOSP database.rs semantics.
- **KeyMetadata fields**, Generated key responses now include modificationTimeMs, Tag.ORIGIN, and normalized KeyDescriptor fields per AOSP Keystore2.
- **Parcel handling**, hasException() preserves reply position for downstream consumers.
+- **Binder error reply format** — Aligned EX_SERVICE_SPECIFIC wire layout with AOSP Status.cpp, including the remote stack trace header field.
+- **Key enumeration** — Corrected list_past_alias pagination order to match AOSP database.rs semantics.
+- **KeyMetadata fields** — Generated key responses now include modificationTimeMs, Tag.ORIGIN, and normalized KeyDescriptor fields per AOSP Keystore2.
+- **Parcel handling** — hasException() preserves reply position for downstream consumers.

 ---

 ## TEESimulator v4.3: Performance & Reliability

- **Debug log gating**, `SystemLogger.debug()` now skipped entirely in release builds, eliminating unnecessary logcat syscalls on every intercepted transaction.
- **Supervisor backoff**, Exponential restart delay (500ms → 30s cap) prevents CPU spin if the daemon crashes repeatedly. Resets automatically once stable.
- **Process priority**, Daemon runs at nice=10, yielding CPU to foreground apps on constrained devices.
- **Map eviction**, Rate limiter and file lock maps now evict stale entries instead of growing unbounded.
- **CI pipeline**, Single-trigger build→release pipeline with proper changelog extraction and correctly sized artifacts.
+- **Debug log gating** — `SystemLogger.debug()` now skipped entirely in release builds, eliminating unnecessary logcat syscalls on every intercepted transaction.
+- **Supervisor backoff** — Exponential restart delay (500ms → 30s cap) prevents CPU spin if the daemon crashes repeatedly. Resets automatically once stable.
+- **Process priority** — Daemon runs at nice=10, yielding CPU to foreground apps on constrained devices.
+- **Map eviction** — Rate limiter and file lock maps now evict stale entries instead of growing unbounded.
+- **CI pipeline** — Single-trigger build→release pipeline with proper changelog extraction and correctly sized artifacts.

 ---

@@ -330,9 +221,9 @@ Fixes 6 detection vectors flagged by attestation validator apps.

 Replicate AOSP keystore2's `add_required_parameters()` validation that our software keygen path was bypassing:

- **CREATION_DATETIME**, Reject caller-provided input with `INVALID_ARGUMENT (20)`, matching `security_level.rs:424`. Our cert gen still adds its own timestamp, same as real keystore2.
- **Device ID attestation**, Reject ATTESTATION_ID_SERIAL, IMEI, MEID, SECOND_IMEI, and DEVICE_UNIQUE_ATTESTATION with `CANNOT_ATTEST_IDS (-66)`. No consumer app has READ_PRIVILEGED_PHONE_STATE.
- **Error reply format**, Fixed AIDL ServiceSpecificException parcel write order (was errorCode→message, now message→errorCode).
+- **CREATION_DATETIME** — Reject caller-provided input with `INVALID_ARGUMENT (20)`, matching `security_level.rs:424`. Our cert gen still adds its own timestamp, same as real keystore2.
+- **Device ID attestation** — Reject ATTESTATION_ID_SERIAL, IMEI, MEID, SECOND_IMEI, and DEVICE_UNIQUE_ATTESTATION with `CANNOT_ATTEST_IDS (-66)`. No consumer app has READ_PRIVILEGED_PHONE_STATE.
+- **Error reply format** — Fixed AIDL ServiceSpecificException parcel write order (was errorCode→message, now message→errorCode).

 ### Certificate Fix

@@ -360,15 +251,15 @@ Major release. Certificate chain generation rebuilt from the ground up in Rust,

 ### Native Cert Generation

-The headline feature. `libcertgen.so` generates X.509 certificate chains using `ring` (EC-P256/P384) and `rsa` (RSA-2048/4096) with manual DER assembly. No more BouncyCastle quirks, issuer/subject DN bytes are injected directly from the keybox, ensuring byte-perfect chain linkage. BouncyCastle remains as fallback for unsupported curves (P-224, P-521, Curve25519).
+The headline feature. `libcertgen.so` generates X.509 certificate chains using `ring` (EC-P256/P384) and `rsa` (RSA-2048/4096) with manual DER assembly. No more BouncyCastle quirks — issuer/subject DN bytes are injected directly from the keybox, ensuring byte-perfect chain linkage. BouncyCastle remains as fallback for unsupported curves (P-224, P-521, Curve25519).

 ### Anti-Detection Hardening

- **Challenge validation**, Oversized attestation challenges (>128 bytes) now return `INVALID_INPUT_LENGTH (-21)`, matching real KeyMint behavior. Previously accepted silently, DuckDetector exploited this.
- **Per-UID rate limiter**, 2 hardware keygens per 30s burst, 2 concurrent max. Overflow falls back to software certs. Blocks DuckDetector-style keygen flooding that starves GMS.
- **importKey eviction guard**, Retained patch chains prevent generate-then-import attacks that evict cached attestation data.
- **256KB native payload cap**, Oversized binder payloads bypass interception cleanly instead of stalling threads.
- **Alias size rejection**, Oversized key aliases rejected before they hit the binder buffer.
+- **Challenge validation** — Oversized attestation challenges (>128 bytes) now return `INVALID_INPUT_LENGTH (-21)`, matching real KeyMint behavior. Previously accepted silently — DuckDetector exploited this.
+- **Per-UID rate limiter** — 2 hardware keygens per 30s burst, 2 concurrent max. Overflow falls back to software certs. Blocks DuckDetector-style keygen flooding that starves GMS.
+- **importKey eviction guard** — Retained patch chains prevent generate-then-import attacks that evict cached attestation data.
+- **256KB native payload cap** — Oversized binder payloads bypass interception cleanly instead of stalling threads.
+- **Alias size rejection** — Oversized key aliases rejected before they hit the binder buffer.

 ### Key Persistence

@@ -380,7 +271,7 @@ Generated keys now survive reboots. File-backed storage with file-level locking,
 - Correct `module_hash` field to match AOSP Keystore2 format
 - Override pre-existing attest keys instead of skipping them
 - Strip HTML comments from PEM blocks in keybox parsing
- Security patch consistency, `system=prop` forces boot/vendor to match
+- Security patch consistency — `system=prop` forces boot/vendor to match

 ### Module Lifecycle

@@ -391,9 +282,9 @@ Generated keys now survive reboots. File-backed storage with file-level locking,
 ### Stability

 - FileObserver NPE on config deletion fixed
- Global uncaught exception handler, daemon stays alive on unexpected errors
+- Global uncaught exception handler — daemon stays alive on unexpected errors
 - PEM parsing hardened against malformed keybox files

 ### Tested Against

-DuckDetector, Luna, Play Integrity, Key Attestation Demo, all passing on Redmi 14C (Android 14, Beanpod KeyMaster, KSU).
+DuckDetector, Luna, Play Integrity, Key Attestation Demo — all passing on Redmi 14C (Android 14, Beanpod KeyMaster, KSU).
@@ -76,20 +76,6 @@ mv "$MODPATH/libsupervisor.so" "$MODPATH/supervisor"
 chmod 755 "$MODPATH/inject"
 chmod 755 "$MODPATH/supervisor"

-# Debug builds carry diag.sh (the diagnostic plane); release builds do not. Extract it when
-# present; otherwise sweep any external-storage diagnostics a prior debug install left behind,
-# since the release keystore domain has no grant to remove them itself.
-# Detect presence by the extracted FILE, not unzip's exit code: the busybox/toybox unzip in
-# the install environment exits 0 even when the entry is absent, so the sweep never ran.
-unzip -qqjo "$ZIPFILE" "diag.sh" -d "$MODPATH" 2>/dev/null
-if [ -f "$MODPATH/diag.sh" ]; then
-  chmod 644 "$MODPATH/diag.sh"
-  ui_print "- Debug diagnostic plane enabled"
-else
-  rm -rf /data/media/0/TEESimulator /data/local/tmp/teesim
-  ui_print "- Release build: swept stale diagnostics"
-fi
-
 # --- Configuration Files ---
 if [ ! -d "$CONFIG_DIR" ]; then
  ui_print "- Creating configuration directory"
@@ -1,20 +0,0 @@
-#!/system/bin/sh
-# Debug-only diagnostic plane. Shipped solely in debug ZIPs; its presence is the gate that
-# service.sh (setup) and action.sh (export) test before touching external storage.
-DIAG_DIR=/data/media/0/TEESimulator
-
-diag_setup() {
-    mkdir -p "$DIAG_DIR"
-    chmod 0777 "$DIAG_DIR"
-    chcon u:object_r:media_rw_data_file:s0 "$DIAG_DIR" 2>/dev/null
-}
-
-diag_export() {
-    _ts=$(date +%Y%m%d-%H%M%S)
-    _dest=/sdcard/Download/teesim-logs-$_ts
-    mkdir -p "$_dest"
-    cp -f "$DIAG_DIR"/teesim-uid-* "$_dest"/ 2>/dev/null
-    cp -f /data/adb/tricky_store/logs/certgen.log* "$_dest"/ 2>/dev/null
-    logcat -d -s TEESimulator > "$_dest/logcat.txt" 2>/dev/null
-    echo "  ✅ Saved to $_dest"
-}
@@ -1,10 +1,6 @@
 allow keystore {adb_data_file shell_data_file} file *
 allow crash_dump keystore process *

-# SOTER Layer-A (10.C): ptrace inject into soterserver (platform_app). The debug NDJSON
-# media_rw_data_file grant is debug-only — appended for debug builds in app/build.gradle.kts.
-allow crash_dump platform_app process *
-
 allow ksu self:tcp_socket { create connect read write getopt setopt }
 allow ksu node:tcp_socket node_bind
 allow ksu port:tcp_socket name_connect
@@ -4,12 +4,6 @@ cd $MODDIR
 # Fork-based supervisor for instant restart
 ./supervisor ./daemon "$MODDIR" &

-# Debug builds ship diag.sh; its presence enables the external-storage diagnostic plane.
-if [ -f "$MODDIR/diag.sh" ]; then
-  . "$MODDIR/diag.sh"
-  diag_setup
-fi
-
 # Clear logd size persist properties once boot completes
 (
  until [ "$(getprop sys.boot_completed)" = "1" ]; do
@@ -11,6 +11,3 @@ rm -rf "$CONFIG_DIR/persistent_keys"
 rm -f "$CONFIG_DIR/tee_status.txt"
 rm -f "$CONFIG_DIR/boot_hash.bin" "$CONFIG_DIR/boot_key.bin"
 rm -f "$CONFIG_DIR/security_patch.txt" "$CONFIG_DIR/security_patch.txt.next" "$CONFIG_DIR/last_bulletin_fetch.json"
-
-# Debug diagnostics live on external storage; remove them on uninstall.
-rm -rf /data/media/0/TEESimulator
@@ -1,6 +1,6 @@
 {
-  "version": "v6.0.1-307",
-  "versionCode": 307,
-  "zipUrl": "https://github.com/Enginex0/TEESimulator-RS/releases/download/v6.0.1-307/TEESimulator-RS-v6.0.1-307-Release.zip",
+  "version": "v6.0.0-235",
+  "versionCode": 235,
+  "zipUrl": "https://github.com/Enginex0/TEESimulator-RS/releases/download/v6.0.0-235/TEESimulator-RS-v6.0.0-235-Release.zip",
  "changelog": "https://raw.githubusercontent.com/Enginex0/TEESimulator-RS/main/module/changelog.md"
 }
@@ -2,6 +2,12 @@
 # It is not intended for manual editing.
 version = 4

+[[package]]
+name = "adler2"
+version = "2.0.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "320119579fcad9c21884f5c4861d16174d0e06250625266f50fe6898340abefa"
+
 [[package]]
 name = "aho-corasick"
 version = "1.1.4"
@@ -17,6 +23,15 @@ version = "1.0.102"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "7f202df86484c868dbad7eaa557ef785d5c66295e41b460ef922eca0723b842c"

+[[package]]
+name = "arbitrary"
+version = "1.4.2"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "c3d036a3c4ab069c7b410a2ce876bd74808d2d0888a82667669f8e783a898bf1"
+dependencies = [
+ "derive_arbitrary",
+]
+
 [[package]]
 name = "autocfg"
 version = "1.5.0"
@@ -38,6 +53,12 @@ dependencies = [
 "generic-array",
 ]

+[[package]]
+name = "bumpalo"
+version = "3.20.2"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "5d20789868f4b01b2f2caec9f5c4e0213b41e3e5702a50157d699ae31ced2fcb"
+
 [[package]]
 name = "bytes"
 version = "1.11.1"
@@ -62,6 +83,7 @@ dependencies = [
 "const-oid",
 "der",
 "jni",
+ "libc",
 "pkcs8",
 "rand",
 "ring",
@@ -71,6 +93,7 @@ dependencies = [
 "tracing",
 "tracing-subscriber",
 "x509-cert",
+ "zip",
 ]

 [[package]]
@@ -110,6 +133,21 @@ dependencies = [
 "libc",
 ]

+[[package]]
+name = "crc32fast"
+version = "1.5.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "9481c1c90cbf2ac953f07c8d4a58aa3945c425b7185c9154d67a65e4230da511"
+dependencies = [
+ "cfg-if",
+]
+
+[[package]]
+name = "crossbeam-utils"
+version = "0.8.21"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "d0a5c400df2834b80a4c3327b3aad3a4c4cd4de0629063962b03235697506a28"
+
 [[package]]
 name = "crypto-common"
 version = "0.1.7"
@@ -153,6 +191,17 @@ dependencies = [
 "powerfmt",
 ]

+[[package]]
+name = "derive_arbitrary"
+version = "1.4.2"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "1e567bd82dcff979e4b03460c307b3cdc9e96fde3d73bed1496d2bc75d9dd62a"
+dependencies = [
+ "proc-macro2",
+ "quote",
+ "syn",
+]
+
 [[package]]
 name = "digest"
 version = "0.10.7"
@@ -164,6 +213,23 @@ dependencies = [
 "crypto-common",
 ]

+[[package]]
+name = "displaydoc"
+version = "0.2.5"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "97369cbbc041bc366949bc74d34658d6cda5621039731c6310521892a3a20ae0"
+dependencies = [
+ "proc-macro2",
+ "quote",
+ "syn",
+]
+
+[[package]]
+name = "equivalent"
+version = "1.0.2"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "877a4ace8713b0bcf2a4e7eec82529c029f1d0619886d18145fea96c3ffe5c0f"
+
 [[package]]
 name = "find-msvc-tools"
 version = "0.1.9"
@@ -176,6 +242,16 @@ version = "0.4.7"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "b7ac824320a75a52197e8f2d787f6a38b6718bb6897a35142d749af3c0e8f4fe"

+[[package]]
+name = "flate2"
+version = "1.1.9"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "843fba2746e448b37e26a819579957415c8cef339bf08564fe8b7ddbd959573c"
+dependencies = [
+ "crc32fast",
+ "miniz_oxide",
+]
+
 [[package]]
 name = "generic-array"
 version = "0.14.7"
@@ -197,6 +273,22 @@ dependencies = [
 "wasi",
 ]

+[[package]]
+name = "hashbrown"
+version = "0.16.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "841d1cc9bed7f9236f321df977030373f4a4163ae1a7dbfe1a51a2c1a51d9100"
+
+[[package]]
+name = "indexmap"
+version = "2.13.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "7714e70437a7dc3ac8eb7e6f8df75fd8eb422675fc7678aff7364301092b1017"
+dependencies = [
+ "equivalent",
+ "hashbrown",
+]
+
 [[package]]
 name = "itoa"
 version = "1.0.17"
@@ -214,7 +306,7 @@ dependencies = [
 "combine",
 "jni-sys",
 "log",
- "thiserror",
+ "thiserror 1.0.69",
 "walkdir",
 "windows-sys 0.45.0",
 ]
@@ -267,6 +359,16 @@ version = "2.8.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "f8ca58f447f06ed17d5fc4043ce1b10dd205e060fb3ce5b979b8ed8e59ff3f79"

+[[package]]
+name = "miniz_oxide"
+version = "0.8.9"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "1fa76a2c86f704bdb222d66965fb3d63269ce38518b83cb0575fca855ebb6316"
+dependencies = [
+ "adler2",
+ "simd-adler32",
+]
+
 [[package]]
 name = "nu-ansi-term"
 version = "0.50.3"
@@ -572,6 +674,12 @@ dependencies = [
 "rand_core",
 ]

+[[package]]
+name = "simd-adler32"
+version = "0.3.8"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "e320a6c5ad31d271ad523dcf3ad13e2767ad8b1cb8f047f75a8aeaf8da139da2"
+
 [[package]]
 name = "smallvec"
 version = "1.15.1"
@@ -617,7 +725,16 @@ version = "1.0.69"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "b6aaf5339b578ea85b50e080feb250a3e8ae8cfcdff9a461c9ec2904bc923f52"
 dependencies = [
- "thiserror-impl",
+ "thiserror-impl 1.0.69",
+]
+
+[[package]]
+name = "thiserror"
+version = "2.0.18"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "4288b5bcbc7920c07a1149a35cf9590a2aa808e0bc1eafaade0b80947865fbc4"
+dependencies = [
+ "thiserror-impl 2.0.18",
 ]

 [[package]]
@@ -631,6 +748,17 @@ dependencies = [
 "syn",
 ]

+[[package]]
+name = "thiserror-impl"
+version = "2.0.18"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "ebc4ee7f67670e9b64d05fa4253e753e016c6c95ff35b89b7941d6b856dec1d5"
+dependencies = [
+ "proc-macro2",
+ "quote",
+ "syn",
+]
+
 [[package]]
 name = "thread_local"
 version = "1.1.9"
@@ -1002,8 +1130,37 @@ dependencies = [
 "syn",
 ]

+[[package]]
+name = "zip"
+version = "2.4.2"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "fabe6324e908f85a1c52063ce7aa26b68dcb7eb6dbc83a2d148403c9bc3eba50"
+dependencies = [
+ "arbitrary",
+ "crc32fast",
+ "crossbeam-utils",
+ "displaydoc",
+ "flate2",
+ "indexmap",
+ "memchr",
+ "thiserror 2.0.18",
+ "zopfli",
+]
+
 [[package]]
 name = "zmij"
 version = "1.0.21"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "b8848ee67ecc8aedbaf3e4122217aff892639231befc6a1b58d29fff4c2cabaa"
+
+[[package]]
+name = "zopfli"
+version = "0.8.3"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "f05cd8797d63865425ff89b5c4a48804f35ba0ce8d125800027ad6017d2b5249"
+dependencies = [
+ "bumpalo",
+ "crc32fast",
+ "log",
+ "simd-adler32",
+]
@@ -20,6 +20,8 @@ time = { version = "0.3", features = ["std"] }
 anyhow = "1.0"
 tracing = "0.1"
 tracing-subscriber = { version = "0.3", features = ["env-filter"] }
+libc = "0.2"
+zip = { version = "2.2", default-features = false, features = ["deflate"] }
 serde_json = "1.0"

 [profile.release]
@@ -9,7 +9,7 @@ pub mod certbuilder;
 pub mod logging;

 use jni::objects::{JByteArray, JClass, JIntArray, JObject, JString};
-use jni::sys::{jboolean, jbyteArray};
+use jni::sys::{jboolean, jbyteArray, jstring};
 use jni::JNIEnv;

 use crate::error::{CertGenError, Result};
@@ -64,41 +64,18 @@ fn generate_attested_inner(env: &mut JNIEnv, config: &JObject) -> Result<jbyteAr

    let cert_chain = if params.attestation_challenge.is_some() {
        let attest_ext = attestation::build_attestation_extension(&params)?;
-        // Ground truth of what the Rust forger emitted, keyed to the app. Gated on the APK debug
-        // variant so release builds never dump the extension.
-        if params.debug_logging {
-            tracing::info!(
-                uid = params.uid,
-                ext_hex = %hex_encode(&attest_ext),
-                "produced attestation extension"
-            );
-        }
        certbuilder::build_certificate_chain(&key_pair, Some(&attest_ext), &keybox, &params)?
    } else {
-        tracing::info!(
-            uid = params.uid,
-            "no attestation challenge, generating self-signed cert (depth 1)"
-        );
+        tracing::info!("no attestation challenge, generating self-signed cert (depth 1)");
        certbuilder::build_self_signed_cert(&key_pair, &params)?
    };

    let blob = assemble_result(&key_pair.private_key_pkcs8, &cert_chain);
-    tracing::info!(uid = params.uid, certs = cert_chain.len(), "assembled native cert result");

    let out = env.byte_array_from_slice(&blob)?;
    Ok(out.into_raw())
 }

-/// Lowercase hex of a byte slice for diagnostic dumps; the crate has no `hex` dependency.
-fn hex_encode(bytes: &[u8]) -> String {
-    use std::fmt::Write as _;
-    let mut out = String::with_capacity(bytes.len() * 2);
-    for b in bytes {
-        let _ = write!(out, "{:02x}", b);
-    }
-    out
-}
-
 // ---------------------------------------------------------------------------
 // JNI entry: initLogging
 // ---------------------------------------------------------------------------
@@ -140,6 +117,44 @@ fn init_logging_inner(env: &mut JNIEnv, verbose: jboolean, log_dir: &JString) ->
    Ok(())
 }

+// ---------------------------------------------------------------------------
+// JNI entry: dumpLogs
+// ---------------------------------------------------------------------------
+
+#[no_mangle]
+pub extern "system" fn Java_org_matrix_TEESimulator_pki_NativeCertGen_dumpLogs(
+    mut env: JNIEnv,
+    _class: JClass,
+) -> jstring {
+    let result = std::panic::catch_unwind(std::panic::AssertUnwindSafe(|| {
+        dump_logs_inner(&mut env)
+    }));
+
+    match result {
+        Ok(Ok(raw)) => raw,
+        Ok(Err(e)) => {
+            tracing::error!(%e, "dumpLogs failed");
+            std::ptr::null_mut()
+        }
+        Err(_) => {
+            tracing::error!("dumpLogs panicked");
+            std::ptr::null_mut()
+        }
+    }
+}
+
+fn dump_logs_inner(env: &mut JNIEnv) -> Result<jstring> {
+    logging::dump::execute_dump()
+        .map_err(|e| CertGenError::Jni(format!("dump failed: {e}")))?;
+
+    // Read the dump path written by execute_dump
+    let path = std::fs::read_to_string("/data/adb/tricky_store/.dump_path")
+        .map_err(|e| CertGenError::Jni(format!("read dump path: {e}")))?;
+
+    let jpath = env.new_string(&path)?;
+    Ok(jpath.into_raw())
+}
+
 // ---------------------------------------------------------------------------
 // Config extraction from Java CertGenConfig object
 // ---------------------------------------------------------------------------
@@ -190,8 +205,6 @@ fn extract_config(env: &mut JNIEnv, config: &JObject) -> Result<CertGenParams> {
    let caller_nonce = get_boolean(env, config, "callerNonce")?;
    let unlocked_device_required = get_boolean(env, config, "unlockedDeviceRequired")?;
    let no_auth_required = get_boolean(env, config, "noAuthRequired")?;
-    let uid = get_int(env, config, "uid")?;
-    let debug_logging = get_boolean(env, config, "debugLogging")?;

    Ok(CertGenParams {
        algorithm: Algorithm::try_from(algorithm)?,
@@ -239,8 +252,6 @@ fn extract_config(env: &mut JNIEnv, config: &JObject) -> Result<CertGenParams> {
        caller_nonce,
        unlocked_device_required,
        no_auth_required,
-        uid,
-        debug_logging,
    })
 }

@@ -0,0 +1,208 @@
+use std::fs::{self, File};
+use std::io::{Read, Write};
+use std::path::Path;
+use std::process::Command;
+use std::time::{SystemTime, UNIX_EPOCH};
+
+const DUMP_DIR: &str = "/sdcard/Download";
+const LOCK_PATH: &str = "/data/adb/tricky_store/.dump_lock";
+const DUMP_PATH_FILE: &str = "/data/adb/tricky_store/.dump_path";
+const LOG_DIR: &str = "/data/adb/tricky_store/logs";
+const BASE_DIR: &str = "/data/adb/tricky_store";
+const LOGCAT_SIZE_LIMIT: usize = 2 * 1024 * 1024;
+
+struct FlockGuard {
+    _file: File,
+}
+
+impl FlockGuard {
+    fn acquire() -> Result<Self, Box<dyn std::error::Error>> {
+        if let Some(parent) = Path::new(LOCK_PATH).parent() {
+            fs::create_dir_all(parent)?;
+        }
+        let file = File::create(LOCK_PATH)?;
+        let fd = {
+            use std::os::unix::io::AsRawFd;
+            file.as_raw_fd()
+        };
+        let ret = unsafe { libc::flock(fd, libc::LOCK_EX | libc::LOCK_NB) };
+        if ret != 0 {
+            return Err("dump already in progress".into());
+        }
+        Ok(Self { _file: file })
+    }
+}
+
+impl Drop for FlockGuard {
+    fn drop(&mut self) {
+        // flock released automatically when file descriptor closes
+    }
+}
+
+fn random_name(len: usize) -> String {
+    use rand::Rng;
+    let mut rng = rand::thread_rng();
+    (0..len)
+        .map(|_| {
+            let idx = rng.gen_range(0..36u8);
+            if idx < 10 {
+                (b'0' + idx) as char
+            } else {
+                (b'a' + idx - 10) as char
+            }
+        })
+        .collect()
+}
+
+fn collect_logcat(tag: &str) -> Vec<u8> {
+    let output = Command::new("logcat")
+        .args(["-d", "-s", tag])
+        .output();
+
+    match output {
+        Ok(o) => {
+            let mut data = o.stdout;
+            data.truncate(LOGCAT_SIZE_LIMIT);
+            data
+        }
+        Err(_) => Vec::new(),
+    }
+}
+
+fn collect_device_info() -> String {
+    let mut info = String::new();
+
+    if let Ok(output) = Command::new("uname").arg("-a").output() {
+        info.push_str(&format!(
+            "uname={}\n",
+            String::from_utf8_lossy(&output.stdout).trim()
+        ));
+    }
+
+    for (key, prop) in [
+        ("device", "ro.product.device"),
+        ("build", "ro.build.display.id"),
+        ("android", "ro.build.version.release"),
+    ] {
+        if let Ok(output) = Command::new("getprop").arg(prop).output() {
+            info.push_str(&format!(
+                "{}={}\n",
+                key,
+                String::from_utf8_lossy(&output.stdout).trim()
+            ));
+        }
+    }
+
+    // KSU version
+    if let Ok(ver) = fs::read_to_string("/data/adb/ksu/version") {
+        info.push_str(&format!("ksu={}\n", ver.trim()));
+    }
+
+    // Module version from module.prop
+    if let Ok(prop) = fs::read_to_string("/data/adb/modules/tricky_store/module.prop") {
+        for line in prop.lines() {
+            if let Some(ver) = line.strip_prefix("version=") {
+                info.push_str(&format!("module={}\n", ver.trim()));
+                break;
+            }
+        }
+    }
+
+    info
+}
+
+fn read_file_bytes(path: &str) -> Option<Vec<u8>> {
+    let mut buf = Vec::new();
+    File::open(path).ok()?.read_to_end(&mut buf).ok()?;
+    Some(buf)
+}
+
+fn epoch_millis() -> u64 {
+    SystemTime::now()
+        .duration_since(UNIX_EPOCH)
+        .map(|d| d.as_millis() as u64)
+        .unwrap_or(0)
+}
+
+pub fn execute_dump() -> Result<(), Box<dyn std::error::Error>> {
+    let _lock = FlockGuard::acquire()?;
+
+    let _ = fs::create_dir_all(DUMP_DIR);
+    let zip_name = format!("{}.zip", random_name(8));
+    let zip_path = format!("{}/{}", DUMP_DIR, zip_name);
+
+    let zip_file = File::create(&zip_path)?;
+    let mut zip = zip::ZipWriter::new(zip_file);
+    let options =
+        zip::write::SimpleFileOptions::default().compression_method(zip::CompressionMethod::Deflated);
+
+    let mut file_count = 0u32;
+
+    // Log files
+    let log_files = [
+        "certgen.log",
+        "certgen.log.1",
+        "certgen.log.2",
+        "certgen.log.3",
+        "certgen.log.4",
+    ];
+    for name in &log_files {
+        let path = format!("{}/{}", LOG_DIR, name);
+        if let Some(data) = read_file_bytes(&path) {
+            zip.start_file(*name, options)?;
+            zip.write_all(&data)?;
+            file_count += 1;
+        }
+    }
+
+    // Logcat
+    let logcat = collect_logcat("TEESimulator");
+    if !logcat.is_empty() {
+        zip.start_file("logcat-teesimulator.log", options)?;
+        zip.write_all(&logcat)?;
+        file_count += 1;
+    }
+
+    // Config files
+    for name in ["tee_status.txt", "security_patch.txt"] {
+        let path = format!("{}/{}", BASE_DIR, name);
+        if let Some(data) = read_file_bytes(&path) {
+            zip.start_file(name, options)?;
+            zip.write_all(&data)?;
+            file_count += 1;
+        }
+    }
+
+    // Device info
+    let device_info = collect_device_info();
+    if !device_info.is_empty() {
+        zip.start_file("device-info.txt", options)?;
+        zip.write_all(device_info.as_bytes())?;
+        file_count += 1;
+    }
+
+    // Manifest
+    let manifest = serde_json::json!({
+        "timestamp": epoch_millis(),
+        "version": env!("CARGO_PKG_VERSION"),
+        "files": file_count,
+    });
+    zip.start_file("manifest.json", options)?;
+    zip.write_all(manifest.to_string().as_bytes())?;
+
+    zip.finish()?;
+
+    let zip_size = fs::metadata(&zip_path).map(|m| m.len()).unwrap_or(0);
+    fs::write(DUMP_PATH_FILE, &zip_path)?;
+
+    let result = serde_json::json!({
+        "zip": zip_path,
+        "size": zip_size,
+        "files": file_count + 1, // +1 for manifest
+    });
+    println!("{}", result);
+
+    tracing::info!(path = %zip_path, size = zip_size, "diagnostic dump created");
+
+    Ok(())
+}
@@ -1,5 +1,7 @@
 mod kmsg;
 mod rotating;
+pub mod sysfs;
+pub mod dump;

 use std::path::Path;
 use tracing_subscriber::{layer::SubscriberExt, util::SubscriberInitExt, EnvFilter};
@@ -0,0 +1,38 @@
+use std::fs;
+use std::path::Path;
+
+const VERBOSE_MARKER: &str = "/data/adb/tricky_store/.verbose";
+
+pub fn is_verbose() -> bool {
+    Path::new(VERBOSE_MARKER).exists()
+}
+
+pub fn set_verbose_marker(enabled: bool) -> Result<(), Box<dyn std::error::Error>> {
+    if enabled {
+        if let Some(parent) = Path::new(VERBOSE_MARKER).parent() {
+            fs::create_dir_all(parent)?;
+        }
+        fs::write(VERBOSE_MARKER, "")?;
+    } else if Path::new(VERBOSE_MARKER).exists() {
+        fs::remove_file(VERBOSE_MARKER)?;
+    }
+    Ok(())
+}
+
+pub fn enable() -> Result<(), Box<dyn std::error::Error>> {
+    set_verbose_marker(true)?;
+    tracing::info!("verbose logging enabled via marker file");
+    Ok(())
+}
+
+pub fn disable() -> Result<(), Box<dyn std::error::Error>> {
+    set_verbose_marker(false)?;
+    tracing::info!("verbose logging disabled, marker file removed");
+    Ok(())
+}
+
+pub fn status() -> Result<(), Box<dyn std::error::Error>> {
+    let state = if is_verbose() { "enabled" } else { "disabled" };
+    tracing::info!(verbose = state, "verbose marker status");
+    Ok(())
+}
@@ -93,11 +93,6 @@ pub struct CertGenParams {
    pub caller_nonce: bool,
    pub unlocked_device_required: bool,
    pub no_auth_required: bool,
-
-    /// Calling app UID, used only to key diagnostic log lines to the requesting app.
-    pub uid: i32,
-    /// Mirrors the APK debug variant; gates the produced-extension dump so release stays quiet.
-    pub debug_logging: bool,
 }

 pub struct GeneratedKeyPair {
@@ -27,7 +27,6 @@ REBOOT=false
 VERIFY=false
 BUILD_RUST=false
 CLEAR_KEYS=false
-CLEAR_LOGS=false
 TRACE=false
 ROOT_PROVIDER="ksu"

@@ -53,7 +52,6 @@ Deploy options:
  --deploy           Push ZIP to device and install
  --reboot           Reboot device after install
  --clear-keys       Clear persistent_keys before deploy
-  --clear-logs       Clear per-UID diagnostic logs before deploy
  --verify           Run logcat verification after deploy
  --root PROVIDER    Root provider: ksu (default), magisk, apatch

@@ -75,7 +73,6 @@ while [[ $# -gt 0 ]]; do
        --verify)     VERIFY=true; shift ;;
        --rust)       BUILD_RUST=true; shift ;;
        --clear-keys) CLEAR_KEYS=true; shift ;;
-        --clear-logs) CLEAR_LOGS=true; shift ;;
        -v|--verbose) TRACE=true; shift ;;
        --root)       ROOT_PROVIDER="$2"; shift 2 ;;
        --help|-h)    usage ;;
@@ -156,11 +153,6 @@ deploy_zip() {
        adb shell "rm -rf /data/adb/tricky_store/persistent_keys/*" 2>/dev/null || true
    fi

-    if [[ "$CLEAR_LOGS" == true ]]; then
-        bold "==> Clearing per-UID diagnostic logs"
-        adb shell "rm -rf /data/media/0/TEESimulator /data/local/tmp/teesim" 2>/dev/null || true
-    fi
-
    bold "==> Deploying $name"
    adb push "$zip" /data/local/tmp/module.zip
    adb shell "su -c '$INSTALL_CMD /data/local/tmp/module.zip'"
@@ -1,22 +0,0 @@
-package android.security.maintenance;
-
-import android.os.IBinder;
-
-/**
- * Compile-time stub for the hidden keystore2 maintenance binder
- * ({@code android.security.maintenance.IKeystoreMaintenance}).
- *
- * <p>This module is a {@code compileOnly} dependency, so the real framework class
- * (which carries the actual {@code TRANSACTION_*} codes) is loaded at runtime. We
- * only need the {@link #DESCRIPTOR} token to parse the transaction parcel and the
- * inner {@code Stub} class so {@code getTransactCode} can reflect the real codes.
- */
-public interface IKeystoreMaintenance {
-    String DESCRIPTOR = "android.security.maintenance.IKeystoreMaintenance";
-
-    class Stub {
-        public static IKeystoreMaintenance asInterface(IBinder b) {
-            throw new UnsupportedOperationException("STUB!");
-        }
-    }
-}