From 460a8c2430ff9c6fc4eda152f9f28044e244348b Mon Sep 17 00:00:00 2001 From: euxane Date: Fri, 4 Oct 2024 23:42:32 +0200 Subject: config: restrict automatic storage migration to networks subdir Avoiding aborting automatic migration when other system files are already present at the root app files target directory (such as "profileInstalled" or "rList". --- app/src/main/java/org/pacien/tincapp/context/StorageMigrator.kt | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) (limited to 'app/src/main') diff --git a/app/src/main/java/org/pacien/tincapp/context/StorageMigrator.kt b/app/src/main/java/org/pacien/tincapp/context/StorageMigrator.kt index 6bde2f6..b920522 100644 --- a/app/src/main/java/org/pacien/tincapp/context/StorageMigrator.kt +++ b/app/src/main/java/org/pacien/tincapp/context/StorageMigrator.kt @@ -1,6 +1,6 @@ /* * Tinc Mesh VPN: Android client and user interface - * Copyright (C) 2017-2023 Euxane P. TRAN-GIRARD + * Copyright (C) 2017-2024 Euxane P. TRAN-GIRARD * * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by @@ -40,11 +40,12 @@ class StorageMigrator { } private fun migrateConfigurationDirectory() { - val oldConfigDir = context.getExternalFilesDir(null) - if (oldConfigDir == null || oldConfigDir.listFiles().isNullOrEmpty()) return // nothing to do + val externalStorageDir = context.getExternalFilesDir(null) ?: return + val oldConfigDir = File(externalStorageDir, "networks") + if (!oldConfigDir.exists()) return try { - val newConfigDir = context.filesDir + val newConfigDir = AppPaths.confDir() log.info( "Migrating files present in old configuration directory at {} to {}", oldConfigDir.absolutePath, -- cgit v1.2.3