Page MenuHomeFreeBSD
Differential D15839

Correct logic error
ClosedPublic
Actions

Authored by allanjude on Jun 16 2018, 1:47 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, May 17, 10:56 PM
Unknown Object (File)
Sun, May 17, 10:56 PM
Unknown Object (File)
Sun, May 17, 10:42 PM
Unknown Object (File)
Apr 25 2026, 6:02 AM
Unknown Object (File)
Apr 22 2026, 10:37 AM
Unknown Object (File)
Apr 19 2026, 12:44 PM
Unknown Object (File)
Apr 18 2026, 2:30 PM
Unknown Object (File)
Apr 12 2026, 5:02 PM
View All 68 Files
Subscribers
None

Details

Reviewers
imp
tsoome
kevans
• ian
Commits
rS335245: Correct logic error in biosdisk.c:bd_realstrategy()
Summary

While investigating an issue with GELI boot, I found what looks like a logic error. The contents of the if statement are definately meant to run if disk_ioctl successed, but actually only run if it fails. This also results in the 'We should not get here' code running in most cases

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

allanjude created this revision.Jun 16 2018, 1:47 AM
Harbormaster completed remote builds in B17322: Diff 43859.Jun 16 2018, 1:47 AM
imp accepted this revision.Jun 16 2018, 2:46 AM

Yes sir! We have a winner!

This revision is now accepted and ready to land.Jun 16 2018, 2:46 AM
Closed by commit rS335245: Correct logic error in biosdisk.c:bd_realstrategy() (authored by allanjude). · Explain WhyJun 16 2018, 2:50 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
stand/
i386/
libi386/
4 lines

Diff 43861

View Options

head/stand/i386/libi386/biosdisk.c

Loading...