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/08/29 14:33:39 UTC

svn commit: r990574 - /subversion/branches/performance/subversion/libsvn_subr/cache-inprocess.c

Author: stefan2
Date: Sun Aug 29 12:33:39 2010
New Revision: 990574

URL: http://svn.apache.org/viewvc?rev=990574&view=rev
Log:
Fix a crash when membuffer caching has been disabled (svnserve -M0):
the get_partical implementation for inprocess_cache used an outdated
callback API where the size could be 0 for non-empty entries in certain cases.

* subversion/libsvn_subr/cache-inprocess.c
  (inprocess_cache_get_partial): actually pass the size parameter to func

Modified:
    subversion/branches/performance/subversion/libsvn_subr/cache-inprocess.c

Modified: subversion/branches/performance/subversion/libsvn_subr/cache-inprocess.c
URL: http://svn.apache.org/viewvc/subversion/branches/performance/subversion/libsvn_subr/cache-inprocess.c?rev=990574&r1=990573&r2=990574&view=diff
==============================================================================
--- subversion/branches/performance/subversion/libsvn_subr/cache-inprocess.c (original)
+++ subversion/branches/performance/subversion/libsvn_subr/cache-inprocess.c Sun Aug 29 12:33:39 2010
@@ -462,7 +462,7 @@ inprocess_cache_get_partial(void **value
   move_page_to_front(cache, entry->page);
 
   *found = TRUE;
-  err = func(value_p, entry->value, 0, baton, pool);
+  err = func(value_p, entry->value, entry->size, baton, pool);
   return unlock_cache(cache, err);
 }