Re: [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]

 



On 11/12/2009 10:11 PM, Chris Lumens wrote:
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)

Fixes for bugs that don't have numbers always makes me nervous on
rhel5-branch.  Were you hitting this in testing your other patches?  Do
you have any idea how widespread it is?

- Chris

Chris,

this was originally a part of #502768 patch which was provided by Masahiro Matsuya (mmatsuya@xxxxxxxxxx). I assumed he was hitting it since he is indirectly calling it from the code of his patch.

Anyway -- you guys asked me to split this into a separate patch so we can track it more easily, I just forgot to write the BZ# in the msg.

Ales

_______________________________________________
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