You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by br...@apache.org on 2017/05/30 00:26:14 UTC

svn commit: r1796720 - /subversion/trunk/subversion/bindings/javahl/native/RemoteSession.cpp

Author: brane
Date: Tue May 30 00:26:13 2017
New Revision: 1796720

URL: http://svn.apache.org/viewvc?rev=1796720&view=rev
Log:
Fix a bug in JavaHL that caused the RA connect logic to ignore redirects.

* subversion/bindings/javahl/native/RemoteSession.cpp
  (RemoteSession::RemoteSession): Follow redirects during connect.

Found by: Marc Strapetz

Modified:
    subversion/trunk/subversion/bindings/javahl/native/RemoteSession.cpp

Modified: subversion/trunk/subversion/bindings/javahl/native/RemoteSession.cpp
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/bindings/javahl/native/RemoteSession.cpp?rev=1796720&r1=1796719&r2=1796720&view=diff
==============================================================================
--- subversion/trunk/subversion/bindings/javahl/native/RemoteSession.cpp (original)
+++ subversion/trunk/subversion/bindings/javahl/native/RemoteSession.cpp Tue May 30 00:26:13 2017
@@ -214,8 +214,9 @@ RemoteSession::RemoteSession(int retryAt
           cycle_detected = true;
           break;
         }
-      /* ### Shouldn't url be updated for the next attempt?
-         ### There is no real cycle if we just do the same thing twice? */
+
+      url = corrected_url;
+      corrected_url = NULL;
     }
 
   if (cycle_detected)