krbd: change krbd_unmap_by_spec() to take "" instead of NULL

krbd_map() and krbd_is_mapped() take "", krbd_unmap_by_spec() is the
odd one out.

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>

diff --git a/src/krbd.cc b/src/krbd.cc
index 33c7fb5d76a2780548b620e28dd933440c1d05da..3072045bc50568aef75d201c28b8d36dd32b78f0 100644 (file)
--- a/src/krbd.cc
+++ b/src/krbd.cc
@@ -610,7 +610,7 @@ static int unmap_image(struct krbd_ctx *ctx, const char *pool,
   string id;
   int r;
 
-  if (!snap)
+  if (!*snap)
     snap = "-";
 
   r = spec_to_devno_and_krbd_id(ctx->udev, pool, image, snap, &devno, &id);

diff --git a/src/tools/rbd/action/Kernel.cc b/src/tools/rbd/action/Kernel.cc
index ef2c8f30ff0860914454fc4d5849eb6c341de52a..91715e329fc0cf339c555bbeb88ce6b340cb5bb4 100644 (file)
--- a/src/tools/rbd/action/Kernel.cc
+++ b/src/tools/rbd/action/Kernel.cc
@@ -498,8 +498,7 @@ int execute_unmap(const po::variables_map &vm,
   utils::init_context();
 
   r = do_kernel_unmap(device_name.empty() ? nullptr : device_name.c_str(),
-                      pool_name.c_str(), image_name.c_str(),
-                      snap_name.empty() ? nullptr : snap_name.c_str());
+                      pool_name.c_str(), image_name.c_str(), snap_name.c_str());
   if (r < 0) {
     std::cerr << "rbd: unmap failed: " << cpp_strerror(r) << std::endl;
     return r;