Page MenuHomeFreeBSD

acquire inp lock around ip6_pcbopt to fix IPV6_TCLASS panic
ClosedPublic

Authored by jason_eggnet.com on Jul 9 2018, 7:47 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Aug 22, 3:47 PM
Unknown Object (File)
Tue, Aug 18, 6:33 AM
Unknown Object (File)
Mon, Aug 17, 3:48 AM
Unknown Object (File)
Fri, Aug 14, 5:03 PM
Unknown Object (File)
Tue, Aug 11, 6:13 PM
Unknown Object (File)
Tue, Aug 4, 9:56 PM
Unknown Object (File)
Tue, Aug 4, 3:52 AM
Unknown Object (File)
Tue, Aug 4, 3:46 AM
Subscribers

Details

Summary

Hopefully simple fix to address panics relating to setting IPV6_TCLASS with setsockopt().

The premise of this fix is that it is ok to call malloc with M_NOWAIT while holding a lock on the in6p.

If it is not ok, then major surgery will be required, as ip6_setpktopt() will have to be fixed (as it also calls malloc with M_NOWAIT) which pulls in the ip6_pcbopts(), ip6_setpktopts(), ip6_setpktopt() call chain. The other call chain does have to be fixed but it can be done in a different review (and with relative ease) if this strategy works out.

Diff Detail

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

Event Timeline

This revision is now accepted and ready to land.Jul 10 2018, 11:24 PM
This revision was automatically updated to reflect the committed changes.