[Libguestfs] [PATCH] Don't abort inspection if mdadm.conf ARRAY doesn't have a uuid
Matthew Booth
mbooth at redhat.com
Wed Apr 18 09:09:58 UTC 2012
On 17/04/12 16:07, Richard W.M. Jones wrote:
> 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.
valgrind tests pass. I've pushed this.
Matt
--
Matthew Booth, RHCA, RHCSS
Red Hat Engineering, Virtualisation Team
GPG ID: D33C3490
GPG FPR: 3733 612D 2D05 5458 8A8A 1600 3441 EA19 D33C 3490
More information about the Libguestfs
mailing list