Differential D15156

vmmdev: return EFAULT when trying to read beyond VM system memory max address
ClosedPublic
Actions

Authored by antoine on Apr 22 2018, 7:30 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Nov 17, 11:43 AM

	Unknown Object (File)
	Wed, Oct 30, 1:30 AM

	Unknown Object (File)
	Sep 24 2024, 9:00 AM

	Unknown Object (File)
	Sep 19 2024, 4:20 PM

	Unknown Object (File)
	Sep 13 2024, 9:50 AM

	Unknown Object (File)
	Sep 12 2024, 1:46 PM

	Unknown Object (File)
	Sep 5 2024, 12:40 PM

	Unknown Object (File)
	Sep 4 2024, 1:03 PM

View All 37 Files

Subscribers

Details

Reviewers

rgrimes
imp
grehan
anish

Group Reviewers

Commits

rS333639: vmmdev: return EFAULT when trying to read beyond VM system memory max address

Summary

Currently, when using dd(1) to take a VM memory image, the capture never ends,
reading zeroes when it's beyond VM system memory max address.
Return EFAULT when trying to read beyond VM system memory max address.
With the patch, I'm able to take a memory image from a windows guest with dd(1)
and analyse it with tools like volatility or rekall.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

antoine created this revision.Apr 22 2018, 7:30 AM

Herald added a reviewer: rgrimes. · View Herald TranscriptApr 22 2018, 7:30 AM

Herald added a subscriber: imp. · View Herald Transcript

Harbormaster completed remote builds in B16253: Diff 41720.Apr 22 2018, 7:30 AM

imp accepted this revision.Apr 22 2018, 3:16 PM

This revision is now accepted and ready to land.Apr 22 2018, 3:16 PM

antoine added a reviewer: grehan.Apr 22 2018, 9:32 PM

This revision now requires review to proceed.Apr 22 2018, 9:32 PM

Good to go with the name change.

sys/amd64/include/vmm.h
215 ↗	(On Diff #41720)	Can the name for this be changed to vmm_sysmem_maxaddr ?

This revision is now accepted and ready to land.May 14 2018, 7:13 AM

anish accepted this revision.May 14 2018, 7:00 PM

Closed by commit rS333639: vmmdev: return EFAULT when trying to read beyond VM system memory max address (authored by antoine). · Explain WhyMay 15 2018, 5:21 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

amd64/

include/

1 line

vmm/

6 lines

5 lines

Diff 42591

head/sys/amd64/include/vmm.h

Loading...

head/sys/amd64/vmm/vmm.c

Loading...

head/sys/amd64/vmm/vmm_dev.c

Loading...