Merge pull request #72 from SliDeeN/security-key

Security : Verify signature

Author: SliDeeN

app/src/main/java/com/mg4/control/update/ApkInstaller.kt

@@ -31,17 +31,14 @@ object ApkInstaller {
             return "APK manquant ou vide : ${apkFile.absolutePath}"
         }
 
-        // [DÉSACTIVÉ 2026-08-05] Contrôle de signature (T-901) mis en pause : il refusait
-        // TOUTE mise à jour OTA sur AAOS 9, y compris avec un APK signé de la même clé
-        // plateforme en v1+v2+v3. Cause non encore élucidée (getPackageArchiveInfo ne
-        // remonte probablement pas la signature de l'archive sur cette ROM).
-        // Garde-fous restants : Android refuse une MAJ signée d'une autre clé, et
-        // ApkUrlPolicy restreint toujours l'origine du téléchargement (github/gitlab, https).
-        // Pour réactiver : décommenter le bloc ci-dessous.
-        // if (!ApkSignatureVerifier.matchesRunningApp(context, apkFile)) {
-        //     apkFile.runCatching { delete() }
-        //     return "Signature non conforme — APK rejeté et supprimé"
-        // }
+        // [RÉACTIVÉ 2026-08-05] Contrôle de signature (T-901). Les échecs précédents ont eu
+        // lieu avec une app installée signée à la main (v1 seul) face à un APK CI (v2/v3) :
+        // on retente avec les DEUX côtés produits par la CI, donc signés à l'identique.
+        // L'app tourne en uid.system : on n'installe que ce qui est signé par notre clé.
+        if (!ApkSignatureVerifier.matchesRunningApp(context, apkFile)) {
+            apkFile.runCatching { delete() }
+            return "Signature non conforme — APK rejeté et supprimé"
+        }
 
         // ── Stratégie 1 : pm install depuis le stockage externe ───────────────
         val extDir = context.getExternalFilesDir(null)

app/src/main/java/com/mg4/control/update/UpdateDialogManager.kt

@@ -172,6 +172,20 @@ object UpdateDialogManager {
             .substringBefore('?')
             .ifBlank { "MG4Control-${info.versionName}.apk" }
 
+        // DownloadManager n'écrase JAMAIS un fichier existant : si "…-2.6.5.apk" est déjà là
+        // (téléchargement précédent de la même version), il écrit "…-2.6.5-1.apk". Or le code
+        // relit ensuite le nom d'origine → il vérifierait/installerait l'ANCIEN fichier.
+        // On supprime donc la cible avant d'enfiler la requête, pour garantir que le fichier
+        // téléchargé, le fichier vérifié et le fichier installé soient bien le même.
+        val target = File(
+            Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_DOWNLOADS),
+            fileName
+        )
+        if (target.exists()) {
+            val deleted = target.runCatching { delete() }.getOrDefault(false)
+            Log.i(TAG, "Ancien APK homonyme trouvé (${target.name}) → suppression=$deleted")
+        }
+
         val request = DownloadManager.Request(Uri.parse(info.apkUrl)).apply {
             setTitle("MG4Control ${info.versionName}")
             setDescription(activity.getString(R.string.update_downloading))
@@ -210,19 +224,18 @@ object UpdateDialogManager {
                 when (status) {
                     DownloadManager.STATUS_SUCCESSFUL -> {
                         progressBar.progress = 100
-                        // [DÉSACTIVÉ 2026-08-05] Contrôle de signature (T-901) mis en pause :
-                        // il refusait TOUTE mise à jour OTA sur AAOS 9, même avec un APK
-                        // signé de la même clé plateforme en v1+v2+v3. Voir ApkInstaller.
-                        // Pour réactiver : décommenter le bloc ci-dessous.
-                        // val downloaded = File(
-                        //     Environment.getExternalStoragePublicDirectory(
-                        //         Environment.DIRECTORY_DOWNLOADS), fileName)
-                        // if (!ApkSignatureVerifier.matchesRunningApp(activity, downloaded)) {
-                        //     downloaded.runCatching { delete() }
-                        //     tvStatus.setText(R.string.update_error_signature)
-                        //     btnCancel.setText(R.string.update_close)
-                        //     break
-                        // }
+                        // [RÉACTIVÉ 2026-08-05] Contrôle de signature (T-901) — voir ApkInstaller.
+                        // L'APK est dans un dossier public : on vérifie qu'il est signé
+                        // par NOTRE clé avant d'inviter l'utilisateur à l'installer.
+                        val downloaded = File(
+                            Environment.getExternalStoragePublicDirectory(
+                                Environment.DIRECTORY_DOWNLOADS), fileName)
+                        if (!ApkSignatureVerifier.matchesRunningApp(activity, downloaded)) {
+                            downloaded.runCatching { delete() }
+                            tvStatus.setText(R.string.update_error_signature)
+                            btnCancel.setText(R.string.update_close)
+                            break
+                        }
                         // Nettoie les anciens APK dans Téléchargements (garde les 5 plus récents)
                         ApkCleanup.cleanIfNeeded()
                         // Ouvre le dossier Téléchargements dans le gestionnaire AAOS