[lvm-devel] master - cleanup: error is already logged
Zdenek Kabelac
zkabelac at fedoraproject.org
Mon Oct 6 13:33:49 UTC 2014
Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=916a68585e5078ff3f3190a9f77bee2f7b9b30a2
Commit: 916a68585e5078ff3f3190a9f77bee2f7b9b30a2
Parent: 73c74d582c27400f8cad939a52503e3721c0e153
Author: Zdenek Kabelac <zkabelac at redhat.com>
AuthorDate: Mon Oct 6 12:01:21 2014 +0200
Committer: Zdenek Kabelac <zkabelac at redhat.com>
CommitterDate: Mon Oct 6 15:23:08 2014 +0200
cleanup: error is already logged
Function get_only_segment_using_this_lv() already prints error,
no need to show more internal logic to user as an error.
---
lib/metadata/pool_manip.c | 8 ++------
1 files changed, 2 insertions(+), 6 deletions(-)
diff --git a/lib/metadata/pool_manip.c b/lib/metadata/pool_manip.c
index 6e2f2fe..439b8ed 100644
--- a/lib/metadata/pool_manip.c
+++ b/lib/metadata/pool_manip.c
@@ -215,12 +215,8 @@ struct lv_segment *find_pool_seg(const struct lv_segment *seg)
{
struct lv_segment *pool_seg;
- pool_seg = get_only_segment_using_this_lv(seg->lv);
-
- if (!pool_seg) {
- log_error("Failed to find pool_seg for %s", seg->lv->name);
- return NULL;
- }
+ if (!(pool_seg = get_only_segment_using_this_lv(seg->lv)))
+ return_NULL;
if ((lv_is_thin_type(seg->lv) && !seg_is_pool(pool_seg))) {
log_error("%s on %s is not a %s pool segment",
More information about the lvm-devel
mailing list