commit 5aa6d8278bb61de18b7212f1c09b57204c7683af
author Jens Axboe <axboe@fb.com> Thu Oct 30 08:40:47 2014 -0600
committer Mohamad Ayyash <mkayyash@google.com> Fri Mar 06 17:58:01 2015 -0800
tree 7c304d1a5fe2c24022b3c56b1af2c47ea846985b
parent 6a605302a1f0b63075cfbe7144122bcdc61d61cc

mmap: set 'fd' to -1

This looks like a typo, using '1' makes no sense. This fixes Solaris
anon allocs.

Reported-by: Sébastien Bouchex Bellomié <sbouchex@infovista.com>
Signed-off-by: Jens Axboe <axboe@fb.com>

memory.c

diff --git a/memory.c b/memory.c
index 8c06d94..76da8a8 100644
--- a/memory.c
+++ b/memory.c
@@ -119,7 +119,7 @@
 {
 	int flags = 0;
 
-	td->mmapfd = 1;
+	td->mmapfd = -1;
 
 	if (td->o.mem_type == MEM_MMAPHUGE) {
 		unsigned long mask = td->o.hugepage_size - 1;
@@ -176,7 +176,8 @@
 						td->orig_buffer);
 	munmap(td->orig_buffer, td->orig_buffer_size);
 	if (td->o.mmapfile) {
-		close(td->mmapfd);
+		if (td->mmapfd != -1)
+			close(td->mmapfd);
 		unlink(td->o.mmapfile);
 		free(td->o.mmapfile);
 	}