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)
	Dec 20 2023, 4:21 AM

	Unknown Object (File)
	Nov 13 2023, 9:51 PM

	Unknown Object (File)
	Oct 12 2023, 8:01 AM

	Unknown Object (File)
	Sep 30 2023, 7:27 PM

	Unknown Object (File)
	May 30 2023, 12:14 PM

	Unknown Object (File)
	Jan 13 2023, 10:45 PM

	Unknown Object (File)
	Dec 28 2022, 12:10 PM

	Unknown Object (File)
	Dec 20 2022, 1:29 PM

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