Ivan Zhakov <i...@visualsvn.com> writes: > On Wed, Sep 19, 2012 at 3:07 PM, Roderich Schupp > <roderich.sch...@gmail.com> wrote: >> On Wed, Sep 19, 2012 at 10:24 AM, Roderich Schupp >> <roderich.sch...@gmail.com> wrote: >>> Change the definition of authz_svn__subreq_bypass_func_t (adding a >>> pool parameter) >>> and its one and only implementation subreq_bypass? >> >> Patch attached, grabs the same amount of memory as my first version. >> >>> Since this definition is in include/mod_authz_svn.h (i.e. it's a public >>> interface) >> >> Perhaps one should also bump #define AUTHZ_SVN__SUBREQ_BYPASS_PROV_VER >> in include/mod_authz_svn.h >> > Adding pool parameter to authz_svn__subreq_bypass_func_t is not enough > to fix all memory issues. Request's actively pool used in other places > like get_username_to_authorize() or get_access_conf()
Indeed, although the memory use there is small. In some ways we want to reuse the request pool here because it allows the access file to be cached in the request pool's user data. If we used a new/cleared pool in get_access_conf we would have to reread the file. -- Certified & Supported Apache Subversion Downloads: http://www.wandisco.com/subversion/download
