Fix a memory leak in libctf
ClosedPublic
Actions

Authored by trix_juniper.net on Feb 23 2017, 4:41 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Oct 9 2024, 1:21 AM

	Unknown Object (File)
	Oct 6 2024, 2:11 AM

	Unknown Object (File)
	Oct 5 2024, 6:11 PM

	Unknown Object (File)
	Oct 5 2024, 3:42 PM

	Unknown Object (File)
	Oct 3 2024, 12:05 AM

	Unknown Object (File)
	Oct 2 2024, 11:05 PM

	Unknown Object (File)
	Oct 1 2024, 3:12 AM

	Unknown Object (File)
	Sep 26 2024, 9:09 AM

View All 32 Files

Subscribers

imp

Details

Reviewers

markj
stevek

Summary

ctf_lib.c ctf_fdopen has a memory leak, when the allocation of sp32 is sucessful but next statement fails.

Free sp32 before returning. Assume free handles a null pointer.

Test Plan

Run clang's static analyzer, scan-build, before to reproduce the problem, run after to verify proble resolution.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

trix_juniper.net updated this revision to Diff 25625.Feb 23 2017, 4:41 PM

trix_juniper.net retitled this revision from to Fix a memory leak in libctf.

trix_juniper.net updated this object.

trix_juniper.net edited the test plan for this revision. (Show Details)

trix_juniper.net added reviewers: stevek, markj.

trix_juniper.net set the repository for this revision to rS FreeBSD src repository - subversion.

Herald added a subscriber: imp. · View Herald TranscriptFeb 23 2017, 4:41 PM

Looks good to me, thanks. Would you like me to commit this and the libdtrace change?

This revision is now accepted and ready to land.Feb 23 2017, 5:32 PM

Yes, that would be helpful.
At this time, I do not have any more static analysis fixes for dtrace.
Thanks
Tom

r314153

Revision Contents
Changeset List

Path

Size

cddl/

contrib/

opensolaris/

lib/

libctf/

common/

ctf_lib.c

1 line

Diff 25625

View Options

cddl/contrib/opensolaris/lib/libctf/common/ctf_lib.c

Fix a memory leak in libctfClosedPublicActions