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 2012/03/03 14:14:04 UTC

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

Author: stefan2
Date: Sat Mar  3 13:14:03 2012
New Revision: 1296619

URL: http://svn.apache.org/viewvc?rev=1296619&view=rev
Log:
Fix serialization of property hashes.

Note: This might look like a candidate for a backport but it is not.
This function got only called for txdelta windows, i.e. binary data
that won't care about an *additional* (beyond LEN) NUL char.

* subversion/libsvn_fs_fs/temp_serializer.c
  (serialize_svn_string): we must serialize the terminating NUL, too

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=1296619&r1=1296618&r2=1296619&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_fs_fs/temp_serializer.c (original)
+++ subversion/trunk/subversion/libsvn_fs_fs/temp_serializer.c Sat Mar  3 13:14:03 2012
@@ -136,7 +136,7 @@ serialize_svn_string(svn_temp_serializer
    * Thus, we cannot use svn_temp_serializer__add_string. */
   svn_temp_serializer__push(context,
                             (const void * const *)&string->data,
-                            string->len);
+                            string->len + 1);
 
   /* back to the caller's nesting level */
   svn_temp_serializer__pop(context);