Differential D37486

bhyve: Don't leak uninitialized bits in NVMe completion statuses.
ClosedPublic
Actions

Authored by jhb on Nov 23 2022, 11:16 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Jul 22, 11:44 PM

	Unknown Object (File)
	Wed, Jul 22, 6:26 PM

	Unknown Object (File)
	Tue, Jul 21, 3:31 AM

	Unknown Object (File)
	Mon, Jul 13, 8:27 PM

	Unknown Object (File)
	Mon, Jul 13, 8:27 PM

	Unknown Object (File)
	Sat, Jul 4, 4:54 PM

	Unknown Object (File)
	Wed, Jul 1, 2:07 AM

	Unknown Object (File)
	Mon, Jun 29, 1:21 AM

Subscribers

Details

Reviewers

markj
corvink
chuck

Group Reviewers

Commits

rG3b700550004c: bhyve: Don't leak uninitialized bits in NVMe completion statuses.
rG1d9e8a9e6095: bhyve: Don't leak uninitialized bits in NVMe completion statuses.

Summary

In some cases, some bits in the 16-bit status word were never
initialized.

Reported by: GCC

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 48477
Build 45363: arc lint + arc unit

Event Timeline

jhb created this revision.Nov 23 2022, 11:16 PM

Herald added a reviewer: bhyve. · View Herald TranscriptNov 23 2022, 11:16 PM

Herald added subscribers: dab, bcran, rgrimes, imp. · View Herald Transcript

jhb requested review of this revision.Nov 23 2022, 11:16 PM

Harbormaster completed remote builds in B48477: Diff 113467.Nov 23 2022, 11:16 PM

jhb added a parent revision: D37485: bhyve: Fix sign compare warnings in the e1000 device model..Nov 23 2022, 11:16 PM

jhb added a child revision: D37487: bhyve: Clear lid to 0 for internal device errors for NVMe AENs..

corvink accepted this revision.Nov 24 2022, 6:31 AM

This revision is now accepted and ready to land.Nov 24 2022, 6:31 AM

jhb added a reviewer: chuck.Nov 24 2022, 5:24 PM

LGTM

I'm surprised that clang didn't flag these. I've noticed it has trouble detecting uninitialized variable use in functions containing gotos, but that doesn't apply here.

Closed by commit rG1d9e8a9e6095: bhyve: Don't leak uninitialized bits in NVMe completion statuses. (authored by jhb). · Explain WhyNov 29 2022, 1:11 AM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rG1d9e8a9e6095: bhyve: Don't leak uninitialized bits in NVMe completion statuses..

jhb added a commit: rG3b700550004c: bhyve: Don't leak uninitialized bits in NVMe completion statuses..Jan 26 2023, 8:28 PM

Revision Contents
Changeset List

Path

Size

usr.sbin/

bhyve/

4 lines

Diff 113467

usr.sbin/bhyve/pci_nvme.c

Loading...