Re: problem revealed by issue #2398 (server-side assertion)

On Tue, 2006-02-14 at 10:02 -0800, Garrett Rooney wrote:
> You're missing the fact that this should not be able to be triggered
> by user data over the network. The fact that it can is a bug.

Because of the new way the test is done, the condition could be
triggered by data changing on the disk (the uuid file in the
repository), and thus cannot be checked with an assert.

I object to the entire sense of the commit, personally. I think
svn_fs_merge should be demanding that the two root objects come from the
same in-memory FS object (something you can check with an assert), and
it's only a coincidence that neither back end puts much information in
the in-memory FS object and thus doesn't really care. If mod_dav_svn is
not obeying this constraint, then the bug is in mod_dav_svn.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Feb 24 19:06:51 2006