Page MenuHomeFreeBSD

efivar: Add some explicit casts to uuid_t
ClosedPublic

Authored by imp on Apr 25 2025, 7:33 PM.
Tags
None
Referenced Files
F136011738: D50032.id.diff
Sat, Nov 15, 1:52 AM
F135988407: D50032.id154302.diff
Fri, Nov 14, 8:53 PM
Unknown Object (File)
Fri, Nov 14, 12:27 AM
Unknown Object (File)
Fri, Nov 7, 5:18 AM
Unknown Object (File)
Thu, Nov 6, 12:30 PM
Unknown Object (File)
Mon, Nov 3, 10:59 AM
Unknown Object (File)
Thu, Oct 30, 2:18 PM
Unknown Object (File)
Sun, Oct 26, 1:24 PM
Subscribers
None

Details

Summary

Since we know we're type-punning the EFI_GUID to a uuid_t, add some
explicit casts even though, at the moment, we typedef one to the other.

Sponsored by: Netflix

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

imp requested review of this revision.Apr 25 2025, 7:33 PM
imp created this revision.

I would add some type-checking for the conversion. E.g you could define inlines uuid_t * efi_guid_to_uuid(efi_guid_t *) and const uuid_t *const_efi_guid_to_uuid(const efi_guid_t *) and use them instead of manually writing the cast.

This revision was not accepted when it landed; it landed in state Needs Review.May 1 2025, 6:06 PM
This revision was automatically updated to reflect the committed changes.