Opened 7 years ago
Closed 7 years ago
#528 closed task (fixed)
Memory blows up when processing http://baseplugins.thep.lu.se/ repository
Reported by: | Jari Häkkinen | Owned by: | Jari Häkkinen |
---|---|---|---|
Priority: | major | Milestone: | svndigest 0.10.2 |
Component: | core | Version: | trunk |
Keywords: | Cc: |
Description
At program exit (in memory destruction) a catastrophic memory leaks occurs. The issue has been there for at least 2 years. Parsing other large repositories, e.g. http://base.thep.lu.se) does not trigger the leak (at least not as severe as for baseplugins).
This issue is in all versions of svndigest (svnstat) since v0.3, i.e., since we started using the subversion API.
Change History (6)
comment:1 Changed 7 years ago by
Status: | new → assigned |
---|
comment:2 Changed 7 years ago by
comment:3 follow-up: 4 Changed 7 years ago by
How far back should we copy this fix. I think the current stable 0.10 and future branches are good enough. 0.10 has been out since three years (2012) and no one should use older releases.
comment:4 Changed 7 years ago by
Replying to jari:
How far back should we copy this fix. I think the current stable 0.10 and future branches are good enough. 0.10 has been out since three years (2012) and no one should use older releases.
agreed
comment:5 Changed 7 years ago by
The nightly updates of projects stats at http://cbbp.thep.lu.se/~jari/svndigest/ works perfectly again so I'd say we finally resolved the exit memory leak.
comment:6 Changed 7 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Digging through the code I found enough evidence to realize the we are not initializing properly, and then I found http://subversion.apache.org/docs/community-guide/conventions.html#apr-pools