Prevent debug logging panic when mapping fails

lirm · Jun 6, 2024 · e5cc92d · e5cc92d
1 parent 7530b4c
commit e5cc92d
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/aeron/util/memmap/memmap.go b/aeron/util/memmap/memmap.go
@@ -86,6 +86,9 @@ func MapExisting(filename string, offset int64, length int) (*File, error) {
 
 	logger.Debugf("Mapping existing file: fd: %d, size: %d, offset: %d", f.Fd(), size, offset)
 	mmap, err := doMap(f, offset, mapSize)
+	if err != nil {
+		return nil, err
+	}
 	logger.Debugf("Mapped existing file @%v for %d", mmap.mmap, mmap.size)
 
 	return mmap, err
@@ -112,6 +115,9 @@ func NewFile(filename string, offset int64, length int) (*File, error) {
 	}
 
 	mmap, err := doMap(f, offset, length)
+	if err != nil {
+		return nil, err
+	}
 	logger.Debugf("Mapped a new file @%v for %d", mmap.mmap, mmap.size)
 
 	return mmap, err