Bug 923214 - memory leaks in the C test suite
Summary: memory leaks in the C test suite
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: librepo
Version: 19
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Tomas Mlcoch
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-03-19 12:42 UTC by Ales Kozumplik
Modified: 2014-09-30 23:41 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-04-20 19:28:46 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Ales Kozumplik 2013-03-19 12:42:51 UTC
valgrind reports leaked (though still reachable) memory. I invoke it like this, from librepo/build/tests:

CK_FORK=no valgrind --leak-check=full --show-reachable=yes ./test_main ../../tests/test_data/

Some of these might be known GPGME leaks (see e.g. [1]), others might be real. Ideally, a fix for this will include a valgrind suppressions file checked into the upstream repo, which will contain those leaks that librepo can do nothing about. That way we can run valgrind with it occasionally and be sure nothing unexpected is happening to the memory.

[1] https://mailman.archlinux.org/pipermail/pacman-dev/2011-June/013398.html

Comment 1 Tomas Mlcoch 2013-04-08 12:05:57 UTC
Hi, I've added valgrind suppress file. This should be enough. In current state there are no real memory leaks.
https://github.com/Tojaj/librepo/commit/820749896f69eadf2fd8041b63abb2be906de260

Comment 2 Fedora Update System 2013-04-08 14:40:25 UTC
librepo-0.0.2-3.20130408git720d68d.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/librepo-0.0.2-3.20130408git720d68d.fc19

Comment 3 Fedora Update System 2013-04-20 19:28:48 UTC
librepo-0.0.2-3.20130408git720d68d.fc19 has been pushed to the Fedora 19 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.