ALT Linux Bugzilla – Full Text Bug Listing
| Summary: | Crash | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Sisyphus | Reporter: | Sergey V Turchin <zerg> | ||||
| Component: | mdadm | Assignee: | Олег Соловьев <mcpain> | ||||
| Status: | CLOSED FIXED | QA Contact: | qa-sisyphus | ||||
| Severity: | blocker | ||||||
| Priority: | P5 | CC: | antohami, mcpain, rider, shaba, vitty | ||||
| Version: | unstable | ||||||
| Hardware: | x86_64 | ||||||
| OS: | Linux | ||||||
| Bug Depends on: | |||||||
| Bug Blocks: | 46625 | ||||||
| Attachments: |
|
||||||
|
Description
Sergey V Turchin
2025-03-14 13:59:30 MSK
- Доктор, почему меня все игнорируют?! - Следующий. Починка элементарная. Есть какие-то препятствия? Блок на выпуск дистрибутивов. Отдел тестирования уже нашёл пару падений инсталлятора, о которых поставил мантерйнера в известность. А что за версия mdadm? Бага повешена на sisyphus. Указанный коммит в пакете 4.3-alt1 собран 24 июня 2024. Сейчас в сизифе версия 4.4, и это исправление в нем также присутствует. @zerg распиши подробнее что за краш у тебя происходит ss->ss не провеояется. В этом месте всегда крэш, если его нет. (In reply to Alexey Shabalin from comment #5) > А что за версия mdadm? > Бага повешена на sisyphus. Указанный коммит в пакете > 4.3-alt1 собран 24 июня 2024. > Сейчас в сизифе версия 4.4, и это исправление в нем также присутствует. Здесь нет исправления, есть только падающий с грохотом mdadm: https://git.altlinux.org/gears/m/mdadm.git?p=mdadm.git;a=blob;f=mdadm.c;h=7d3b656bc6da16e5c9458d7a12626e5e327d8ff5;hb=HEAD#l1700 Указатель ss->ss тоже нужно проверять перед разыменованием. (In reply to Alexey Shabalin from comment #5) > Сейчас в сизифе версия 4.4, и это исправление в нем также присутствует. В нём баг. (In reply to Олег Соловьев from comment #8) > (In reply to Alexey Shabalin from comment #5) > > А что за версия mdadm? > > Бага повешена на sisyphus. Указанный коммит в пакете > > 4.3-alt1 собран 24 июня 2024. > > Сейчас в сизифе версия 4.4, и это исправление в нем также присутствует. > > Здесь нет исправления, есть только падающий с грохотом mdadm: > https://git.altlinux.org/gears/m/mdadm.git?p=mdadm.git;a=blob;f=mdadm.c; > h=7d3b656bc6da16e5c9458d7a12626e5e327d8ff5;hb=HEAD#l1700 > > Указатель ss->ss тоже нужно проверять перед разыменованием. Патч есть ? В апстрим баг завесили ? (In reply to Anton Farygin from comment #10) > Патч есть ? В апстрим баг завесили ? Все карты мантейнеру в руки. Ошибка элементарная. (In reply to Sergey V Turchin from comment #11) > (In reply to Anton Farygin from comment #10) > > Патч есть ? В апстрим баг завесили ? > Все карты мантейнеру в руки. Ошибка элементарная. механизм воспроизведения есть ? (Ответ для Anton Farygin на комментарий #12) > механизм воспроизведения есть ? Там элементарного здравого смысла достаточно. Ща патч сделаю. Created attachment 18019 [details]
fix-crash.patch
Если только, там всё запутаннее и надо репортить в апстрим, чтоб перелопатили свой код. Или после free_super() надо ещё раз проверять ss перед тем, как сделать ему free(). Олег, проверь, плиз. Нам жто быстрее будет сделать, а то выпуск дистрибутивов тормозит. (In reply to Sergey V Turchin from comment #17) > Олег, проверь, плиз. Нам жто быстрее будет сделать, а то выпуск > дистрибутивов тормозит. Что проверить? Я делал revert и этого хватило систему поставить mdadm-4.4-alt2 -> sisyphus: Wed Mar 19 2025 Alexey Shabalin <shaba@altlinux> 4.4-alt2 - fix crash (ALT#53454) - backport fixes from upstream master Спасибо! Всё ещё падает из-за double free, патч нужно совсем откатить
if (ss) {
ss->ss->free_super(ss);
free(ss); // <= SIGSEGV, память по адресу уже освобождена в Assemble.c:786
}
# Сначала нужно разобрать все массивы:
/sbin/mdadm -S --scan
/sbin/mdadm --assemble --metadata=imsm /dev/md127 /dev/sdb /dev/sdc
(Ответ для Олег Соловьев на комментарий #21) > Всё ещё падает из-за double free, патч нужно совсем откатить Имеется ввиду, надо мой откатить коммит https://web.git.kernel.org/pub/scm/utils/mdadm/mdadm.git/commit/?id=e62a561ee8b7157a2390eab215dcef6240bd7b03 > Имеется ввиду, надо мой откатить коммит
мой патч и коммит
mdadm-4.4-alt3 -> sisyphus: Thu Mar 20 2025 Alexey Shabalin <shaba@altlinux> 4.4-alt3 - Revert "Fix memory leak in file mdadm" (ALT#53454) Сообщите кто-нибудь в апстрим, плиз. |
