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 2011/04/18 12:03:40 UTC

svn commit: r1094443 - /subversion/trunk/subversion/libsvn_fs_fs/temp_serializer.c

Author: stefan2
Date: Mon Apr 18 10:03:40 2011
New Revision: 1094443

URL: http://svn.apache.org/viewvc?rev=1094443&view=rev
Log:
Fix memory consumption issue in commit: the serializers did not
return the allocated buffer size but just the used buffer size.
Thus, any modification would result in usually unnecessary
buffer re-allocation.

* subversion/libsvn_fs_fs/temp_serializer.c
  (return_serialized_dir_context): report the actual buffer size

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

Modified: subversion/trunk/subversion/libsvn_fs_fs/temp_serializer.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_fs_fs/temp_serializer.c?rev=1094443&r1=1094442&r2=1094443&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_fs_fs/temp_serializer.c (original)
+++ subversion/trunk/subversion/libsvn_fs_fs/temp_serializer.c Mon Apr 18 10:03:40 2011
@@ -694,7 +694,7 @@ return_serialized_dir_context(svn_temp_s
   svn_stringbuf_t *serialized = svn_temp_serializer__get(context);
 
   *data = serialized->data;
-  *data_len = serialized->len;
+  *data_len = serialized->blocksize;
   ((hash_data_t *)serialized->data)->len = serialized->len;
 
   return SVN_NO_ERROR;