vmm: Remove an unneeded NULL pointer check
ClosedPublic
Actions

Authored by markj on Mon, Feb 2, 11:00 PM.

Details

Reviewers

corvink
bnovkov

Group Reviewers

bhyve

Commits

rG361b80310299: vmm: Remove an unneeded NULL pointer check
rG2d0564b9daf4: vmm: Remove an unneeded NULL pointer check

Summary

sc->vm is unconditionally dereferenced earlier in this function. No
functional change intended.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markj created this revision.Mon, Feb 2, 11:00 PM

Herald added a subscriber: imp. · View Herald TranscriptMon, Feb 2, 11:00 PM

Harbormaster completed remote builds in B70358: Diff 171031.Mon, Feb 2, 11:00 PM

markj requested review of this revision.Mon, Feb 2, 11:00 PM

markj added a parent revision: D55068: vmm: Fix a resource leak in an error path.

markj added a child revision: D55070: bhyve: Fix a misleading error message.

bnovkov accepted this revision as: bnovkov.Tue, Feb 3, 12:05 PM

This revision is now accepted and ready to land.Tue, Feb 3, 12:05 PM

Closed by commit rG2d0564b9daf4: vmm: Remove an unneeded NULL pointer check (authored by markj). · Explain WhyTue, Feb 3, 7:17 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rG2d0564b9daf4: vmm: Remove an unneeded NULL pointer check.

markj added a commit: rG361b80310299: vmm: Remove an unneeded NULL pointer check.Mon, Feb 16, 3:44 PM

Revision Contents
Changeset List

Path

Size

sys/

dev/

vmm/

vmm_dev.c

3 lines

Diff 171104

View Options

vmm: Remove an unneeded NULL pointer checkClosedPublicActions