On Tue, Apr 17, 2012 at 10:18:45AM +0100, Matthew Booth wrote: > --- > src/inspect_fs_unix.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/inspect_fs_unix.c b/src/inspect_fs_unix.c > index 1ef4ca4..5695adc 100644 > --- a/src/inspect_fs_unix.c > +++ b/src/inspect_fs_unix.c > @@ -1113,7 +1113,7 @@ map_md_devices(guestfs_h *g, Hash_table **map) > free(uuid_path); > if (!uuid) { > free(dev); > - goto error; > + continue; Looks OK. Does it run under valgrind ('make extra-tests')? That's a good way to catch memory leaks and other problems in these complex and hot inspection paths.