]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
librbd.cc: fix usage of unsigned var
authorDanny Al-Gaaf <danny.al-gaaf@bisect.de>
Fri, 13 Oct 2017 18:43:43 +0000 (20:43 +0200)
committerDanny Al-Gaaf <danny.al-gaaf@bisect.de>
Wed, 25 Oct 2017 16:14:05 +0000 (18:14 +0200)
Fix for:

[src/librbd/librbd.cc:3578]: (style) Checking if unsigned variable
 'data_len' is less than zero.
[src/librbd/librbd.cc:3799]: (style) Checking if unsigned variable
 'data_len' is less than zero.
[src/librbd/librbd.cc:3576]: (style) Checking if unsigned variable
 'data_len' is less than zero.
[src/librbd/librbd.cc:3796]: (style) Checking if unsigned variable
 'data_len' is less than zero.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
src/librbd/librbd.cc

index e65330c2580defce42d9f19129762c1b24eb1fbf..0004a2a87a0046cee2dba286d94b2c11b6d57297 100644 (file)
@@ -3573,9 +3573,9 @@ extern "C" ssize_t rbd_writesame(rbd_image_t image, uint64_t ofs, size_t len,
 {
   librbd::ImageCtx *ictx = (librbd::ImageCtx *)image;
   tracepoint(librbd, writesame_enter, ictx, ictx->name.c_str(), ictx->snap_name.c_str(),
-             ictx->read_only, ofs, len, data_len <= 0 ? NULL : buf, data_len, op_flags);
+             ictx->read_only, ofs, len, data_len == 0 ? NULL : buf, data_len, op_flags);
 
-  if (data_len <= 0 || len % data_len) {
+  if (data_len == 0 || len % data_len) {
     tracepoint(librbd, writesame_exit, -EINVAL);
     return -EINVAL;
   }
@@ -3793,10 +3793,10 @@ extern "C" int rbd_aio_writesame(rbd_image_t image, uint64_t off, size_t len,
   librbd::ImageCtx *ictx = (librbd::ImageCtx *)image;
   librbd::RBD::AioCompletion *comp = (librbd::RBD::AioCompletion *)c;
   tracepoint(librbd, aio_writesame_enter, ictx, ictx->name.c_str(), ictx->snap_name.c_str(),
-             ictx->read_only, off, len, data_len <= 0 ? NULL : buf, data_len, comp->pc,
+             ictx->read_only, off, len, data_len == 0 ? NULL : buf, data_len, comp->pc,
              op_flags);
 
-  if (data_len <= 0 || len % data_len) {
+  if (data_len == 0 || len % data_len) {
     tracepoint(librbd, aio_writesame_exit, -EINVAL);
     return -EINVAL;
   }