Flipper-Android-App 文件路径处理问题分析与修复

问题概述

在Flipper-Android-App项目中，发现了一个涉及文件操作的处理问题。该问题存在于处理通过android.intent.action.SEND意图共享文件的逻辑中，可能会被利用在应用内部存储中执行非预期的文件写入或删除操作。

技术背景

Android应用通过Intent机制实现组件间通信，其中ACTION_SEND意图用于在不同应用间共享内容。当Flipper应用接收到此类意图时，会尝试将共享文件复制到内部缓存目录。问题出在文件路径处理环节，应用未对文件名进行充分校验。

问题详情

问题位于DeepLinkFileUriCopy.kt文件的buildInternalFile方法中。该方法从ContentResolver获取文件名时，直接使用了未经验证的外部输入：

val filename = uri.filename(contentResolver) ?: System.currentTimeMillis().toString()
val temporaryFile = File(cacheDir, filename)

可能会构造包含特殊字符（如../）的文件名，使文件操作指向应用内部存储的非预期位置。这导致两个主要风险：

1. 非预期文件删除：如果目标文件已存在，代码会先执行删除操作
2. 非预期文件写入：可能会控制写入内容到非预期位置

可能的影响

此问题可能造成多种风险，具体取决于使用场景：

1. 数据异常：删除关键配置文件导致应用功能异常
2. 数据不一致：注入非预期配置或数据文件
3. 系统稳定性：结合其他问题可能影响系统稳定性

修复方案

修复措施应包含以下关键点：

1. 文件名处理：移除所有特殊字符
2. 路径验证：确保目标文件位于预期目录内
3. 异常处理：对非法路径操作进行记录和阻断

示例修复代码应添加路径校验：

fun sanitizeFilename(filename: String): String {
    return filename.replace("../", "").replace("/", "_")
}

开发建议

对于Android开发者，处理外部文件时应注意：

1. 始终验证和处理外部输入的文件名
2. 限制文件操作在特定目录内
3. 对ContentProvider交互保持谨慎
4. 实施最小权限原则

此案例再次证明了输入处理在移动应用开发中的重要性，特别是在处理跨应用通信时更需谨慎。