[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: Thu, 20 Sep 2012 14:19:21 +0400

On Wed, Sep 19, 2012 at 6:24 PM, Ivan Zhakov <ivan_at_visualsvn.com> wrote:
> 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.
>
Commited in r1387943.

-- 
Ivan Zhakov
Received on 2012-09-20 12:20:15 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.