In MemoryMappedFile::map(), fd is assigned to ::open() but is not reset to zero if ::open() fails. This can result in an attempt to close an invalid file handle in MemoryMappedFile::close(). Not necessarily a problem, but if we are doing any syscall auditing or if we check the return value of ::close at some point (which wouldn't be a bad idea) we will notice failures.