pf: Enable the syncookie feature for IPv6 and reduce indentation

[kp]

I didn't realise we didn't support synproxy on IPv6, and I'm a little confused, given that we have the pf/synproxy:local_v6 test case which appears to pass.

I'll take a deeper look sometime next week. If you get a moment can you create a test case like synproxy:synproxy for IPv6?

[vegeta_tuxpowered.net]

In D41502#945583, @kp wrote:

I didn't realise we didn't support synproxy on IPv6, and I'm a little confused, given that we have the pf/synproxy:local_v6 test case which appears to pass.

I'll take a deeper look sometime next week. If you get a moment can you create a test case like synproxy:synproxy for IPv6?

I've named the commit wrong. Sorry, this is for syncookie, not for synproxy.

[kp]

Ah, indeed.
I also thought that there was IPv6 support there, but clearly I was wrong.

Same question about the test though ;)

At first glance that looks sane enough. I may end up splitting this into two commits (one for the indentation, one for the IPv6 support). Again, sometime next week. If I've not committed (or taken issue with the commit, but I doubt that) by the 28th feel free to shout at me.

[vegeta_tuxpowered.net]

Add the missing "Respond to SYN with a syncookie" part.

[kp]

I’ve got a couple of ipv6 test cases, and the nostate one does at least fail (without your patches).
So you don’t need to bother with tests. I’ll commit mine with your patches.

[kp]

Also, is this Sponsored by: InnoGames GmbH?

[vegeta_tuxpowered.net]

In D41502#945945, @kp wrote:

Also, is this Sponsored by: InnoGames GmbH?

Yes it is.