AskaEth/MeowOS-For-R3P2
1
0 Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(ncm §30): 真机四连症状——JSON unescape + 封面流式 + 歌单豆腐/分隔符 tofu + 首屏诊断日志

Browse Source 
症状/根因/修法（0 真实 NCM 业务请求；宿主+qemu 全绿）：
- ③ LRC 挤一行/时间戳未解析：ncm_json_get_str_at 不做 JSON 转义还原，
  lrc.lyric 的 \n 转义原样进串 → lrc_parse 无真实换行整段一行。修：ncm_json.c 加
  json_unescape（RFC8259 全转义 + \uXXXX/代理对），get_str_at 返回前还原。
- ⑤ 封面全 failed→give up：cover.c 静态 g_buf 288KB < 封面 1.34MB → NCM_ERR_FULL。
  设备 RAM 仅 56MB。修：cover.c 改流式写文件（ncm_http_download cb→ncm_cache_write +
  meow_img_load_file 临时文件），NCM_COVER_MAX 8MB 改下载上限（非缓冲，bss 未涨）。
- ① 162 首歌单所有条目「三个豆腐」+「加载中」：占位符 ···（U+00B7）在 default 字体缺
  该字形 → 3 豆腐（font.idx glyph 扫实证 U+00B7 MISSING）。修：占位 ···→...、全 App
  · 分隔符→/；playlist_worker 首屏打 first-screen 诊断日志。
- ② 返回首页歌单列表 1 秒方框：字体缺 U+00B7/特殊字形（◢◤/Kannada）tofu；瞬态疑 glyph
  atlas 冷缓存（SDK 内部未深挖）。修：·→/。
验证：test_ncm_core(+test_json_unescape, host+qemu) / listvirt44 / dispatch / box42 /
scroll17 / anim32 / widgets / overlay_suppress / sdk_input_keys / theme / vol / ime_fix /
pinyin / notif_gesture / notif_open / sdk_fixes / sb_collapse / ui_audio_host / notify_ipc /
notify_integration + playlist_race_sim / window_sim / search_sim / cache_evict_sim(1-9) /
player_flow_sim(P1-12) / stream_sim(16+2resume) 全 PASS；mipsel -Wall -Wextra 0 warning。
新 md5：ncm d91cd906、ncm_host b731654a、test_ncm_core_host 7a38194c。

（症状④ VOL_DOWN×3 杀 App 的 launcher 源修复 app_launcher.c 门控 !a->framework 已随
 并发 commit 54a6e82 存入 HEAD；launcher 二进制 03dfcf76 留主会话/固件打包。）
注：build_ncm_host.sh 补 SDK 292/295 的 anim/box/scroll 源（宿主链接）。
This commit is contained in:
AskaEth committed 2026-09-02 02:25:37 +08:00
1 parent 54a6e828b7
commit 3397a702aa
15 files changed
+384 -88
Download Patch File Download Diff File Expand all files Collapse all files

No files matched your search

patch_v2/ncm_app/app/logic/cover.c
+101 -67
View File
@@ -1,6 +1,6 @@
/* logic/cover.c — 封面下载（HTTPS）+ 限幅 + 解码缓存（单张，URL 变更即换；后台线程）。
*
* 内存纪律：下载限幅 NCM_COVER_MAX（~288KB）；解码后的 RGBA 保持单张（换封面先 free 旧再解码）。
* 内存纪律：下载限幅 NCM_COVER_MAX（字节上限）；解码后的 RGBA 保持单张（换封面先 free 旧再解码）。
* 线程模型：一个持久的 cover 工作线程负责「目标 URL → 下载 → 解码 → 换入」；UI 线程只经
* cover_request(url)（提交目标）与 cover_get()/cover_changed()（读取/判变）+ cover_ack()（消费）。
* 工作线程不碰 meow_ui（绘制在 UI 线程 tick 判 cover_changed 后触发 invalidate）。
@@ -13,6 +13,12 @@
* 失败写 WARN 日志（排查资产），不再无声死等。
* - 绘制安全：cover_get() 持锁返回、cover_put() 解锁——旧版 UI 绘制期 worker 可 free
* 同一 meow_img（use-after-free）；调用方须在 cover_get 后配对 cover_put。
*
* §30 修复（真机封面全加载失败）：旧版 `static g_buf[NCM_COVER_MAX(288KB)]` 作为下载缓冲，
* 网易云封面全尺寸实测 ~1.34MB（>288KB）→ ncm_http 回 NCM_ERR_FULL → 下载失败×8 放弃。
* 且设备 RAM 仅 56MB，不能再涨静态 bss。改为**流式写文件**：ncm_http_download 的写回调直接
* 把 body 写进 ncm_cache 写者临时文件（write(2) 直写，无 stdio 缓冲），meow_img_load_file 从
* 临时文件解码，commit 落盘。全程零封面大缓冲、无 bss 上涨；NCM_COVER_MAX 仅作下载字节上限。
*/
#include "../ncm_app_internal.h"
#include <stdio.h>
@@ -32,26 +38,6 @@ static volatile int g_gen; /* 每次换入 +1 */
static int g_acked; /* UI 已消费的版本 */
static char g_cover_path[512]; /* 封面原始文件路径（供 launcher） */
static unsigned char g_buf[NCM_COVER_MAX]; /* 单张下载缓冲（常驻） */
static meow_img *decode_cover(const char *url, size_t *out_len)
{
if (!url || !url[0]) return NULL;
ncm_http_req_t req;
memset(&req, 0, sizeof req);
req.url = url;
req.method = "GET";
req.extra_hdrs = "User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) "
"AppleWebKit/537.36 (KHTML, like Gecko) Chrome/124.0.0.0 Safari/537.36";
req.verify_ca = NCM_HTTP_VERIFY_DEFAULT;
req.timeout_ms = 10000;
ncm_http_resp_t resp;
int rc = ncm_http_request(&req, (char *)g_buf, sizeof g_buf, &resp);
if (rc != NCM_OK || resp.status != 200 || resp.body_len == 0) return NULL;
if (out_len) *out_len = resp.body_len;
return meow_img_load_mem((const void *)resp.body, (int)resp.body_len);
}
/* 封面缓存键：URL 的 djb2 哈希（稳定性由 URL 决定——同一专辑封面 URL 名即可命中）。
* 走 ncm_cache（LRU 口径沿用 ncm_cache，见 REPORT §14.2）：键进 ncm_cache_path => 文件
* `song_cov<hex>.jpg`，与 mp3（song_<id>.mp3）同池，lookup/commit 刷新 mtime（LRU）。 */
@@ -62,28 +48,70 @@ static void cover_cache_id(const char *url, char *out, size_t out_sz)
snprintf(out, out_sz, "cov%lx", h);
}
/* 读整文件到 buf（< cap），成功返回 1，写入 *len。 */
static int read_file_into_buf(const char *path, unsigned char *buf, size_t cap, size_t *len)
/* 文件存在且非空（封面缓存命中用；streaming 后无需整读进内存）。 */
static int file_nonempty(const char *path)
{
FILE *f = fopen(path, "rb");
if (!f) return 0;
size_t n = fread(buf, 1, cap, f);
fclose(f);
if (n == 0) return 0;
if (len) *len = n;
return 1;
struct stat st;
return (path && stat(path, &st) == 0 && st.st_size > 0) ? 1 : 0;
}
/* 写 cover_current.jpg 供 launcher 媒体卡片读（兼容保留）。调用方须已持有 g_cm
* （早前版本在此内再 lock g_cm → 非递归锁自锁死——cover worker 永久卡死、封面永不出现）。 */
static void save_cover_raw(const char *url, size_t len)
/* 流式写回调：body 字节直接写进 ncm_cache 写者临时文件（write(2) 直写）。ctx=cover_sink_t*。 */
typedef struct { ncm_cache_writer_t *w; size_t total; } cover_sink_t;
static int cover_sink_write(void *ctx, const unsigned char *data, size_t len)
{
if (!url || len == 0) return;
cover_sink_t *s = (cover_sink_t *)ctx;
if (ncm_cache_write(s->w, data, len) != 0) return -1;
s->total += len;
return 0;
}
/* 下载 url 到 cache 写者 w（流式，写进其临时文件），成功后从临时文件解码并返回 img。
* 失败返回 NULL（不 commit，调用方 abort）。写入 *out_len（body 字节数）。 */
static meow_img *download_cover(const char *url, ncm_cache_writer_t *w, size_t *out_len)
{
ncm_http_req_t req;
memset(&req, 0, sizeof req);
req.url = url;
req.method = "GET";
req.extra_hdrs = "User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) "
"AppleWebKit/537.36 (KHTML, like Gecko) Chrome/124.0.0.0 Safari/537.36";
req.verify_ca = NCM_HTTP_VERIFY_DEFAULT;
req.timeout_ms = 15000;
ncm_http_resp_t resp;
cover_sink_t sink = { w, 0 };
int rc = ncm_http_download(&req, cover_sink_write, &sink, NCM_COVER_MAX, &resp);
if (rc != NCM_OK || resp.status != 200 || sink.total == 0) return NULL;
if (out_len) *out_len = sink.total;
/* 从写者临时文件解码（未 commit，文件仍在 .tmp；解码到内存 RGBA，与落盘无关） */
const char *tmp = ncm_cache_writer_tmp_path(w);
if (!tmp) return NULL;
return meow_img_load_file(tmp);
}
/* 封面原始文件路径写入（供 launcher 媒体卡片读；兼容保留）。调用方须已持有 g_cm。
* 从 cache 写者临时文件把已下载字节复制到 cover_current.jpg（早前版本在此内再 lock g_cm
* → 非递归锁自锁死——cover worker 永久卡死、封面永不出现）。 */
static void save_cover_raw(const char *path, size_t len)
{
if (!path || len == 0) return;
if (g_st.cache_dir[0]) {
char path[NCM_CACHE_DIR_MAX + 32]; /* >512：宿主 glibc fortify 对 %s 展开宽证明可放 */
snprintf(path, sizeof path, "%s/cover_current.jpg", g_st.cache_dir);
ncm_store_write_file(path, g_buf, len);
ncm_strlcpy(g_cover_path, path, sizeof g_cover_path);
char dst[NCM_CACHE_DIR_MAX + 32];
snprintf(dst, sizeof dst, "%s/cover_current.jpg", g_st.cache_dir);
/* 文件复制（源=临时文件，目标=cover_current.jpg）。 */
FILE *in = fopen(path, "rb");
FILE *out = NULL;
if (in) {
out = fopen(dst, "wb");
if (out) {
char buf[4096];
size_t n;
while ((n = fread(buf, 1, sizeof buf, in)) > 0)
if (fwrite(buf, 1, n, out) != n) { fclose(out); out = NULL; break; }
if (out) fclose(out);
}
fclose(in);
}
ncm_strlcpy(g_cover_path, dst, sizeof g_cover_path);
}
}
@@ -114,44 +142,51 @@ static void *cover_worker(void *arg)
char cpath[512];
if (g_st.ctx.cache.dir[0] && ncm_cache_lookup(&g_st.ctx.cache, key, ".jpg",
cpath, sizeof cpath) == NCM_OK &&
read_file_into_buf(cpath, g_buf, sizeof g_buf, &blen)) {
img = meow_img_load_mem(g_buf, (int)blen);
file_nonempty(cpath)) {
img = meow_img_load_file(cpath);
}
/* ② 未命中 → 下载解码；成功后落进 ncm_cache（LRU 池，超限由 ncm_cache_evict 淘汰） */
/* ② 未命中 → 流式下载解码；成功后落进 ncm_cache（LRU 池，超限由 ncm_cache_evict 淘汰） */
if (!img) {
blen = 0;
img = decode_cover(url, &blen);
if (!img) {
/* 失败：指数退避重试，连续 COVER_RETRY_MAX 次放弃（不无声死等；
* 目标保留期间 UI 每帧重请求由 cover_request 去重挡住）。 */
fails++;
if (fails >= COVER_RETRY_MAX) {
meow_log(MEOW_LOG_WARN, "cover: give up after %d fails url=%.64s", fails, url);
pthread_mutex_lock(&g_cm);
if (strcmp(g_tgt_url, url) == 0) g_tgt_url[0] = 0;
pthread_mutex_unlock(&g_cm);
backoff_ms = 0; fails = 0;
continue;
}
meow_log(MEOW_LOG_WARN, "cover: fetch failed (%d) url=%.64s retry in %dms",
fails, url, backoff_ms ? backoff_ms : 500);
if (g_app_quit) return NULL;
usleep(backoff_ms ? backoff_ms * 1000 : 500 * 1000);
backoff_ms = backoff_ms ? (backoff_ms * 2 > 4000 ? 4000 : backoff_ms * 2) : 500;
continue;
}
if (g_st.ctx.cache.dir[0]) {
/* 写前检查：池大小超限先按 LRU 淘汰（与 mp3 同池；封面约 200KB，先腾空间） */
ncm_cache_evict(&g_st.ctx.cache);
ncm_cache_writer_t *w = NULL;
if (ncm_cache_open(&g_st.ctx.cache, key, ".jpg", &w) == NCM_OK) {
ncm_cache_write(w, g_buf, blen);
ncm_cache_commit(w);
/* 写后检查：新封面可能把池推过上限 → 再按 LRU 淘汰最旧 */
blen = 0;
img = download_cover(url, w, &blen);
if (img) {
pthread_mutex_lock(&g_cm);
save_cover_raw(ncm_cache_writer_tmp_path(w), blen);
pthread_mutex_unlock(&g_cm);
if (ncm_cache_commit(w) != 0) { ncm_cache_abort(w); }
} else {
ncm_cache_abort(w);
}
ncm_cache_evict(&g_st.ctx.cache);
} else {
/* 打开写者失败（罕见）：退避重试，不空转 */
img = NULL;
}
}
}
if (!img) {
/* 失败：指数退避重试，连续 COVER_RETRY_MAX 次放弃（不无声死等；
* 目标保留期间 UI 每帧重请求由 cover_request 去重挡住）。 */
fails++;
if (fails >= COVER_RETRY_MAX) {
meow_log(MEOW_LOG_WARN, "cover: give up after %d fails url=%.64s", fails, url);
pthread_mutex_lock(&g_cm);
if (strcmp(g_tgt_url, url) == 0) g_tgt_url[0] = 0;
pthread_mutex_unlock(&g_cm);
backoff_ms = 0; fails = 0;
continue;
}
meow_log(MEOW_LOG_WARN, "cover: fetch failed (%d) url=%.64s retry in %dms",
fails, url, backoff_ms ? backoff_ms : 500);
if (g_app_quit) return NULL;
usleep(backoff_ms ? backoff_ms * 1000 : 500 * 1000);
backoff_ms = backoff_ms ? (backoff_ms * 2 > 4000 ? 4000 : backoff_ms * 2) : 500;
continue;
}
backoff_ms = 0; fails = 0;
pthread_mutex_lock(&g_cm);
@@ -160,7 +195,6 @@ static void *cover_worker(void *arg)
if (g_cover_img) meow_img_free(g_cover_img);
g_cover_img = img;
ncm_strlcpy(g_cover_url, url, sizeof g_cover_url);
save_cover_raw(url, blen);
g_gen++;
g_tgt_url[0] = 0; /* 目标达成即清：worker 转怠速（修忙等死循环，§18） */
} else {