Opened 5 years ago

Closed 5 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 5 years ago by Jari Häkkinen

Status: newassigned

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

comment:2 Changed 5 years ago by Jari Häkkinen

(In [1574]) Addresses #528. The baseplugins issue is resolved with this fix.

comment:3 Changed 5 years ago by Jari Häkkinen

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 in reply to:  3 Changed 5 years ago by Peter Johansson

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 5 years ago by Jari Häkkinen

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 5 years ago by Jari Häkkinen

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.