vmm: don't free unallocated memory
ClosedPublic
Actions

Authored by corvink on May 5 2023, 6:44 AM.

Tags

None

Referenced Files

	F108209330: D39974.id121555.diff
	Wed, Jan 22, 4:26 PM

	Unknown Object (File)
	Sun, Jan 12, 6:29 PM

	Unknown Object (File)
	Sat, Jan 11, 2:53 AM

	Unknown Object (File)
	Tue, Jan 7, 8:58 PM

	Unknown Object (File)
	Tue, Jan 7, 8:34 PM

	Unknown Object (File)
	Tue, Jan 7, 7:58 PM

	Unknown Object (File)
	Mon, Jan 6, 9:23 AM

	Unknown Object (File)
	Mon, Jan 6, 8:40 AM

Subscribers

Details

Reviewers

jhb
markj

Group Reviewers

bhyve

Commits

rGbe14baa34b82: vmm: don't free unallocated memory
rGb10e100d1696: vmm: don't free unallocated memory

Summary

If vmx or svm is disabled in BIOS or the device isn't supported by vmm,
modinit won't allocate these state save areas. As kmem_free panics when
passing a NULL pointer to it, loading the vmm kernel module causes a
panic too.

PR: 271251

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

corvink created this revision.May 5 2023, 6:44 AM

Herald added subscribers: bcran, rgrimes, imp. · View Herald TranscriptMay 5 2023, 6:44 AM

corvink requested review of this revision.May 5 2023, 6:44 AM

Harbormaster completed remote builds in B51337: Diff 121555.May 5 2023, 6:44 AM

Thanks!

This revision is now accepted and ready to land.May 5 2023, 1:15 PM

Closed by commit rGb10e100d1696: vmm: don't free unallocated memory (authored by corvink). · Explain WhyMay 5 2023, 1:34 PM

This revision was automatically updated to reflect the committed changes.

corvink added a commit: rGb10e100d1696: vmm: don't free unallocated memory.

markj added a commit: rGbe14baa34b82: vmm: don't free unallocated memory.May 11 2023, 1:33 PM

Revision Contents
Changeset List

Path

Size

sys/

amd64/

vmm/

amd/

svm.c

5 lines

intel/

vmx.c

4 lines

Diff 121572

View Options

sys/amd64/vmm/amd/svm.c