diff mercurial/repoview.py @ 18443:64848f7fb764

repoview: protect `base` computation from weird phase root If for some reason the phase roots contains nullid, the set of filtered revs will contains -1. That confuse Mercurial a lot. In particular this corrupt the branchcache. Standard code path does not result in nullid phase root. It can only result from altered `.hg/store/phaseroots` or buggy extension. However better safe than sorry.
author Pierre-Yves David <pierre-yves.david@logilab.fr>
date Thu, 17 Jan 2013 17:51:30 +0100
parents f3b21beb9802
children 4d92e2d75cff
line wrap: on
line diff
--- a/mercurial/repoview.py	Fri Jan 18 15:55:16 2013 -0800
+++ b/mercurial/repoview.py	Thu Jan 17 17:51:30 2013 +0100
@@ -90,6 +90,8 @@
     for roots in repo._phasecache.phaseroots[1:]:
         if roots:
             firstmutable = min(firstmutable, min(cl.rev(r) for r in roots))
+    # protect from nullrev root
+    firstmutable = max(0, firstmutable)
     return frozenset(xrange(firstmutable, len(cl)))
 
 # function to compute filtered set