Page MenuHomeFreeBSD

netlink: fix size comparison
ClosedPublic

Authored by glebius on Jan 3 2025, 6:29 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 24, 5:02 AM
Unknown Object (File)
Sat, Nov 29, 1:09 AM
Unknown Object (File)
Nov 26 2025, 4:47 PM
Unknown Object (File)
Nov 25 2025, 5:25 PM
Unknown Object (File)
Nov 22 2025, 12:44 PM
Unknown Object (File)
Nov 6 2025, 7:44 AM
Unknown Object (File)
Nov 6 2025, 4:40 AM
Unknown Object (File)
Nov 6 2025, 4:29 AM
Subscribers

Details

Summary

We want to check the size of the header, not a pointer to it.

Diff Detail

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

Event Timeline

markj added a subscriber: markj.
markj added inline comments.
sys/netlink/netlink_generic.c
70–71

It's nicer IMO to use sizeof(*hdr), although netlink is inconsistent about this.

This revision is now accepted and ready to land.Jan 3 2025, 6:32 PM
sys/netlink/netlink_generic.c
70–71

I personally prefer sizeof(type) when coding protocols that exchange data, cause a protocol specification talks about types not about our local variable names. That makes it easier to check that code follow protocol spec. For other cases I prefer sizeof(*var), cause var type may change, and often does.

This revision was automatically updated to reflect the committed changes.