Memory leak in constant use

Bug #734672 reported by David Raviv
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Diodon
Fix Released
Critical
Oliver Sauder

Bug Description

I'm using Diodon 0.2.2 on Ubuntu 10.10.
When starting Diodon it initially consumes 2-6MB.
After one day of constant use its memory consumption reaches about 70MB and after several days I had to kill it when I noticed it consumes 1.4GB.
Usage pattern: I constantly use Diodon, Clipboard size is 10, I usually copy short texts but once in a while I do copy large segments of code.

Related branches

Revision history for this message
Oliver Sauder (sao) wrote :

Did some memory leak checking with valgrind. Can confirm that there are memory leaks. Trying to track them down.

Changed in diodon:
status: New → Triaged
importance: Undecided → Critical
assignee: nobody → Oliver Sauder (sao)
Oliver Sauder (sao)
Changed in diodon:
status: Triaged → In Progress
Revision history for this message
Oliver Sauder (sao) wrote :

Found one nasty memory leak due to a bug in the gtk vapi. Implemented a workaround in diodon but trying to fix it upstream.

Could you check out the new version, if it is working for you?

Use following commands:
sudo apt-get build-dep diodon
bzr branch lp:~diodon-team/diodon/fix-memory-leak
cd fix-memory-leak
./waf configure
./waf build
sudo ./waf install

(Hint just call sudo ./waf uninstall do uninstall diodon again, so the original version of the debian package is used)

Changed in diodon:
status: In Progress → Fix Committed
Revision history for this message
David Raviv (davrav) wrote :

Tested it over the weekend and the leak seems to be restrained. Other than that, Diodon worked as expected.

When do you intend to merge it into trunk?

Thanks for the super fast response and the help with the installation.

Revision history for this message
Oliver Sauder (sao) wrote :

Thanks for testing. Hope to get this branch merged and released by next Wednesday.

Changed in diodon:
milestone: none → 0.3.0
Revision history for this message
Oliver Sauder (sao) wrote :

Merged into trunk and available in the daily ppa. However it is gonna take a little bit longer for an official release.

Oliver Sauder (sao)
Changed in diodon:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.