You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by st...@apache.org on 2010/11/22 02:31:21 UTC

svn commit: r1037586 - /subversion/branches/performance/subversion/libsvn_fs_fs/fs_fs.c

Author: stefan2
Date: Mon Nov 22 01:31:20 2010
New Revision: 1037586

URL: http://svn.apache.org/viewvc?rev=1037586&view=rev
Log:
File handle caching causes a potential svn_fs_fs__path_rev_absolute
race condition if revision files get deleted during a FSFS session.
Mention that in the doc string.

* subversion/libsvn_fs_fs/fs_fs.c
  (svn_fs_fs__path_rev_absolute): update doc string

Modified:
    subversion/branches/performance/subversion/libsvn_fs_fs/fs_fs.c

Modified: subversion/branches/performance/subversion/libsvn_fs_fs/fs_fs.c
URL: http://svn.apache.org/viewvc/subversion/branches/performance/subversion/libsvn_fs_fs/fs_fs.c?rev=1037586&r1=1037585&r2=1037586&view=diff
==============================================================================
--- subversion/branches/performance/subversion/libsvn_fs_fs/fs_fs.c (original)
+++ subversion/branches/performance/subversion/libsvn_fs_fs/fs_fs.c Mon Nov 22 01:31:20 2010
@@ -259,7 +259,10 @@ path_rev(svn_fs_t *fs, svn_revnum_t rev,
 }
 
 /* Returns the path of REV in FS, whether in a pack file or not.
-   Allocate in POOL. */
+   Allocate in POOL.
+   NOTE. The file given by path is only guaranteed to actually
+   exist if it has not been deleted since this FSFS session has
+   been opened. */
 svn_error_t *
 svn_fs_fs__path_rev_absolute(const char **path,
                              svn_fs_t *fs,