Bug 747003 - Coverity scan results
Summary: Coverity scan results
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: freeradius2
Version: 5.8
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: rc
: ---
Assignee: John Dennis
QA Contact: BaseOS QE Security Team
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-10-18 14:37 UTC by Michal Luscon
Modified: 2011-11-08 16:14 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-11-08 16:14:02 UTC
Target Upstream Version:

Attachments (Terms of Use)
patch for radius.c resource leak (500 bytes, patch)
2011-10-18 14:37 UTC, Michal Luscon 		no flags Details | Diff
patch for rlm_perl.c resource leak (464 bytes, patch)
2011-10-18 14:42 UTC, Michal Luscon 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Description Michal Luscon 2011-10-18 14:37:20 UTC 
Created attachment 528822 [details]
patch for radius.c resource leak

Coverity scan analyses has revealed an introduction of some newly added defects between RHEL-5.7 and RHEL-5.8 version of freeradius2 package.

#1 - Resource leak on radius.c:862

Static function VAULE_PAIR returns on the line #862 without freeing previously allocated variable tlv.

#2 - Resource leak on rlm_perl.c:428

The same as above one with variable embed.

#3 - Please check missing break on conffile.c:1564. I am not sure whether this break is necessary or not.

You can find my proposed solutions in the attachment section.

Version-Release number of selected component (if applicable):
freeradius2-2.1.12-1.el5
Comment 1 Michal Luscon 2011-10-18 14:42:43 UTC 
Created attachment 528823 [details]
patch for rlm_perl.c resource leak
Note You need to log in before you can comment on or make changes to this bug.