V1.4.10: 修复广告过滤失效（gz.ac16.vip 广告被当成歌曲下载播放）

现场事故（新设备 SM-S9180 / Android 16，WebView 正常 → 走 JS 取址）: 277 JS [KtvApi] URL OK: 7303235 ls=1 via http://w.w345.my 278 点歌编号=7303235 标题=情歌(HD) 请求源=http://w.w345.my 状态=成功 urlHost=gz.ac16.vip 279 PLAY id=7303235 title=情歌(HD) src=local 即 gz.ac16.vip 的广告视频被当成歌曲下载、缓存并播放。三处漏洞叠加: 1. ktv_api.js 的 isDemoUrl 不认识广告域名；且"ts 文件名与编号比对"用的是 /\/(\d+)\.ts/，只匹配纯数字名 —— my_ad_video.ts 这类非数字名字直接放行； 2. SongOkDownloadManager.buildDownloadUrl 的**成功分支完全没有广告校验** （AdVideoFilter.isAdVideo 全项目只在"缓存地址"与"原生解析"两处被调用）； 3. SongApiClient.fetchFromDb 只做 isLikelyDemoUrl（纯 IP / r2.dev），挡不住广告域名。且**内容校验挡不住广告**: 该广告通过 SongFileValidator（≥6MB/≥60s/TS 包完整），被 isDownloaded() 一直复用，广告会反复播。修复（单一闸门，全链路把关）: - AdVideoFilter 新增 isTrustedSongUrl(url, musicno)：非 http(s)、命中广告特征（路径 /ad_files/、my_ad_video、**域名 *.ac16.vip**）、或 URL 里 .ts 文件名不等于点歌编号 → 一律不可信。真源恒为 /<musicno>.ts（已实测 txog.ktvsky.com 与 download.origjoy.com 两个 CDN 域共 6 首全部相符）。 - 该闸门接入全部出入口: JS isDemoUrl（同步加固 + KTV_BRIDGE_API 8→9）、 SongUrlResolver.isValidSongUrl、SongApiClient.fetchFromDb、以及最终兜底 SongOkDownloadManager.buildDownloadUrl（成功/缓存两条分支）。 - 判定不可信时**同时删除已落盘的歌曲文件**（purgeCachedSongFile），否则 isDownloaded() 会一直复用那个广告文件。 - 新增 SongOkDownloadManager.purgeLegacySongCacheOnce(): 升级后一次性清理历史遗留的可疑歌曲缓存（marker: filesDir/song_cache_purged_v1）。只是让歌曲重新下载，不动用户数据。 - 新增回归测试 AdVideoFilterTest（6 例，用日志里的真实 URL 形态，含两个真源 CDN 域）；该测试当场抓到本修复自身的一个 bug：正则把扩展名也捕获进去导致真源被误杀，已修正。版本 1.4.9(29) 到 1.4.10(30)；签名证书不变。
2026-09-26 21:17:04 +08:00
parent b3b17210f1
commit 669766ba60
10 changed files with 234 additions and 23 deletions
@@ -0,0 +1,66 @@
+package com.local.ktv;
+
+import org.junit.Test;
+
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertTrue;
+
+/**
+ * 广告地址拦截的回归测试。
+ *
+ * 对应 2026-09-26 现场事故：新设备（WebView 正常 → 走 JS 取址）点歌 7303235（情歌 HD）
+ * 拿到 `gz.ac16.vip` 的广告视频，被当成歌曲下载并播放。当时的漏洞有三处：
+ * 1. JS 的 isDemoUrl 不认识该广告域名，且只比对"纯数字文件名"，非数字名字直接放行；
+ * 2. Kotlin 的 buildDownloadUrl 成功分支**完全没做广告校验**；
+ * 3. DB 回退路径只做 isLikelyDemoUrl，也挡不住广告域名。
+ * 这里用日志里的**真实 URL 形态**锁住入参校验行为。
+ */
+public final class AdVideoFilterTest {
+
+    /** 真源地址：两个 CDN 域实测文件名都严格等于点歌编号，必须放行。 */
+    @Test
+    public void acceptsRealSourceUrls() {
+        assertTrue(AdVideoFilter.isTrustedSongUrl(
+                "http://txog.ktvsky.com/264/lvbr/480/E/ts/47.1/202209220333/480p/7303235.ts?token=526e505e",
+                "7303235"));
+        assertTrue(AdVideoFilter.isTrustedSongUrl(
+                "http://download.origjoy.com/264/lvbr/480/E/ts/39.6/202309070206/480p/7445418.ts?token=abc",
+                "7445418"));
+    }
+
+    /** 广告域名：即使文件名伪装成正确的点歌编号，也必须拦下。 */
+    @Test
+    public void rejectsAdHostEvenWithMatchingName() {
+        assertFalse(AdVideoFilter.isTrustedSongUrl("http://gz.ac16.vip/ad/7303235.ts", "7303235"));
+        assertFalse(AdVideoFilter.isTrustedSongUrl("http://cdn.ac16.vip/x/7303235.ts", "7303235"));
+    }
+
+    /** 广告文件名的经典形态：非数字名字，旧规则的正则匹配不到。 */
+    @Test
+    public void rejectsAdFileName() {
+        assertFalse(AdVideoFilter.isTrustedSongUrl("http://example.com/ad_files/my_ad_video.ts", "7303235"));
+        assertFalse(AdVideoFilter.isTrustedSongUrl("http://example.com/x/my_ad_video.ts", "7303235"));
+    }
+
+    /** .ts 文件名与点歌编号不符（即使不命中任何广告域名）也视为不可信。 */
+    @Test
+    public void rejectsMismatchedFileName() {
+        assertFalse(AdVideoFilter.isTrustedSongUrl("http://example.com/x/9999999.ts", "7303235"));
+        assertFalse(AdVideoFilter.isTrustedSongUrl("http://example.com/x/promo.ts", "7303235"));
+    }
+
+    /** 非 http(s) 一律不可信。 */
+    @Test
+    public void rejectsNonHttpSchemes() {
+        assertFalse(AdVideoFilter.isTrustedSongUrl("file:///sdcard/x/7303235.ts", "7303235"));
+        assertFalse(AdVideoFilter.isTrustedSongUrl("ftp://example.com/7303235.ts", "7303235"));
+        assertFalse(AdVideoFilter.isTrustedSongUrl("", "7303235"));
+        assertFalse(AdVideoFilter.isTrustedSongUrl(null, "7303235"));
+    }
+
+    /** 没有 .ts 文件名时不做文件名比对（避免误杀非 ts 的真源）。 */
+    @Test
+    public void allowsNonTsUrlWithoutAdMarkers() {
+        assertTrue(AdVideoFilter.isTrustedSongUrl("http://cdn.example.com/path/stream?token=abc", "7303235"));
+    }
+}