# HG changeset patch
# User Pierre-Yves David <pierre-yves.david@octobus.net>
# Date 1618580340 -7200
# Node ID a3720569a43fe0be36cf15d69060e423545200d5
# Parent  99aed233aa8d873f6f2e3249b7e086a9df9227b6
nodemap: deal with data mmap error

If the file is too small, the mmapread call would raise a ValueError. We catch
that and ignore nodemap content (as we do without mmap). This make the repository
slightly slower (until the next write) but usable. Unlike the current crash.

Differential Revision: https://phab.mercurial-scm.org/D10458

diff -r 99aed233aa8d -r a3720569a43f mercurial/revlogutils/nodemap.py
--- a/mercurial/revlogutils/nodemap.py	Fri Apr 16 14:59:13 2021 +0200
+++ b/mercurial/revlogutils/nodemap.py	Fri Apr 16 15:39:00 2021 +0200
@@ -53,7 +53,11 @@
     try:
         with revlog.opener(filename) as fd:
             if use_mmap:
-                data = util.buffer(util.mmapread(fd, data_length))
+                try:
+                    data = util.buffer(util.mmapread(fd, data_length))
+                except ValueError:
+                    # raised when the read file is too small
+                    data = b''
             else:
                 data = fd.read(data_length)
     except (IOError, OSError) as e:
diff -r 99aed233aa8d -r a3720569a43f tests/test-persistent-nodemap.t
--- a/tests/test-persistent-nodemap.t	Fri Apr 16 14:59:13 2021 +0200
+++ b/tests/test-persistent-nodemap.t	Fri Apr 16 15:39:00 2021 +0200
@@ -799,9 +799,13 @@
 
 Check that Mercurial reaction to this event
 
-  $ hg -R corruption-test-repo log -r .
-  abort: index 00changelog.i is corrupted
-  [50]
+  $ hg -R corruption-test-repo log -r . --traceback
+  changeset:   5005:90d5d3ba2fc4
+  tag:         tip
+  user:        test
+  date:        Thu Jan 01 00:00:00 1970 +0000
+  summary:     a2
+  
 
 
 stream clone