bluestore: Fix _setattr() with rare memory alignments

Fix case bufferlist is length 0.

Signed-off-by: Adam Kupczyk <akupczyk@ibm.com>

diff --git a/src/os/bluestore/BlueStore.cc b/src/os/bluestore/BlueStore.cc
index b097c6a9680c7db3b405f754b8291668a0c7a7d8..31262c93315871a5916e1bc3efc241e6d082a0bb 100644 (file)
--- a/src/os/bluestore/BlueStore.cc
+++ b/src/os/bluestore/BlueStore.cc
@@ -18051,11 +18051,15 @@ int BlueStore::_setattr(TransContext *txc,
           << " " << name << " (" << val.length() << " bytes)"
           << dendl;
   int r = 0;
-
-  if (!val.is_contiguous() || val.front().is_partial()) {
-    val.rebuild();
+  auto& b = o->onode.attrs[name.c_str()];
+  if (val.length() == 0) {
+    b = bufferptr("", 0);
+  } else {
+    if (!val.is_contiguous() || val.front().is_partial()) {
+      val.rebuild();
+    }
+    b = val.front();
   }
-  auto& b = o->onode.attrs[name.c_str()] = val.front();
   b.reassign_to_mempool(mempool::mempool_bluestore_cache_meta);
 
   txc->write_onode(o);