This is an archive of the discontinued LLVM Phabricator instance.

Remove struct_rtentry_sz on FreeBSD
ClosedPublic

Authored by dim on Feb 10 2017, 9:42 AM.

Details

Summary

Since struct rtentry is an internal kernel-only structure on FreeBSD,
and SIOCADDRT and SIOCDELRT are not supported anyway, stop including
socketvar.h and attempting to get at the definition of struct rtentry,
and move the line with struct_rtentry_sz to the SANIZER_LINUX block.

Event Timeline

dim created this revision.Feb 10 2017, 9:42 AM
kcc accepted this revision.Feb 10 2017, 3:42 PM

LGTM

This revision is now accepted and ready to land.Feb 10 2017, 3:42 PM
emaste accepted this revision.Feb 10 2017, 3:55 PM
dim closed this revision.Feb 10 2017, 4:00 PM
dim added a subscriber: hans.Feb 10 2017, 4:00 PM

As this is a pretty minor fix, affecting FreeBSD only, I'd like to merge it to release_40. @hans, any objections?

hans added a comment.Feb 10 2017, 4:58 PM
In D29832#674150, @dim wrote:

As this is a pretty minor fix, affecting FreeBSD only, I'd like to merge it to release_40. @hans, any objections?

Sounds good to me. Please let the buidlbots chew on it for a bit, and then go ahead and merge.