ha: Delete cache entry inside the locked mutex

Signed-off-by: Thomas Egerer <thomas.egerer@secunet.com>

swanctl: Fix documented directory name for remote pubkeys

Version bump to 5.4.0

kernel-netlink: Fix lookup of next hops for destinations with prefix

References #1347.

imc-os: Terminate buffer after fread(3) call to make Coverity happy

imc-os: Correctly check return value of ftell(2)

Fix some Doxygen issues

Updated NEWS

man: Updated default proposals in ipsec.conf(5)

identification: Make `written` signed to fix error checking when printing ranges

vici: Don't hold write lock while running or undoing start actions

Running or undoing start actions might require enumerating IKE_SAs,
which in turn might have to enumerate peer configs concurrently, which
requires acquiring a read lock.  So if we keep holding the write lock while
enumerating the SAs we provoke a deadlock.

By preventing other threads from acquiring the write lock while handling
actions, and thus preventing the modification of the configs, we largely
maintain the current synchronous behavior.  This way we also don't need to
acquire additional refs for config objects as they won't get modified/removed.

Fixes #1185.

Initialize ts variable

forecast: Compare the complete rules when deleting them

Same as the change in the connmark plugin.

References #1229.

connmark: Don't restore CONNMARK for packets that already have a mark set

This allows e.g. modified versions of xl2tpd to set the mark in
situations where two clients are using the same source port behind the
same NAT, which CONNMARK can't restore properly as only one conntrack entry
will exist with the mark set to that of the client that sent the last packet.

Fixes #1230.

connmark: Compare the complete rules when deleting them

By settings a matchmask that covers the complete rule we ensure that the
correct rule is deleted (i.e. matches and targets with potentially different
marks are also compared).

Since data after the passed pointer is actually dereferenced when
comparing we definitely have to pass an array that is at least as long as
the ipt_entry.

Fixes #1229.

Merge branch 'subnet-identities'

Implemented IKEv1 IPv4/IPv6 address subnet and range identities to
be used as owners for shared secrets.

swanctl supports configuration of traffic selectors with IPv4/IPv6
address ranges.

Support of IP address ranges in traffic selectors

Updated swanctl/rw-psk-ikev1 scenario

Implemented IPv4/IPv6 subnet and range identities

The IKEv1 IPV4_ADDR_SUBNET, IPV6_ADDR_SUBNET, IPV4_ADDR_RANGE and
IPV6_ADDR_RANGE identities have been fully implemented and can be
used as owners of shared secrets (PSKs).

Merge branch 'p-cscf'

This adds the p-cscf plugin that can request P-CSCF server addresses from
an ePDG via IKEv2 (RFC 7651).  Addresses of the same families as requested
virtual IPs are requested if enabled in strongswan.conf for a particular
connection.  The plugin currently writes received addresses to the log.

attr: Only enumerate attributes matching the IKE version of the current IKE_SA

Numerically configured attributes are currently sent for both versions.

attr: Add p-cscf keyword for P-CSCF server addresses

p-cscf: Make sending requests configurable and disable it by default

p-cscf: Only send requests if virtual IPs of the same family are requested

p-cscf: Add attribute handler for P-CSCF server addresses

p-cscf: Add plugin stub

payloads: Verify P-CSCF configuration attributes like others carrying IP addresses

attributes: Define P-CSCF address attributes described in RFC 7651

Merge branch 'mbb-reauth-online-revocation'

With these changes initiators of make-before-break reauthentications
suspend online revocation checks until after the new IKE_SA and all
CHILD_SAs are established.  See f1cbacc5d1be for details why that's
necessary.

NEWS: Added note on online revocation checks during make-before-break reauthentication

testing: Add ikev2/reauth-mbb-revoked scenario

testing: Generate a CRL that has moon's actual certificate revoked

ike-sa: Improve interaction between flush_auth_cfg and delayed revocation checks

ikev2: Delay online revocation checks during make-before-break reauthentication

We do these checks after the SA is fully established.

When establishing an SA the responder is always able to install the
CHILD_SA created with the IKE_SA before the initiator can do so.
During make-before-break reauthentication this could cause traffic sent
by the responder to get dropped if the installation of the SA on the
initiator is delayed e.g. by OCSP/CRL checks.

In particular, if the OCSP/CRL URIs are reachable via IPsec tunnel (e.g.
with rightsubnet=0.0.0.0/0) the initiator is unable to reach them during
make-before-break reauthentication as it wouldn't be able to decrypt the
response that the responder sends using the new CHILD_SA.

By delaying the revocation checks until the make-before-break
reauthentication is completed we avoid the problems described above.
Since this only affects reauthentication, not the original IKE_SA, and the
delay until the checks are performed is usually not that long this
doesn't impose much of a reduction in the overall security.

ikev2: Add task that verifies a peer's certificate

On failure the SA is deleted and reestablished as configured.  The task
is activated after the REAUTH_COMPLETE task so a make-before-break reauth
is completed before the new SA might get torn down.

ikev2: Initiate other tasks after a no-op task

ikev2: Don't do online revocation checks in pubkey authenticator if requested

We also update the auth config so the constraints are not enforced.

ike-sa: Add condition to suspend online certificate revocation checks for an IKE_SA

ike-sa: Add method to verify certificates in completed authentication rounds

auth-cfg: Add a rule to suspend certificate validation constraints

credential-manager: Check cache queue when destroying trusted certificate enumerator

We already do this in the trusted public key enumerator (which
internally uses the trusted certificate enumerator) but should do so
also when this enumerator is used directly (since the public key
enumerator has the read lock the additional call will just be skipped
there).

credential-manager: Make online revocation checks optional for public key enumerator

Merge branch 'charon-conf-fallback'

Makes charon-systemd and charon-svc also load settings from the charon
section in strongswan.conf.

Fixes #1300.

charon-svc: Inherit all settings from the charon section

Same as with charon-systemd.

charon-systemd: Inherit all settings from the charon section

Our default config files are very charon specific.  So to avoid
confusion when only charon-systemd is installed we just default to all
settings defined for charon.  Since charon-systemd probably won't be used
together with charon this should not cause conflicts (settings may still
be overridden via the charon-systemd section).

library: Add option to register additional namespaces before calling library_init()

Because settings are already accessed in library_init(), calling
add_fallback() externally after calling library_init() is not ideal.
This way namespaces already serve as fallback while library_init() is
executed and they are also in the correct order so that libstrongswan is
always the last root section.

vici: Replace child configs atomically

This also leaves unmodified configs as they are.

peer-cfg: Add method to atomically replace child configs

ike-cfg: Use new method to compare proposal lists in equals()

peer-cfg: Use new method to compare linked lists in equals()

This also compares the complete lists not only the first two items.

child-cfg: Add equals() method

linked-list: Add method to compare two lists of objects for equality

vici: Order auth rounds by optional `round` parameter instead of by position in the request

ikev1: Send NAT-D payloads after vendor ID payloads in Aggressive Mode messages

Some implementations might otherwise not recognize the NAT-D payload
type.  Also moves SIG and HASH payloads last in these messages.

Fixes #1239.

ike-sa-manager: Log a checkin/failure message for every checkout

Signed-off-by: Thomas Egerer <thomas.egerer@secunet.com>

testing: Added swanctl/mult-auth-rsa-eap-sim-id scenario

testing: Added swanctl/xauth-rsa scenario

Display IKE ports with swanctl --list-sas

Version bump to 5.4.0rc1

testing: attr-sql is a charon plugin

testing: Added swanctl/rw-psk-ikev1 scenario

testing: Include IKE port information in evaltests

Version bump to 5.4.0dr8

ike-sa-manager: Log some additional details like SPIs when checking out SAs

smp: Correctly return IKE SPIs stored in network order

vici: Correctly return IKE SPIs stored in network order

stroke: Correctly print IKE SPIs stored in network order

byteorder: Simplify htoun64/untoh64 functions

byteorder: Always define be64toh/htobe64 macros

Merge branch 'ike-sig-contraints'

Signature scheme constraints against IKEv2 authentication may now be
configured independently of constraints against trustchains.

NEWS: Add note about IKEv2 signature scheme constraints

swanctl: Document signature scheme constraints

vici: Add support for pubkey constraints with EAP-TLS

This is a feature currently supported by stroke.

auth-cfg: Make IKE signature schemes configurable

This also restores the charon.signature_authentication_constraints
functionality, that is, if no explicit IKE signature schemes are
configured we apply all regular signature constraints as IKE constraints.

ikev2: Always store signature scheme in auth-cfg

As we use a different rule we can always store the scheme.

ikev2: Diversify signature scheme rule

This allows for different signature schemes for IKE authentication and
trustchain verification.

Signed-off-by: Thomas Egerer <thomas.egerer@secunet.com>

NEWS: Document RFC 5685 support

Merge branch 'ike-redirect'

This adds support for IKEv2 redirection (RFC 5685).  There is currently
no default implementation of the redirect_provider_t interface provided.
Plugins may implement the interface to decide if and when to redirect
connecting clients.  It is also possible to redirect established IKE_SAs
via VICI/swanctl.

ike-init: Verify REDIRECT notify before processing IKE_SA_INIT message

An attacker could blindly send a message with invalid nonce data (or none
at all) to DoS an initiator if we just destroy the SA.  To prevent this we
ignore the message and wait for the one by the correct responder.

ikev2: Allow tasks to verify request messages before processing them

ikev2: Allow tasks to verify response messages before processing them

task: Add optional pre_process() method

This will eventually allow tasks to pre-process and verify received
messages.

testing: Add ikev2/redirect-active scenario

ike-init: Ignore notifies related to redirects during rekeying

Also don't query redirect providers in this case.

ike-sa: Add limit for the number of redirects within a defined time period

ike-sa: Reauthenticate to the same addresses we currently use

If the SA got redirected this would otherwise cause a reauthentication with
the original gateway.  Reestablishing the SA to the original gateway, if e.g.
the new gateway is not reachable makes sense though.

vici: Don't redirect all SAs if no selectors are given

This avoid confusion and redirecting all SAs can now easily be done
explicitly (e.g. peer_ip=0.0.0.0/0).

vici: Match subnets and ranges against peer IP in redirect command

vici: Match identity with wildcards against remote ID in redirect command

swanctl: Add --redirect command

vici: Add redirect command

This allows redirecting IKE_SAs by multiple different selectors, if none
are given all SAs are redirected.

redirect-job: Add job to redirect an active IKE_SA

ike-sa: Add redirect() method to actively redirect an IKE_SA

ike-redirect: Add task to redirect active IKE_SAs

ike-auth: Handle REDIRECT notifies during IKE_AUTH

ike-sa: Handle redirect requests for established SAs as reestablishment

We handle this similar to how we do reestablishing IKE_SAs with all CHILD_SAs,
which also includes the one actively queued during IKE_AUTH.

To delete the old SA we use the recently added ike_reauth_complete task.

ike-auth: Send REDIRECT notify during IKE_AUTH if requested by providers

To prevent the creation of the CHILD_SA we set a condition on the
IKE_SA.  We also schedule a delete job in case the client does not
terminate the IKE_SA (which is a SHOULD in RFC 5685).

ike-config: Do not assign attributes for redirected IKE_SAs

child-create: Don't create CHILD_SA if the IKE_SA got redirected in IKE_AUTH

ike-sa: Add a condition to mark redirected IKE_SAs