Re: svn commit: rev 6318 - trunk/subversion/libsvn_delta

cmpilato@collab.net writes:

> Thanks for the find-n-fix, Philip. I'm still confused as to what made
> this bug appear, but I'm content to know it's been killed.

The editor driver was opening the same directory twice in succession,
I didn't track the bug all the way through the RA layers as the driver
was obviously doing the wrong thing.

I don't really know the ra_dav editor code that well, but looking at
it now see how commit_open_dir in libsvn_ra_dav/commit.c only uses the
last component of the input path, the rest comes from the "known"
parent. This meant that the ra_dav editor would construct an invalid
path. Perhaps this function should do a sanity check on the whole
path? Or perhaps the editor interface should change and only pass the
last component?

-- 
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org