[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Excessive memory use with "SVNPathAuthz short_circuit"

From: Ivan Zhakov <ivan_at_visualsvn.com>
Date: Wed, 19 Sep 2012 18:24:38 +0400

On Wed, Sep 19, 2012 at 6:06 PM, Philip Martin
<philip.martin_at_wandisco.com> wrote:
> Ivan Zhakov <ivan_at_visualsvn.com> writes:
>
>> -get_access_conf(request_rec *r, authz_svn_config_rec *conf)
>> +get_access_conf(request_rec *r, authz_svn_config_rec *conf, apr_pool_t *scratch_pool)
>
> That line is longer than 80 characters.
>
>> +subreq_bypass2(request_rec *r,
>> + const char *repos_path,
>> + const char *repos_name,
>> + apr_pool_t *scratch_pool)
>> {
>> svn_error_t *svn_err = NULL;
>> svn_authz_t *access_conf = NULL;
>> @@ -595,7 +597,7 @@
>>
>> conf = ap_get_module_config(r->per_dir_config,
>> &authz_svn_module);
>> - username_to_authorize = get_username_to_authorize(r, conf);
>> + username_to_authorize = get_username_to_authorize(r, conf, scratch_pool);
>>
>> /* If configured properly, this should never be true, but just in case. */
>> if (!conf->anonymous
>> @@ -606,7 +608,7 @@
>> }
>>
>> /* Retrieve authorization file */
>> - access_conf = get_access_conf(r, conf);
>> + access_conf = get_access_conf(r, conf, scratch_pool);
>> if (access_conf == NULL)
>> return HTTP_FORBIDDEN;
>>
>
> The call svn_repos_authz_check_access needs to be changed to pass
> scratch_pool.
>
Oops, I need commit more often :) See updated patch.

-- 
Ivan Zhakov

Received on 2012-09-19 16:25:31 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.