Merge pull request #860 from igchor/stable-1.9_defrag_fix

concurrent_hash_map: make double to size_t cast explicit in defragment()

Author: Szymon Romik

include/libpmemobj++/container/concurrent_hash_map.hpp

@@ -2851,8 +2851,14 @@ class concurrent_hash_map
 		}
 
 		size_t max_index = mask().load(std::memory_order_acquire);
-		size_t start_index = (start_percent * max_index) / 100;
-		size_t end_index = (end_percent * max_index) / 100;
+		size_t start_index =
+			static_cast<size_t>((start_percent * max_index) / 100);
+		size_t end_index =
+			static_cast<size_t>((end_percent * max_index) / 100);
+
+		/* Make sure we do not use too big index, even in case of
+		 * rounding errors. */
+		end_index = (std::min)(end_index, max_index);
 
 #if LIBPMEMOBJ_CPP_VG_HELGRIND_ENABLED
 		ANNOTATE_HAPPENS_AFTER(&(this->my_mask));