https://web.git.kernel.org/pub/scm/utils/mdadm/mdadm.git/commit/?id=e62a561ee8b7157a2390eab215dcef6240bd7b03 2-й указатель тоже надо проверять, видимо.
- Доктор, почему меня все игнорируют?! - Следующий.
Починка элементарная. Есть какие-то препятствия?
Блок на выпуск дистрибутивов.
Отдел тестирования уже нашёл пару падений инсталлятора, о которых поставил мантерйнера в известность.
А что за версия 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)
p11 https://packages.altlinux.org/ru/tasks/378740/
Сообщите кто-нибудь в апстрим, плиз.
