Re: Lack of validation in svn_repos_open()

On Tue, Dec 06, 2005 at 03:51:36PM -0500, Greg Hudson wrote:
> On Tue, 2005-12-06 at 19:29 +0000, Malcolm Rowe wrote:
> > Ow. That seems... extreme, not to mention unexpected. Unless I'm
> > missing something, we don't actually seem to document this anywhere
> > either: certainly not in svn_repos.h, and not in svn_path.h either.
>
> It's in svn_path.h:
>
> * All paths passed to the @c svn_path_xxx functions, with the exceptions of
> * the svn_path_canonicalize() and svn_path_internal_style() functions,
> * must be in canonical form.
>

I saw that, but I'm not calling an svn_path_ function, I'm calling
svn_repos_open() ;-)

Pernickety, perhaps.

> You're correct that higher-level functions which inherit this
> requirement do not appear to document it.

Yes, that's the thing. I'm also kinda surprised that we abort() when
we find a path with a trailing slash (for comparison, we return an error
if we find an invalid UTF-8 string).

Regards,
Malcolm

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Dec 6 23:55:03 2005