Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Various fixes #313

Merged
merged 4 commits into from
Jun 24, 2021
Merged

Various fixes #313

merged 4 commits into from
Jun 24, 2021

Conversation

fredericdalleau
Copy link
Contributor

This series add some fixes for various cases that have been reported that allow the guest to trigger a crash of the virtual machine.

The first two are NULL pointer dereference and are already handled in bhyve.
https://cgit.freebsd.org/src/tree/usr.sbin/bhyve/virtio.c#n606
https://cgit.freebsd.org/src/tree/usr.sbin/bhyve/virtio.c#n725

The other two are unhandled returned errors in vq_getchain. For now, trace and return as in pci_vt9p_notify.

Let me know what you think.

…21-054)

This is backport of what is done in bhyve

Signed-off-by: Frederic Dalleau <frederic.dalleau@docker.com>
…2021-055)

This is a backport of what is done in bhyve.

Signed-off-by: Frederic Dalleau <frederic.dalleau@docker.com>
Signed-off-by: Frederic Dalleau <frederic.dalleau@docker.com>
…1-057)

Signed-off-by: Frederic Dalleau <frederic.dalleau@docker.com>
Copy link
Collaborator

@djs55 djs55 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants