Add missing no-longer-used segs_using_this_lv test to check_lv_segments.

diff --git a/WHATS_NEW b/WHATS_NEW
index 9c2e127b0ac6d552d6458488712f486af6dc00d1..7c337edc73993ee9c302b1cc41f7c7d96f05e34f 100644 (file)
--- a/WHATS_NEW
+++ b/WHATS_NEW
@@ -1,17 +1,18 @@
 Version 2.02.34 -
 ===================================
-  Remove redundant if-before-free tests in clvmd.c.
-  Avoid a compiler warning: make is_orphan's parameter "const".
+  Add missing no-longer-used segs_using_this_lv test to check_lv_segments.
+  Remove redundant non-NULL tests before calling free in clvmd.c.
+  Avoid a compiler warning: make is_orphan's parameter const.
   Fix lvconvert detection of mirror conversion in progress. (2.02.30)
   Avoid automatic lvconvert polldaemon invocation when -R specified. (2.02.30)
   Fix 'pvs -a' to detect VGs of PVs without metadata areas.
   Divide up internal orphan volume group by format type.
   Update usage message for clvmd.
-  Fix clvmd man page printing <br>, clarified debug options.
-  Fix lvresize to support /dev/mapper prefix in the lvname
-  Fix unfilled paramater passed to fsadm from lvresize
-  Update fsadm to call lvresize if the partition size differs (with option -l)
-  Fix fsadm to support vg/lv name (like the rest of lv-tools)
+  Fix clvmd man page not to print <br> and clarified debug options.
+  Fix lvresize to support /dev/mapper prefix in the LV name.
+  Fix unfilled parameter passed to fsadm from lvresize.
+  Update fsadm to call lvresize if the partition size differs (with option -l).
+  Fix fsadm to support VG/LV names.
 
 Version 2.02.33 - 31st January 2008
 ===================================

diff --git a/lib/metadata/lv_manip.c b/lib/metadata/lv_manip.c
index eef39a7dd129e3502116473d156295b05337ef85..40597ae5422bf89ab15a93b78057eb831220ef13 100644 (file)
--- a/lib/metadata/lv_manip.c
+++ b/lib/metadata/lv_manip.c
@@ -2308,6 +2308,9 @@ int remove_layer_from_lv(struct logical_volume *lv,
            parent->le_count != layer_lv->le_count)
                return_0;
 
+       if (!lv_empty(parent))
+               return_0;
+
        if (!_move_lv_segments(parent, layer_lv, 0, 0))
                return_0;