[rhel5-branch] Fixes a bug in DmDriveCache.rename: before the method passed a drive name with 'mapper/' prepended.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Prepending 'mapper/' incorrectly modifies the name, 'mapper/' is be prepended by DmDriveCache.remove() internally.
---
 dmraid.py |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/dmraid.py b/dmraid.py
index 4fb7129..ed05341 100644
--- a/dmraid.py
+++ b/dmraid.py
@@ -82,7 +82,7 @@ class DmDriveCache:
             if self.cache.has_key(obj.name):
                 del self.cache[obj.name][obj.name]
                 for k,v in self.cache[obj.name].items():
-                    log.debug("adding %s to isys cache" % (name,))
+                    log.debug("adding %s to isys cache" % (k,))
                     isys.cachedDrives[k] = v
                 log.debug("removing %s from dm cache" % (obj,))
                 del self.cache[obj.name]
@@ -91,7 +91,7 @@ class DmDriveCache:
         oldname = 'mapper/' + obj.name
         if isys.cachedDrives.has_key(oldname):
             dmNameUpdates[obj.name] = newname
-            self.remove(oldname)
+            self.remove(obj.name)
             # XXX why doesn't setting the property work?
             obj.set_name(newname)
             self.add(obj)
-- 
1.6.2.5

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux