[Bug] SDIO 驱动注销失败与节点内存泄漏风险

Aperta
#11,801 3 commenti 0 reazioni 1 assegnatario Vedi su GitHub

@Huoyanlifusu ci sta già lavorando.

Dal 15/9/2026.

Valutazione

Questa issue non è ancora stata valutata.

Descrizione

bug Component component: drivers in progress
RT-Thread Version

master e538ad76e475ba6026e563a9c605b58a1f2f9cc0

Affected area

Device drivers

Hardware/BSP vendor

Not applicable / Other

Architecture

Not applicable / Other

Board and hardware details

无特定版型要求

Develop Toolchain

GCC

Describe the bug

尽管多数 sdio 设备（比如 mmc/sd卡）不会热插拔，出于代码稳定角度考虑，建议进行这几处优化。

  1. 系统加载多个 sdio 驱动，在卸载驱动时，链表未及时退出，sd 可能会被下一轮循环覆盖为 NULL
    for (l = (&sdio_drivers)->next; l != &sdio_drivers; l = l->next)
    {
        sd = (struct sdio_driver *)rt_list_entry(l, struct sdio_driver, list);
        if (sd->drv != driver)
        {
            sd = RT_NULL;
        }
    }
  1. 目前释放 sd 指针动作在软件找到卡后才进行，有内存泄漏风险，这个指针的申请动作不依赖 SDIO 卡，因此释放也不需要依赖 SDIO 卡匹配到与否。
if (card != RT_NULL)
{
    driver->remove(card);
    rt_list_remove(&sd->list);
    rt_free(sd);
}
  1. 可选优化：使用 rt_list_for_each_entry 替代 rt_list_entry + for 的写法
Other additional context

No response

Lingua principale
C
Stelle
12.2k
Fork
5.4k
Merge medio
4g 12h
PR unite (30g)
40

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di RT-Thread/rt-thread

Tutte le issue di RT-Thread/rt-thread

Issue simili

Altre issue su C

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.