Skip to content

Commit aa34584

Browse files
committed
Address feedback
1 parent 8e9789c commit aa34584

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

src/util/rbs_allocator.c

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020
#include <unistd.h>
2121
#include <sys/types.h>
2222
#include <sys/mman.h>
23+
#include <fcntl.h>
2324
#endif
2425

2526
struct rbs_allocator {
@@ -30,17 +31,14 @@ struct rbs_allocator {
3031
static void *portable_mmap_anon(size_t size) {
3132
void *ptr;
3233

33-
#ifdef _WIN32
34-
ptr = VirtualAlloc(NULL, size, MEM_RESERVE | MEM_COMMIT, PAGE_READWRITE);
35-
if (ptr == NULL) return NULL;
36-
#elif defined(MAP_ANONYMOUS)
34+
#if defined(MAP_ANONYMOUS)
3735
ptr = mmap(NULL, size, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANONYMOUS, -1, 0);
3836
#elif defined(MAP_ANON)
3937
ptr = mmap(NULL, size, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANON, -1, 0);
4038
#else
4139
/* Fallback to /dev/zero for systems without anonymous mapping */
4240
int fd = open("/dev/zero", O_RDWR);
43-
if (fd == -1) return MAP_FAILED;
41+
rbs_assert(fd != -1, "open('/dev/zero') failed");
4442

4543
ptr = mmap(NULL, size, PROT_READ | PROT_WRITE, MAP_PRIVATE, fd, 0);
4644
close(fd); /* Can close fd after mapping */

0 commit comments

Comments
 (0)