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]

 



> 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

_______________________________________________
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