svn commit: r1514628 - /subversion/trunk/subversion/libsvn_ra_serf/serf.c

[ivan]

Author: ivan
Date: Fri Aug 16 09:18:19 2013
New Revision: 1514628

URL: http://svn.apache.org/r1514628
Log:
Follow-up to r1514315: Avoid potential issue with '%' character in
SVN_VER_NUMBER or SVN_BUILD_TARGET.

Found by: danielsh

* subversion/libsvn_ra_serf/serf.c
  (get_user_agent_string): Do not use SVN_VER_NUMBER and SVN_BUILD_TARGET in 
   format string.

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

Modified: subversion/trunk/subversion/libsvn_ra_serf/serf.c
URL: 
http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_ra_serf/serf.c?rev=1514628&r1=1514627&r2=1514628&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_ra_serf/serf.c (original)
+++ subversion/trunk/subversion/libsvn_ra_serf/serf.c Fri Aug 16 09:18:19 2013
@@ -423,8 +423,8 @@ get_user_agent_string(apr_pool_t *pool)
   int major, minor, patch;
   serf_lib_version(&major, &minor, &patch);
 
-  return apr_psprintf(pool, "SVN/" SVN_VER_NUMBER " (" SVN_BUILD_TARGET ") "
-                      " serf/%d.%d.%d",
+  return apr_psprintf(pool, "SVN/%s (%s) serf/%d.%d.%d",
+                      SVN_VER_NUMBER, SVN_BUILD_TARGET,
                       major, minor, patch);
 }