util/HugeAllocator: fix division by zero due to inverted check

There were two ways this could fail: 1. division by zero when sysconf(_SC_PAGESIZE)==0 2. mmap() failure because the size parameter is not aligned to page size Neither ever happened: sysconf() never fails, and the only caller passes a size that is already aligned. Phew.

util/HugeAllocator: fix division by zero due to inverted check
13f9f031 · Max Kellermann · 1532ffe2 · 13f9f031
Commit 13f9f031 authored Mar 06, 2016 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

HugeAllocator.cxx src/util/HugeAllocator.cxx +1 -1

No files found.
--- a/src/util/HugeAllocator.cxx
+++ b/src/util/HugeAllocator.cxx
@@ -46,7 +46,7 @@ static size_t
 AlignToPageSize(size_t size)
 {
 	static const long page_size = sysconf(_SC_PAGESIZE);
-	if (page_size > 0)
+	if (page_size == 0)
 		return size;

 	size_t ps(page_size);