[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#57513] [PATCH] installer: Fix segfault on double logical partition
From: |
Josselin Poiret |
Subject: |
[bug#57513] [PATCH] installer: Fix segfault on double logical partition removal. |
Date: |
Thu, 01 Sep 2022 21:16:02 +0200 |
Hey Mathieu,
Mathieu Othacehe <othacehe@gnu.org> writes:
> I was able to reproduce the issue by creating an extended partition
> containing a single logical partition using the manual partitioning tool
> then, the automatic one right after.
>
> It resulted in a segfault, which is fixed by your patch, that's a very
> nice catch!
I have to thank KE0VVT on IRC, who provided a core dump file! This was
surprisingly easier to debug than I thought, for those interested, I
built the installer using the same Guix commit, and loaded the guile
core dump file in gdb. I then used `guix build parted
--with-debug-info=parted` and loaded the resulting libparted.so library
using `info sections` to find out where the .text of libparted.so was
loaded in the core file, and `add-symbol-file
/gnu/store/path/to/libparted.so 0xaddress` to load the symbols. That
way, I could see that ped_disk_remove_partition was invoked for a disk
that had an empty partition list, hence leading me to this double remove
problem!
> Pushed as 4989f6acff3b3fcfbd9dde3e3c2767bd2cd6d49e.
>
> Thanks,
>
> Mathieu
Thank you for reviewing this so fast!
Best,
--
Josselin Poiret