ALT Linux Bugzilla – #53454

- Доктор, почему меня все игнорируют?!
- Следующий.

Починка элементарная. Есть какие-то препятствия?

Блок на выпуск дистрибутивов.

Отдел тестирования уже нашёл пару падений инсталлятора, о которых поставил мантерйнера в известность.

А что за версия 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/

Сообщите кто-нибудь в апстрим, плиз.