ceph_disk/main.py: fixed bugs from theutology report

Signed-off-by: Willem Jan Withagen <wjw@digiware.nl>

diff --git a/src/ceph-disk/ceph_disk/main.py b/src/ceph-disk/ceph_disk/main.py
index ff3b93966f3b0a24ac4d6a8113d44357191fe215..ddc6bcfa23d98464bf1ae1922a06dfd8af8a498b 100755 (executable)
--- a/src/ceph-disk/ceph_disk/main.py
+++ b/src/ceph-disk/ceph_disk/main.py
@@ -671,9 +671,7 @@ def stmode_is_diskdevice(dmode):
     else:
         # FreeBSD does not have block devices
         # All disks are character devices
-        if FREEBSD and stat.S_ISCHR(dmode):
-            return True
-    return False
+        return FREEBSD and stat.S_ISCHR(dmode)
 
 
 def dev_is_diskdevice(dev):
@@ -803,7 +801,7 @@ def get_partition_base(dev):
     Get the base device for a partition
     """
     dev = os.path.realpath(dev)
-    if ldev_is_diskdevice(dev):
+    if not ldev_is_diskdevice(dev):
         raise Error('not a block device', dev)
 
     name = get_dev_name(dev)
@@ -845,7 +843,7 @@ def is_partition(dev):
 
     dev = os.path.realpath(dev)
     st = os.lstat(dev)
-    if not stmode_is_diskdevice(st.mode):
+    if not stmode_is_diskdevice(st.st_mode):
         raise Error('not a block device', dev)
 
     name = get_dev_name(dev)