loader.efi: find_currdev() can leak memory
ClosedPublic
Actions

Authored by tsoome on Jan 15 2017, 7:31 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Jan 19, 3:05 PM

	Unknown Object (File)
	Sun, Jan 12, 9:03 PM

	Unknown Object (File)
	Sat, Jan 11, 5:25 AM

	Unknown Object (File)
	Dec 20 2024, 9:00 PM

	Unknown Object (File)
	Nov 4 2024, 8:22 PM

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

	Unknown Object (File)
	Sep 16 2024, 12:58 PM

	Unknown Object (File)
	Sep 8 2024, 10:01 AM

View All 45 Files

Subscribers

None

Details

Reviewers

eric_metricspace.net
allanjude
imp

Commits

rS312237: loader.efi: find_currdev() can leak memory

Summary

The find_currdev() is using variable "copy" to store the reference to trimmed devpath pointer,
if for some reason the efi_devpath_handle() fails, we will leak this copy.

Also we can simplify the code there a bit.

Test Plan

build and boot

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 6788
Build 6999: arc lint + arc unit

Event Timeline

tsoome updated this revision to Diff 24037.Jan 15 2017, 7:31 PM

tsoome retitled this revision from to loader.efi: find_currdev() can leak memory.

tsoome updated this object.

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

tsoome added reviewers: allanjude, imp.

Approved for commit

This revision is now accepted and ready to land.Jan 15 2017, 7:40 PM

Closed by commit rS312237: loader.efi: find_currdev() can leak memory (authored by tsoome). · Explain WhyJan 15 2017, 8:03 PM

This revision was automatically updated to reflect the committed changes.

allanjude added a reviewer: eric_metricspace.net.Jan 15 2017, 9:02 PM

Revision Contents
Changeset List

Path

Size

sys/

boot/

efi/

loader/

main.c

11 lines

Diff 24037

View Options

sys/boot/efi/loader/main.c

loader.efi: find_currdev() can leak memoryClosedPublicActions