subversion-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From danie...@apache.org
Subject svn commit: r1235131 - /subversion/trunk/subversion/include/private/svn_repos_private.h
Date Tue, 24 Jan 2012 04:22:49 GMT
Author: danielsh
Date: Tue Jan 24 04:22:48 2012
New Revision: 1235131

URL: http://svn.apache.org/viewvc?rev=1235131&view=rev
Log:
Document some common knowledge.

* subversion/include/private/svn_repos_private.h
  (svn_repos__validate_prop):
    Document the implications of making this validator function stricter.

Modified:
    subversion/trunk/subversion/include/private/svn_repos_private.h

Modified: subversion/trunk/subversion/include/private/svn_repos_private.h
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/include/private/svn_repos_private.h?rev=1235131&r1=1235130&r2=1235131&view=diff
==============================================================================
--- subversion/trunk/subversion/include/private/svn_repos_private.h (original)
+++ subversion/trunk/subversion/include/private/svn_repos_private.h Tue Jan 24 04:22:48 2012
@@ -44,6 +44,14 @@ extern "C" {
  *
  * Use @a pool for temporary allocations.
  *
+ * @note This function is used to implement server-side validation.
+ * Consequently, if you make this function stricter in what it accepts, you
+ * (a) break svnsync'ing of existing repositories that contain now-invalid
+ * properties, (b) do not preclude such invalid values from entering the
+ * repository via tools that use the svn_fs_* API directly (possibly
+ * including svnadmin and svnlook).  This has happened before and there
+ * are known (documented, but unsupported) upgrade paths in some cases.
+ * 
  * @since New in 1.7.
  */
 svn_error_t *



Mime
View raw message