[OpenAFS] Re: Salvageserver 1.6.1-3+deb7u1 core dump

Harald Barth haba@kth.se
Thu, 19 Jun 2014 19:05:02 +0200 (CEST)


> All directories are supposed to have a .. entry by this point

I disagree. Not if something has removed "..". For example a buggy
salvager like 1.6.1. Or bitrot. Or whatever. As this is a salvage
program, that should not core dump even if unexpected things happen, I
think it is very very very hard to justify such a assert in production
code for the salvager. Especially because the next layer that does
call the salvager does not handle that the salvager asserts.

Harald.