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 2014/01/26 01:00:31 UTC

svn commit: r1561426 - /subversion/trunk/subversion/libsvn_fs_fs/transaction.c

Author: stefan2
Date: Sun Jan 26 00:00:31 2014
New Revision: 1561426

URL: http://svn.apache.org/r1561426
Log:
Fix a backward compat in FSFS: when writing repositories formats that
don't support rep sharing, we must not store the uniquifiers.

* subversion/libsvn_fs_fs/transaction.c
  (set_uniquifier): Write uniquifiers only when supported.

Found by: Evgeny Kotkov <evgeny.kotkov{_AT_}visualsvn.com>

Modified:
    subversion/trunk/subversion/libsvn_fs_fs/transaction.c

Modified: subversion/trunk/subversion/libsvn_fs_fs/transaction.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_fs_fs/transaction.c?rev=1561426&r1=1561425&r2=1561426&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_fs_fs/transaction.c (original)
+++ subversion/trunk/subversion/libsvn_fs_fs/transaction.c Sun Jan 26 00:00:31 2014
@@ -1445,10 +1445,14 @@ set_uniquifier(svn_fs_t *fs,
                apr_pool_t *pool)
 {
   svn_fs_fs__id_part_t temp;
+  fs_fs_data_t *ffd = fs->fsap_data;
 
-  SVN_ERR(get_new_txn_node_id(&temp, fs, &rep->txn_id, pool));
-  rep->uniquifier.noderev_txn_id = rep->txn_id;
-  rep->uniquifier.number = temp.number;
+  if (ffd->format >= SVN_FS_FS__MIN_REP_SHARING_FORMAT)
+    {
+      SVN_ERR(get_new_txn_node_id(&temp, fs, &rep->txn_id, pool));
+      rep->uniquifier.noderev_txn_id = rep->txn_id;
+      rep->uniquifier.number = temp.number;
+    }
 
   return SVN_NO_ERROR;
 }