os/bluestore: fix potential csum_order overflow

Signed-off-by: xie xingguo <[email protected]>
att · Dec 6, 2016 · ef7fdc0 · ef7fdc0
1 parent dded3de
commit ef7fdc0
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/src/os/bluestore/BlueStore.cc b/src/os/bluestore/BlueStore.cc
@@ -8064,8 +8064,13 @@ int BlueStore::_do_write(
 			CEPH_OSD_ALLOC_HINT_FLAG_APPEND_ONLY)) &&
       (alloc_hints & CEPH_OSD_ALLOC_HINT_FLAG_RANDOM_WRITE) == 0) {
     dout(20) << __func__ << " will prefer large blob and csum sizes" << dendl;
-    wctx.csum_order = std::max(min_alloc_size_order,
-			       (size_t)ctzl(o->onode.expected_write_size));
+    if (o->onode.expected_write_size) {
+      wctx.csum_order = std::max(min_alloc_size_order,
+			         (size_t)ctzl(o->onode.expected_write_size));
+    } else {
+      wctx.csum_order = min_alloc_size_order;
+    }
+
     if (wctx.compress) {
       wctx.target_blob_size = select_option(
         "compression_max_blob_size",