svn commit: r1568872 - /subversion/trunk/subversion/libsvn_ra_serf/log.c

[br...@apache.org]

Author: brane
Date: Mon Feb 17 02:10:47 2014
New Revision: 1568872

URL: http://svn.apache.org/r1568872
Log:
Follow up to r1568871: Fix a bug in ra_serf's implementation of svn_ra_get_log
where the wrong peg revision would be chosen if the start of the range is
SVN_INVALID_REVNUM, but the end of the range is an actual revision number.

* subversion/libsvn_ra_serf/log.c (svn_ra_serf__get_log):
   When the start revision is SVN_INVALID_REVNUM, it's always "larger"
   than any valid end revision, so use it for the peg.

Modified:
    subversion/trunk/subversion/libsvn_ra_serf/log.c

Modified: subversion/trunk/subversion/libsvn_ra_serf/log.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_ra_serf/log.c?rev=1568872&r1=1568871&r2=1568872&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_ra_serf/log.c (original)
+++ subversion/trunk/subversion/libsvn_ra_serf/log.c Mon Feb 17 02:10:47 2014
@@ -602,7 +602,7 @@ svn_ra_serf__get_log(svn_ra_session_t *r
   /* At this point, we may have a deleted file.  So, we'll match ra_neon's
    * behavior and use the larger of start or end as our 'peg' rev.
    */
-  peg_rev = (start > end) ? start : end;
+  peg_rev = (start == SVN_INVALID_REVNUM || start > end) ? start : end;
 
   SVN_ERR(svn_ra_serf__get_stable_url(&req_url, NULL /* latest_revnum */,
                                       session, NULL /* conn */,