Page MenuHomeFreeBSD

ata_xpt: Use the correct union member when accessing valid
ClosedPublic

Authored by brooks on Jul 19 2019, 10:29 PM.
Tags
None
Referenced Files
F109127806: D21002.id.diff
Sat, Feb 1, 4:16 AM
Unknown Object (File)
Thu, Jan 23, 6:08 PM
Unknown Object (File)
Tue, Jan 21, 2:43 PM
Unknown Object (File)
Jan 1 2025, 11:46 PM
Unknown Object (File)
Dec 20 2024, 10:09 PM
Unknown Object (File)
Dec 20 2024, 6:37 PM
Unknown Object (File)
Dec 12 2024, 8:02 PM
Unknown Object (File)
Dec 9 2024, 11:07 AM
Subscribers

Details

Summary

In principle this should not matter as it's a union and they point to
the same memory location but based on the code above we should be
accessing .sata and not .ata

Diff Detail

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

Event Timeline

I found this in our CheriBSD diffs. I'm not sure how @arichardson ran into it.

I'm not sure why I made that change. I just remember that we had some compiler bugs in that function that went away when I added a printf. I think this was just one of the suspicious things that I saw so I made the change hoping that it would fix the miscompilation.

This revision is now accepted and ready to land.Jul 20 2019, 6:07 AM