[#STDCXX-583] purify reports memory leak inside rw_putenv()

C++ Standard Library

purify reports memory leak inside rw_putenv()

Created: 09/Oct/07 10:51 PM Updated: 11/Jul/09 12:06 AM

Return to search

Component/s:

Test Driver

Affects Version/s:

4.2.0

Fix Version/s:

4.3.0

Time Tracking:

Original Estimate:

Remaining Estimate:

Time Spent:

Not Specified

File Attachments:

	File Name	Date Attached	Attached By	Size
	0.printf.log	2007-10-09 10:52 PM	Travis Vitek	11 kB

Severity:

Resource Leak

Description

« Hide

rw_putenv() uses putenv() to add variables to the environment. Unfortunately putenv() actually places the user provided string into the environment, so the string cannot be deallocated immediately. The result is false positive memory leaks in tests that use rw_putenv().

One solution would be to use setenv() and unsetenv() on platforms that support them. This avoids the memory leak. Another option would be to have rw_putenv() track the environment strings that were added, then remove and deallocate them when the test driver is taken down.

All

Comments

Work Log

Change History

Subversion Commits

Sort Order:

[ Permlink | « Hide ]

Travis Vitek added a comment - 09/Oct/07 10:52 PM

Attaching purify log for 0.printf test.

Travis Vitek made changes - 09/Oct/07 10:52 PM

Field	Original Value	New Value
Attachment		0.printf.log [ 12367406 ]

[ Permlink | « Hide ]

Martin Sebor added a comment - 20/May/08 07:23 PM

Scheduled for 4.3 when we'll need to re-run Purify against newly added code and assigned to Travis.

Martin Sebor made changes - 20/May/08 07:23 PM

Affects Version/s		4.2.0 [ 12311945 ]
Fix Version/s		4.3 [ 12312692 ]

[ Permlink | « Hide ]

Martin Sebor added a comment - 02/Jun/08 07:23 PM

Travis, can you check to see if this is still a problem and decide if it's worth our while to try to do something about it?

Martin Sebor made changes - 02/Jun/08 07:23 PM

Assignee

Travis Vitek [ vitek ]

[ Permlink | « Hide ]

Martin Sebor added a comment - 02/Jun/08 07:24 PM

Set severity and estimated effort.

Martin Sebor made changes - 02/Jun/08 07:24 PM

Severity	Cosmetic	Resource Leak
Remaining Estimate		2h [ 7200 ]
Original Estimate		2h [ 7200 ]

Travis Vitek made changes - 11/Jul/09 12:06 AM

Assignee

Travis Vitek [ vitek ]