Thanks for the feedback about cleanup. In hindsight, yes, the cleanup did go a little farther than I intended. I could split it into two commits, but I'd have to re-test, twice. I'll try to be more restrained next time. It was just so satisfying.

• SCTP handles PMTU changes, too

Coverity often surprises me, in both cleverness and ignorance. I suppose the same can be said of all software, and indeed, of most human pursuits.

I share kib's curiosity, but it clearly works, so LGTM. Nice find!

Thank you for asking, but I didn't actually do the last update. @jasone did it, but had to revert it. I simply un-reverted his work.

• remove DT_CLOSE_HANG, which never happens on FreeBSD

• rebase onto D41173; store flags in dt_proc

In light of @hselasky's untimely departure, would someone on network like to review this, or suggest another reviewer?

• fix comment style

What do you think, @markj?

The DTrace test suite had fewer failures with this change. They were not fixed by this change; they're just unreliable.

• wordsmith comment

This needs a change in etc/mtree/BSD.tests.dist and perhaps ObsoleteFiles.inc.

In D40260#919297, @emaste wrote:

I have no objection to this approach (but have not reviewed the diff in detail yet).

Thank you both for the feedback. I like the idea of using OpenBSD as an upstream, but that would take a lot more time than I have right now. (Most or all of my FreeBSD time is on Dell's clock.)

I think the best approach would be to import through the vendor branch and put it in contrib/cron, and altering the makefiles to contrib/cron instead. The vendor branch provides a better degree of maintainability.

please add the CID to the commit message
(unless this is not from Coverity Scan's FreeBSD report)

It's not clear to me whether this is actually possible or just theoretical.

Common practice for attributing the submitter with git is: