svn commit: r298199 - head/sys/netinet

[Michael Tuexen]

> On 18 Apr 2016, at 19:48, Ian Lepore <ian at FreeBSD.org> wrote:
> 
> On Mon, 2016-04-18 at 18:58 +0200, Michael Tuexen wrote:
>>> On 18 Apr 2016, at 17:10, Ian Lepore <ian at FreeBSD.org> wrote:
>>> 
>>> On Mon, 2016-04-18 at 09:29 +0000, Michael Tuexen wrote:
>>>> Author: tuexen
>>>> Date: Mon Apr 18 09:29:14 2016
>>>> New Revision: 298199
>>>> URL: https://svnweb.freebsd.org/changeset/base/298199
>>>> 
>>>> Log:
>>>> Put panic() calls under INVARIANTS.
>>>> 
>>> 
>>> A panic that's only active when INVARIANTS is defined is usually
>>> spelled KASSERT.
>> OK. The change happens because when using the code in the userland
>> SCTP stack, we don't provide a panic() replacement unless we
>> compile the INVARIANTS defined. Therefore I catch this when compiling
>> the userland stack...
>> 
>> Best regards
>> Michael
>>> 
> 
> Ahh, no KASSERT in userland, thanks.  I probably could have figured
> that out for myself if I had put a bit of effort into looking.
Well, we also have KASSERT in the userland code. All panic() calls
in the SCTP code are under INVARIANTS and only the last integrated
changes broke with the rule. I detected this when getting these
changes into the userland code and therefore I fixed it.
I'll try to convert them to KASSERTs sometime in the future. But
this has not high priority right now...

Thanks for making me aware of it.

Best regards
Michael
> 
> -- Ian
>