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)
	Wed, Oct 15, 2:49 PM

	Unknown Object (File)
	Tue, Sep 23, 5:36 PM

	Unknown Object (File)
	Sep 18 2025, 6:06 AM

	Unknown Object (File)
	Sep 14 2025, 9:04 PM

	Unknown Object (File)
	Sep 14 2025, 8:19 PM

	Unknown Object (File)
	Sep 14 2025, 9:50 AM

	Unknown Object (File)
	Aug 21 2025, 6:32 AM

	Unknown Object (File)
	Jul 20 2025, 6:25 AM

View All 65 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