53454 2025-03-14 13:59:30 +0300 Crash 2025-03-21 09:43:57 +0300 1 1 4 Development Sisyphus mdadm unstable x86_64 Linux CLOSED FIXED P5 blocker --- 46625 1 zerg mcpain antohami mcpain rider shaba vitty qa-sisyphus oldest_to_newest 261040 0 zerg 2025-03-14 13:59:30 +0300 https://web.git.kernel.org/pub/scm/utils/mdadm/mdadm.git/commit/?id=e62a561ee8b7157a2390eab215dcef6240bd7b03 2-й указатель тоже надо проверять, видимо. 261143 1 zerg 2025-03-17 10:08:16 +0300 - Доктор, почему меня все игнорируют?! - Следующий. 261151 2 zerg 2025-03-17 10:58:55 +0300 Починка элементарная. Есть какие-то препятствия? 261248 3 zerg 2025-03-18 11:18:41 +0300 Блок на выпуск дистрибутивов. 261287 4 zerg 2025-03-18 13:32:09 +0300 Отдел тестирования уже нашёл пару падений инсталлятора, о которых поставил мантерйнера в известность. 261347 5 shaba 2025-03-18 18:47:45 +0300 А что за версия mdadm? Бага повешена на sisyphus. Указанный коммит в пакете 4.3-alt1 собран 24 июня 2024. Сейчас в сизифе версия 4.4, и это исправление в нем также присутствует. 261348 6 rider 2025-03-18 18:55:52 +0300 @zerg распиши подробнее что за краш у тебя происходит 261350 7 zerg 2025-03-18 18:57:56 +0300 ss->ss не провеояется. В этом месте всегда крэш, если его нет. 261351 8 mcpain 2025-03-18 18:58:39 +0300 (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 тоже нужно проверять перед разыменованием. 261352 9 zerg 2025-03-18 18:58:59 +0300 (In reply to Alexey Shabalin from comment #5) > Сейчас в сизифе версия 4.4, и это исправление в нем также присутствует. В нём баг. 261354 10 rider 2025-03-18 19:05:11 +0300 (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 тоже нужно проверять перед разыменованием. Патч есть ? В апстрим баг завесили ? 261356 11 zerg 2025-03-18 19:16:36 +0300 (In reply to Anton Farygin from comment #10) > Патч есть ? В апстрим баг завесили ? Все карты мантейнеру в руки. Ошибка элементарная. 261357 12 rider 2025-03-18 19:17:42 +0300 (In reply to Sergey V Turchin from comment #11) > (In reply to Anton Farygin from comment #10) > > Патч есть ? В апстрим баг завесили ? > Все карты мантейнеру в руки. Ошибка элементарная. механизм воспроизведения есть ? 261358 13 zerg 2025-03-18 19:33:00 +0300 (Ответ для Anton Farygin на комментарий #12) > механизм воспроизведения есть ? Там элементарного здравого смысла достаточно. Ща патч сделаю. 261359 14 18019 zerg 2025-03-18 19:36:44 +0300 Created attachment 18019 fix-crash.patch 261360 15 zerg 2025-03-18 19:39:18 +0300 Если только, там всё запутаннее и надо репортить в апстрим, чтоб перелопатили свой код. 261361 16 zerg 2025-03-18 19:44:13 +0300 Или после free_super() надо ещё раз проверять ss перед тем, как сделать ему free(). 261362 17 zerg 2025-03-18 19:46:34 +0300 Олег, проверь, плиз. Нам жто быстрее будет сделать, а то выпуск дистрибутивов тормозит. 261378 18 mcpain 2025-03-19 10:51:53 +0300 (In reply to Sergey V Turchin from comment #17) > Олег, проверь, плиз. Нам жто быстрее будет сделать, а то выпуск > дистрибутивов тормозит. Что проверить? Я делал revert и этого хватило систему поставить 261398 19 repository-robot 2025-03-19 12:59:32 +0300 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 261415 20 zerg 2025-03-19 14:53:36 +0300 Спасибо! 261483 21 mcpain 2025-03-20 14:07:48 +0300 Всё ещё падает из-за 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 261502 22 zerg 2025-03-20 15:17:04 +0300 (Ответ для Олег Соловьев на комментарий #21) > Всё ещё падает из-за double free, патч нужно совсем откатить Имеется ввиду, надо мой откатить коммит https://web.git.kernel.org/pub/scm/utils/mdadm/mdadm.git/commit/?id=e62a561ee8b7157a2390eab215dcef6240bd7b03 261503 23 zerg 2025-03-20 15:17:41 +0300 > Имеется ввиду, надо мой откатить коммит мой патч и коммит 261527 24 repository-robot 2025-03-20 18:34:20 +0300 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) 261557 25 zerg 2025-03-21 09:43:35 +0300 p11 https://packages.altlinux.org/ru/tasks/378740/ 261558 26 zerg 2025-03-21 09:43:57 +0300 Сообщите кто-нибудь в апстрим, плиз. 18019 2025-03-18 19:36:44 +0300 2025-03-18 19:36:44 +0300 fix-crash.patch fix-crash.patch text/plain 214 zerg LS0tIG1kYWRtLmN+CisrKyBtZGFkbS5jCkBAIC0xNjk4LDcgKzE2OTgsOSBAQCBpbnQgbWFpbihp bnQgYXJnYywgY2hhciAqYXJndltdKQogCQlicmVhazsKIAl9CiAJaWYgKHNzKSB7Ci0JCXNzLT5z cy0+ZnJlZV9zdXBlcihzcyk7CisJCWlmIChzcy0+c3MpIHsKKwkJCXNzLT5zcy0+ZnJlZV9zdXBl cihzcyk7CisJCX0KIAkJZnJlZShzcyk7CiAJfQogCWlmIChsb2NrZWQpCg==