Paths

Table of Contentst

Fix additional memory leak in process_mapfile
ClosedPublic
Actions

Authored by bret_ketchum_dell.com on Sep 17 2020, 4:10 PM.

Tags

None

Referenced Files

	F154899416: D26462.id77136.diff
	Wed, Apr 29, 9:29 PM

	F154871865: D26462.id77156.diff
	Wed, Apr 29, 4:03 PM

	Unknown Object (File)
	Tue, Apr 28, 4:13 AM

	Unknown Object (File)
	Mon, Apr 27, 7:19 AM

	Unknown Object (File)
	Fri, Apr 24, 6:19 AM

	Unknown Object (File)
	Sat, Apr 18, 8:21 AM

	Unknown Object (File)
	Thu, Apr 9, 6:51 PM

	Unknown Object (File)
	Tue, Apr 7, 12:09 AM

View All Files

Subscribers

imp

Details

Reviewers

cem
emaste
vangyzen

Commits

rS365845: Fix additional memory leak in process_mapfile

Summary

Additional Coverity detected memory leak fix.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bret_ketchum_dell.com requested review of this revision.Sep 17 2020, 4:10 PM

bret_ketchum_dell.com created this revision.

vangyzen accepted this revision.Sep 17 2020, 6:02 PM

This revision is now accepted and ready to land.Sep 17 2020, 6:02 PM

emaste accepted this revision.Sep 17 2020, 6:18 PM

Thanks.

lib/libpmc/pmu-events/jevents.c
783–806 ↗	(On Diff #77136)	If `goto out` is acceptable at ~786, maybe we can just do that here too and use a variable for the return value. This fix seems ok as-is, though. Either way.

Closed by commit rS365845: Fix additional memory leak in process_mapfile (authored by vangyzen). · Explain WhySep 17 2020, 6:25 PM

This revision was automatically updated to reflect the committed changes.

vangyzen added a commit: rS365845: Fix additional memory leak in process_mapfile.

Herald added a subscriber: imp. · View Herald TranscriptSep 17 2020, 6:25 PM

Revision Contents
Changeset List

Path

Size

head/

lib/

libpmc/

pmu-events/

jevents.c

2 lines

Diff 77156

View Options

head/lib/libpmc/pmu-events/jevents.c

Fix additional memory leak in process_mapfileClosedPublicActions