2019-12-10 09:29:09 +00:00
|
|
|
From 7cfbf3a394c2663fbeed17705320b83e69781720 Mon Sep 17 00:00:00 2001
|
|
|
|
From: David Teigland <teigland@redhat.com>
|
|
|
|
Date: Thu, 29 Aug 2019 11:35:46 -0500
|
|
|
|
Subject: [PATCH] fix segfault for invalid characters in vg name
|
|
|
|
|
|
|
|
Fixes a regression from commit ba7ff96faff0
|
|
|
|
"improve reading and repairing vg metadata"
|
|
|
|
|
|
|
|
where the error path for a vg name with invalid
|
|
|
|
charaters was missing an error flag, which led
|
|
|
|
to the caller not recognizing an error occured.
|
|
|
|
Previously, an error flag was hidden in the old
|
|
|
|
_vg_make_handle function.
|
|
|
|
---
|
|
|
|
lib/metadata/metadata.c | 3 ++-
|
|
|
|
1 file changed, 2 insertions(+), 1 deletion(-)
|
|
|
|
|
2019-12-25 02:03:58 +00:00
|
|
|
diff --git a/lib/metadata/metadata.c b/lib/metadata/metadata.c
|
|
|
|
index 2c61bdeca2..6d21ff99cc 100644
|
|
|
|
--- a/lib/metadata/metadata.c
|
|
|
|
+++ b/lib/metadata/metadata.c
|
|
|
|
@@ -4900,7 +4900,8 @@ struct volume_group *vg_read(struct cmd_context *cmd, const char *vg_name, const
|
2019-12-10 09:29:09 +00:00
|
|
|
|
|
|
|
if (!validate_name(vg_name)) {
|
|
|
|
log_error("Volume group name \"%s\" has invalid characters.", vg_name);
|
|
|
|
- return NULL;
|
|
|
|
+ failure |= FAILED_NOTFOUND;
|
|
|
|
+ goto_bad;
|
|
|
|
}
|
|
|
|
|
2019-12-25 02:03:58 +00:00
|
|
|
/*
|
|
|
|
--
|
|
|
|
2.16.4
|
|
|
|
|