Re: [Libguestfs] [PATCH v1] ldmtool: fix NULL pointer dereference

2023-06-19 Thread Laszlo Ersek
On 6/19/23 16:36, Vincent Mailhol wrote: > If /sys/block can not be opened, get_devices() returns NULL. > > cmdline() does not check this result and below code snippet: > > scanned = get_devices(); > devices = (gchar **) scanned->data; > > results in a segmentation fault. >

Re: [Libguestfs] libldm crashes in a linux-sandbox context

2023-06-19 Thread Laszlo Ersek
On 6/19/23 16:32, Vincent Mailhol wrote: > On Mon 19 June 2023 at 21:16, Laszlo Ersek wrote: >> On 6/19/23 13:18, Vincent MAILHOL wrote: >>> On Fri. 16 juin 2023 at 16:34, Richard W.M. Jones wrote: >>> (...) > Last thing, the segfault on ldmtool [1] still seems a valid issue. > Even if I

[Libguestfs] [v2v PATCH] test-data/phony-guests: fix prerequisite list of "fedora-luks-on-lvm.img"

2023-06-19 Thread Laszlo Ersek
In the virt-v2v repo, commit 1e75569aa074 ("test-data/phony-guests: Allow virt-v2v to work against phony Fedora") is an ancestor of commit e4efe4b7d240 ("tests: add LUKS-on-LVM test"). The latter created a state where "fedora-static-bin" and LUKS on LVM testing would coexist (i.e., where "fedora-st

[Libguestfs] [PATCH v1] ldmtool: fix NULL pointer dereference

2023-06-19 Thread Vincent Mailhol
If /sys/block can not be opened, get_devices() returns NULL. cmdline() does not check this result and below code snippet: scanned = get_devices(); devices = (gchar **) scanned->data; results in a segmentation fault. Add a check on scanned. Relevant logs: Unable to open /

Re: [Libguestfs] libldm crashes in a linux-sandbox context

2023-06-19 Thread Laszlo Ersek
On 6/19/23 13:18, Vincent MAILHOL wrote: > On Fri. 16 juin 2023 at 16:34, Richard W.M. Jones wrote: > (...) >>> Last thing, the segfault on ldmtool [1] still seems a valid issue. >>> Even if I now do have a workaround for my problem, that segfault might >>> be worth a bit more investigation. >> >>

Re: [Libguestfs] [libnbd PATCH v4 1/4] states: Document our reliance on type overlaps

2023-06-19 Thread Laszlo Ersek
On 6/12/23 20:10, Richard W.M. Jones wrote: > On Fri, Jun 09, 2023 at 03:39:19PM -0500, Eric Blake wrote: >> [Bah - I typed up a longer response, but lost it when accidentally >> trying to send through the wrong SMTP server, so now I have to >> remember what I had...] >> >> On Fri, Jun 09, 2023 at

Re: [Libguestfs] libldm crashes in a linux-sandbox context

2023-06-19 Thread Vincent MAILHOL
On Fri. 16 juin 2023 at 16:34, Richard W.M. Jones wrote: (...) > > Last thing, the segfault on ldmtool [1] still seems a valid issue. > > Even if I now do have a workaround for my problem, that segfault might > > be worth a bit more investigation. > > Yes that does look like a real problem. Does

Re: [Libguestfs] [libnbd PATCH v4 1/4] states: Document our reliance on type overlaps

2023-06-19 Thread Laszlo Ersek
On 6/9/23 22:39, Eric Blake wrote: > [Bah - I typed up a longer response, but lost it when accidentally > trying to send through the wrong SMTP server, so now I have to > remember what I had...] > > On Fri, Jun 09, 2023 at 02:45:56PM +0200, Laszlo Ersek wrote: >> On 6/9/23 04:17, Eric Blake wrote: