Fix sign-extension error in adagetparams
ClosedPublic
Actions

Authored by asomers on Nov 22 2017, 10:20 PM.

Tags

None

Referenced Files

	F135917422: D13198.id36209.diff
	Fri, Nov 14, 3:34 AM

	Unknown Object (File)
	Fri, Nov 7, 2:35 AM

	Unknown Object (File)
	Sun, Oct 26, 2:10 PM

	Unknown Object (File)
	Sun, Oct 26, 2:57 AM

	Unknown Object (File)
	Sat, Oct 25, 12:42 AM

	Unknown Object (File)
	Sat, Oct 25, 12:39 AM

	Unknown Object (File)
	Sun, Oct 19, 9:18 PM

	Unknown Object (File)
	Thu, Oct 16, 5:34 PM

View All 84 Files

Subscribers

imp

Details

Reviewers

ken

Commits

rS327849: MFC r326624:
rS326624: cam: fix sign-extension error in adagetparams

Summary

Fix sign-extension error in adagetparams

adagetparams contains a sign-extension error that will cause the sector
count to be incorrectly calculated for ATA disks of >=1TiB that still use
CHS addressing. Disks using LBA48 addressing are unaffected.

Reported by: Coverity
CID: 1007296

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

asomers created this revision.Nov 22 2017, 10:20 PM

Herald added a subscriber: imp. · View Herald TranscriptNov 22 2017, 10:20 PM

imp added inline comments.Nov 22 2017, 10:39 PM

sys/cam/ata/ata_da.c
3378 ↗	(On Diff #35618)	cut and paste artifact, no? otherwise \ doesn't make sense here.

asomers added inline comments.Dec 4 2017, 10:47 PM

sys/cam/ata/ata_da.c
3378 ↗	(On Diff #35618)	<sigh> not a copy/paste bug. That's just me writing too much Ruby and Go.

Fix syntax oddity

Harbormaster completed remote builds in B13336: Diff 36209.Dec 4 2017, 10:48 PM

Looks good to me, thanks!

This revision is now accepted and ready to land.Dec 6 2017, 4:36 PM

Closed by commit rS326624: cam: fix sign-extension error in adagetparams (authored by asomers). · Explain WhyDec 6 2017, 5:01 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

cam/

ata/

ata_da.c

3 lines

Diff 36306

View Options

Fix sign-extension error in adagetparamsClosedPublicActions