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

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

Tags

None

Referenced Files

	F147374001: D9191.id24037.diff
	Tue, Mar 10, 10:44 AM

	Unknown Object (File)
	Fri, Mar 6, 8:13 PM

	Unknown Object (File)
	Thu, Mar 5, 8:40 PM

	Unknown Object (File)
	Thu, Mar 5, 2:48 AM

	Unknown Object (File)
	Tue, Mar 3, 4:39 PM

	Unknown Object (File)
	Jan 28 2026, 2:15 AM

	Unknown Object (File)
	Jan 16 2026, 10:22 PM

	Unknown Object (File)
	Jan 16 2026, 7:27 PM

View All 84 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 Not Applicable

Unit

Tests Not Applicable

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

head/

sys/

boot/

efi/

loader/

main.c

11 lines

Diff 24038

View Options

head/sys/boot/efi/loader/main.c

loader.efi: find_currdev() can leak memoryClosedPublicActions