Re: Error in abs-path handling in normalize_merge_sources()

Julian Foad wrote:
> I spotted an error in the conversion to use abs-paths:
>
> [[[
> static svn_error_t *
> normalize_merge_sources(apr_array_header_t **merge_sources_p,
[...]
> SVN_ERR(svn_dirent_get_absolute(&source_abspath, source, pool));
> ]]]
>
> In that last line shown, 'source' can be (and often is) a URL so this is
> wrong.

I suggest adding

  SVN_ERR_ASSERT(!svn_path_is_url(relative));

inside svn_dirent_get_absolute(), at least for debugging, as we should
never be calling it on any path that looks like a URL. When I tried
doing that, almost all tests failed, presumably because some universal
operation is calling it on a URL.

I'm not planning to tackle this.

- Julian

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2424621

Please start new threads on the <dev_at_subversion.apache.org> mailing list.
To subscribe to the new list, send an empty e-mail to <dev-subscribe_at_subversion.apache.org>.
Received on 2009-11-26 16:52:40 CET