+1; we already validate reps given in node-revision headers. (could you CC me on the issue?)
Stefan Sperling wrote on Wed, Jul 13, 2011 at 15:59:36 +0200: > Hi all, > > while repairing some bad repository breakage we noticed that > svnadmin verify does not check whether offsets mentioned in > rep-cache.db point to valid reps. > > In some cases I had to append repaired reps after the changed-paths > section in revision files and leave broken reps in place to avoid > adjusting offsets everywhere. > > So now 'svnadmin verify' runs fine. But when the rep-cache is > referred to during commit we can still run into reps that are broken. > > Obviously we'll either have to fix up the rep-cache as well, > or simply delete it. But the point is that 'svnadmin verify' > does not complain. > > Would it be feasable to add this as a separate pass of 'svnadmin verify'? > Should I file an issue about this? > > For instance, good output could look like this: > > $ svnadmin verify repos > * Verified revision 1. > ... > * Verified revision N. > * Verified rep-cache. > > $ > > And errors like this: > > $ svnadmin verify repos > * Verified revision 1. > ... > * Verified revision N. > svnamdin: rep-cache entry '8fe47fa166154fc6e7f2e78366591c470ba75cf7' is > invalid > svnadmin: Malformed representation header > > $
