Currently RelocationHolder::reloc() is const, but returns a non-const pointer to its _relocbuf member (cast to Relocation*). We should probably have two overloads, one for const and one non-const, returning a suitably const result. There are callers that need to be fixed in order to do that.