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/09/09 19:32:48 UTC
svn commit: r995508 - /subversion/trunk/subversion/svnserve/cyrus_auth.c
Author: stsp
Date: Thu Sep 9 17:32:47 2010
New Revision: 995508
URL: http://svn.apache.org/viewvc?rev=995508&view=rev
Log:
* subversion/svnserve/cyrus_auth.c
(cyrus_auth_request): Replace use of atoi() with svn_cstring_atoui().
The values being parsed are supposed to be unsigned integers as per
Cyrus-SASL documentation.
Modified:
subversion/trunk/subversion/svnserve/cyrus_auth.c
Modified: subversion/trunk/subversion/svnserve/cyrus_auth.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/svnserve/cyrus_auth.c?rev=995508&r1=995507&r2=995508&view=diff
==============================================================================
--- subversion/trunk/subversion/svnserve/cyrus_auth.c (original)
+++ subversion/trunk/subversion/svnserve/cyrus_auth.c Thu Sep 9 17:32:47 2010
@@ -297,13 +297,13 @@ svn_error_t *cyrus_auth_request(svn_ra_s
SVN_CONFIG_SECTION_SASL,
SVN_CONFIG_OPTION_MIN_SSF,
"0");
- secprops.min_ssf = atoi(val);
+ SVN_ERR(svn_cstring_atoui(&secprops.min_ssf, val));
svn_config_get(b->cfg, &val,
SVN_CONFIG_SECTION_SASL,
SVN_CONFIG_OPTION_MAX_SSF,
"256");
- secprops.max_ssf = atoi(val);
+ SVN_ERR(svn_cstring_atoui(&secprops.max_ssf, val));
/* Set security properties. */
result = sasl_setprop(sasl_ctx, SASL_SEC_PROPS, &secprops);