svn commit: r1667845 - /subversion/branches/1.9.x/STATUS

[rh...@apache.org]

Author: rhuijben
Date: Thu Mar 19 18:33:26 2015
New Revision: 1667845

URL: http://svn.apache.org/r1667845
Log:
* STATUS: Extend justification on some nominations.

Modified:
    subversion/branches/1.9.x/STATUS

Modified: subversion/branches/1.9.x/STATUS
URL: http://svn.apache.org/viewvc/subversion/branches/1.9.x/STATUS?rev=1667845&r1=1667844&r2=1667845&view=diff
==============================================================================
--- subversion/branches/1.9.x/STATUS (original)
+++ subversion/branches/1.9.x/STATUS Thu Mar 19 18:33:26 2015
@@ -59,6 +59,12 @@ Other candidate changes:
    Justification:
      Without this fix the next session usage after this operation may
      accidentally process the result of a still outstanding request.
+     .
+     svn_ra_get_dir2() and svn_ra_get_log2() are the easy to trigger examples,
+     but propfind requests are just as vulnerable for keeping requests
+     registered after returning with some error... If there wasn't a lucky pool
+     cleanup inbetween further ra operations might access already freed memory,
+     because the request continues processing.
    Notes:
      If caller just passed a scratch pool for a single request things would
      usually have worked fine, but if not...
@@ -74,6 +80,10 @@ Other candidate changes:
      in turn will help clients produce a more stable result.
      (This resolves multiple long standing AnkhSVN issues that I wasn't
       aware of)
+     .
+     Without this patch some ra layers report out of dates via the callback
+     while others (read: serf) fail directly, causing locks not to be stored
+     in the working copy, etc.
    Notes:
      The regression test on trunk tests a bit more behavior than is backported
      here. Check r1665319 for details on what is ensured on trunk now.