Bug 752951 - totemip_parse leaks memory
Summary: totemip_parse leaks memory
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 6
Classification: Red Hat
Component: corosync
Version: 6.2
Hardware: All
OS: Linux
low
low
Target Milestone: rc
: ---
Assignee: Jan Friesse
QA Contact: Cluster QE
URL:
Whiteboard:
Depends On:
Blocks: 752952 756082
TreeView+ depends on / blocked
 
Reported: 2011-11-10 20:28 UTC by Steven Dake
Modified: 2016-04-26 15:16 UTC (History)
2 users (show)

Fixed In Version: corosync-1.4.1-5.el6
Doc Type: Bug Fix
Doc Text:
Cause Corosync is running Consequence Corosync leaks small amount of memory in long term runs inside getaddrinfo function Fix Free leaked memory by calling freeadrrinfo function Result Corosync no longer leaks memory returned by getaddrinfo
Clone Of:
: 752952 (view as bug list)
Environment:
Last Closed: 2012-06-20 12:22:59 UTC
Target Upstream Version:

Attachments (Terms of Use)
Backported patch (793 bytes, patch)
2012-01-31 08:35 UTC, Jan Friesse 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2012:0777 0 normal SHIPPED_LIVE corosync bug fix and enhancement update 2012-06-19 20:35:04 UTC

Description Steven Dake 2011-11-10 20:28:02 UTC 
Description of problem:
totemip_parse uses getaddrinfo without using an associated freeaddrinfo call.

Version-Release number of selected component (if applicable):
corosync-1.4.1-1.el6

How reproducible:
requires valgrind analysis

Steps to Reproduce:
1.
2.
3.
  
Actual results:
some minimal amount of memory will be leaked when this api is called.  This function is not called often, but is called enough that over several months it could add up.

Expected results:
no leaks with getaddrinfo.

Additional info:
[corosync] [PATCH 1/7] Free mem allocated by getaddrinfo
Comment 3 Jan Friesse 2012-01-31 08:35:39 UTC 
Created attachment 558564 [details]
Backported patch

Free mem allocated by getaddrinfo
Comment 5 Jan Friesse 2012-03-07 08:49:17 UTC 
    Technical note added. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    New Contents:
Cause
Corosync is running

Consequence
Corosync leaks small amount of memory in long term runs inside getaddrinfo function

Fix
Free leaked memory by calling freeadrrinfo function

Result
Corosync no longer leaks memory returned by getaddrinfo
Comment 8 errata-xmlrpc 2012-06-20 12:22:59 UTC 
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHBA-2012-0777.html
Note You need to log in before you can comment on or make changes to this bug.