iommu_gas: avoid overflow in bounds check
ClosedPublic
Actions

Authored by dougm on Dec 20 2022, 9:03 PM.

Tags

None

Referenced Files

	F102190536: D37764.diff
	Fri, Nov 8, 5:49 PM

	Unknown Object (File)
	Mon, Oct 28, 7:11 PM

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

	Unknown Object (File)
	Sep 17 2024, 1:52 AM

	Unknown Object (File)
	Sep 16 2024, 4:01 PM

	Unknown Object (File)
	Sep 15 2024, 4:55 PM

	Unknown Object (File)
	Sep 2 2024, 6:21 PM

	Unknown Object (File)
	Aug 20 2024, 4:07 PM

View All 53 Files

Subscribers

imp

Details

Reviewers

andrew
alc

Commits

rG8765df60dc26: iommu_gas: avoid overflow in bounds check
rG5b9b55fbc432: iommu_gas: avoid overflow in bounds check

Summary

Change the range test in iommu_gas_match_one from '< ubound' to '<= ubound', and pass a smaller-by-one ubound parameter to it, to avoid overflow in ubound calculation.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

dougm requested review of this revision.Dec 20 2022, 9:03 PM

dougm created this revision.

andrew accepted this revision.Dec 21 2022, 11:18 AM

This revision is now accepted and ready to land.Dec 21 2022, 11:18 AM

Correct bounds check, where one was added when it should have been subtracted.

This revision now requires review to proceed.Dec 21 2022, 9:30 PM

This revision was not accepted when it landed; it landed in state Needs Review.Dec 22 2022, 8:34 PM

Closed by commit rG5b9b55fbc432: iommu_gas: avoid overflow in bounds check (authored by dougm). · Explain Why

This revision was automatically updated to reflect the committed changes.

dougm added a commit: rG5b9b55fbc432: iommu_gas: avoid overflow in bounds check.

Herald added a subscriber: imp. · View Herald TranscriptDec 22 2022, 8:34 PM

dougm added a commit: rG8765df60dc26: iommu_gas: avoid overflow in bounds check.Dec 25 2022, 7:02 AM

Revision Contents
Changeset List

Path

Size

sys/

dev/

iommu/

iommu_gas.c

18 lines

Diff 114339

View Options

iommu_gas: avoid overflow in bounds checkClosedPublicActions