5 years agoVersion bump to 5.5.3dr1 5.5.3dr1
Andreas Steffen [Wed, 26 Apr 2017 19:29:42 +0000 (21:29 +0200)]
Version bump to 5.5.3dr1

5 years agoconfigure: Include curve25519 in the pki default plugin list
Martin Willi [Wed, 29 Mar 2017 15:04:03 +0000 (17:04 +0200)]
configure: Include curve25519 in the pki default plugin list

The plugin provides ed25519 public key support, and is required to generate
keys or sign certificates with pki.

5 years agotesting: Created swanctl/rw-eap-aka-sql-rsa scenario
Andreas Steffen [Wed, 26 Apr 2017 10:43:40 +0000 (12:43 +0200)]
testing: Created swanctl/rw-eap-aka-sql-rsa scenario

5 years agotesting: Created ikev2/rw-eap-aka-sql-rsa scenario
Andreas Steffen [Sun, 23 Apr 2017 07:19:22 +0000 (09:19 +0200)]
testing: Created ikev2/rw-eap-aka-sql-rsa scenario

This test scenario tests the eap-simaka-sql plugin.

5 years agoeap-simaka-sql: Fixed database column from use to used
Andreas Steffen [Wed, 26 Apr 2017 08:37:45 +0000 (10:37 +0200)]
eap-simaka-sql: Fixed database column from use to used

5 years agopki: Reset variable so error handling works properly
Tobias Brunner [Wed, 19 Apr 2017 16:56:43 +0000 (18:56 +0200)]
pki: Reset variable so error handling works properly

If we jump to `end` without this we crash (not necessarily visibly) due
to a double free and the actual error message is not printed.

5 years agovici: Fix type error exception in Python bindings
odi79 [Fri, 14 Apr 2017 14:40:57 +0000 (16:40 +0200)]
vici: Fix type error exception in Python bindings

Line 66 yields "TypeError: can't concat bytes to str" using Python 3.4.
"requestdata" was introduced in 22f08609f1b6 but is not actually used.
Since the original "request" is not used anywhere else this can be changed
to be similar to the other UTF-8 encoding changes in that commit.

Fixes: 22f08609f1b6 ("vici: Explicitly set the Python encoding type").
Closes strongswan/strongswan#66.

5 years agoVersion bump to 5.5.2 5.5.2
Andreas Steffen [Mon, 27 Mar 2017 14:57:03 +0000 (16:57 +0200)]
Version bump to 5.5.2

5 years agotesting: List BLIS certs in swanctl/rw-newhope-bliss scenario
Andreas Steffen [Mon, 27 Mar 2017 14:56:50 +0000 (16:56 +0200)]
testing: List BLIS certs in swanctl/rw-newhope-bliss scenario

5 years agokernel-netlink: Avoid O(n^2) copy operations when concatenating Netlink responses
Jiri Horky [Thu, 23 Mar 2017 21:59:45 +0000 (22:59 +0100)]
kernel-netlink: Avoid O(n^2) copy operations when concatenating Netlink responses

When constructing the result, all responses from Netlink were concatenated
iteratively, i.e. for each response, the previously acquired result was
copied to newly allocated memory and the current response appended to it.
This results in O(n^2) copy operations. Instead, we now check for the
total final length of the result and copy the individual responses to it
in one pass, i.e. in O(n) copy operations. In particular, this issue caused
very high CPU usage in memcpy() function as the result is copied over and
over. Common way how to hit the issue is when having 1000+ routes and 5+
connecting clients a second. In that case, the memcpy() function can
take 50%+ of one CPU thread on a decent CPU and the whole charon daemon
is stuck just reading routes and concatenating them together (connecting
clients are blocked in that particular case as this is done under mutex).

Closes strongswan/strongswan#65.
References #2055.

5 years agolibtls: Replace expired certificates for unit tests
Tobias Brunner [Fri, 24 Mar 2017 09:18:32 +0000 (10:18 +0100)]
libtls: Replace expired certificates for unit tests

Only the tests with client authentication failed, the client accepted
the trusted self-signed certificate even when it was expired.  On the
server the lookup (based on the pre-configured SAN) first found the ECDSA
cert, which it dismissed for the RSA authentication the client used, and
since only the first "pretrusted" cert is considered the following RSA
cert was verified more thoroughly.
The lookup on the client always uses the full DN of the server certificate
not the pre-configured identity so it found the correct certificate on
the first try.

5 years agopki: Actually make the default key type KEY_ANY for --self
Tobias Brunner [Fri, 24 Mar 2017 09:10:13 +0000 (10:10 +0100)]
pki: Actually make the default key type KEY_ANY for --self

Fixes: 05ccde0a8bd9 ("pki: Add generic 'priv' key type that loads any
type of private key")

5 years agoaddrblock: Narrow selectors when rekeying a CHILD_SA as original responder
Martin Willi [Thu, 23 Mar 2017 07:48:46 +0000 (08:48 +0100)]
addrblock: Narrow selectors when rekeying a CHILD_SA as original responder

If a the original responder narrows the selectors of its peer in addrblock,
the peer gets a subset of that selectors. However, once the original responder
initiates rekeying of that CHILD_SA, it sends the full selectors to the peer,
and then narrows the received selectors locally for the installation, only.

This is insufficient, as the peer ends up with wider selectors, sending traffic
that the original responder will reject to the stricter IPsec policy. So
additionally narrow the selectors when rekeying CHILD_SAs before sending the
TS list to the peer.

5 years agoconf: Document recommended lower limit for SPIs
Tobias Brunner [Thu, 23 Mar 2017 16:29:47 +0000 (17:29 +0100)]
conf: Document recommended lower limit for SPIs

5 years agotravis: aikpub2 was removed, no need to disable it anymore
Tobias Brunner [Thu, 23 Mar 2017 09:48:04 +0000 (10:48 +0100)]
travis: aikpub2 was removed, no need to disable it anymore

5 years agoconf: Remove snippet for aikpub2
Tobias Brunner [Thu, 23 Mar 2017 09:47:25 +0000 (10:47 +0100)]
conf: Remove snippet for aikpub2

5 years agotravis: Build Windows-specific plugins
Tobias Brunner [Wed, 22 Mar 2017 17:04:05 +0000 (18:04 +0100)]
travis: Build Windows-specific plugins

The plugins can only be built on x64 as the MinGW headers on Ubuntu 12.04,
which we have to use for x86 due to another issue with MinGW, are too old.

5 years agokernel-wfp: Don't redefine IPPROTO_IP* if already defined
Tobias Brunner [Wed, 22 Mar 2017 16:59:20 +0000 (17:59 +0100)]
kernel-wfp: Don't redefine IPPROTO_IP* if already defined

5 years agopki: Cast length derived from pointer arithmetic to int
Tobias Brunner [Wed, 22 Mar 2017 16:30:10 +0000 (17:30 +0100)]
pki: Cast length derived from pointer arithmetic to int

5 years agovici: Don't fall back to uninstalling traps if a matching shunt was found
Tobias Brunner [Wed, 22 Mar 2017 16:19:31 +0000 (17:19 +0100)]
vici: Don't fall back to uninstalling traps if a matching shunt was found

This is different if `ike` and `child` are provided and uninstall()
fails as we call that without knowing whether a matching shunt exists.
But if `ike` is not provided we explicitly search for a matching shunt
and if found don't need to look for a trap policy.

5 years agoconfigure: Fix test for libunwind
Tobias Brunner [Wed, 22 Mar 2017 15:33:27 +0000 (16:33 +0100)]
configure: Fix test for libunwind

Most functions in libunwind.h are actually mapped via macros to obscure
function names, so checking for these would require some elaborate test
via AC_LINK_IFELSE().  However, unw_backtrace() seems to be one of the few
actual functions so lets use this for now, even though we don't call it
ourselves later.

Fixes: 016228c15843 ("configure: Check for actual functions in libraries

5 years agoFixed some typos, courtesy of codespell
Tobias Brunner [Wed, 22 Mar 2017 12:13:56 +0000 (13:13 +0100)]
Fixed some typos, courtesy of codespell

5 years agoswanctl: Reformulate IKEv1 selector restriction, describe problems with TS narrowing
Noel Kuntze [Mon, 13 Mar 2017 15:26:10 +0000 (16:26 +0100)]
swanctl: Reformulate IKEv1 selector restriction, describe problems with TS narrowing

5 years agoswanctl: Mention including files when referring to strongswan.conf(5)
Tobias Brunner [Tue, 21 Mar 2017 08:52:44 +0000 (09:52 +0100)]
swanctl: Mention including files when referring to strongswan.conf(5)

5 years agoman: Describe the tunneling of several subnets with IKEv1 in more detail
Noel Kuntze [Mon, 13 Mar 2017 15:20:39 +0000 (16:20 +0100)]
man: Describe the tunneling of several subnets with IKEv1 in more detail

5 years agoman: Add note about modeconfig having to match
Noel Kuntze [Mon, 13 Mar 2017 16:18:48 +0000 (17:18 +0100)]
man: Add note about modeconfig having to match

5 years agoVersion bump to 5.2.2rc1 5.5.2rc1
Andreas Steffen [Tue, 21 Mar 2017 08:09:43 +0000 (09:09 +0100)]
Version bump to 5.2.2rc1

5 years agotesting: Updated OCSP certificate for carol
Andreas Steffen [Tue, 21 Mar 2017 08:09:06 +0000 (09:09 +0100)]
testing: Updated OCSP certificate for carol

5 years agoAllow x25519 as an alias of the curve25519 KE algorithm
Andreas Steffen [Mon, 20 Mar 2017 06:24:29 +0000 (07:24 +0100)]
Allow x25519 as an alias of the curve25519 KE algorithm

5 years agoReference Edwards-curve signature RFCs
Andreas Steffen [Fri, 10 Feb 2017 08:05:54 +0000 (09:05 +0100)]
Reference Edwards-curve signature RFCs

5 years agoThe tpm plugin offers random number generation
Andreas Steffen [Sat, 18 Mar 2017 13:51:30 +0000 (14:51 +0100)]
The tpm plugin offers random number generation

The tpm plugin can be used to derive true random numbers from a
TPM 2.0 device. The get_random method must be explicitly enabled
in strongswan.conf with the plugin.tpm.use_rng = yes option.

5 years agovici: Document how we pronounce the vici protocol and plugin
Martin Willi [Fri, 17 Feb 2017 09:47:44 +0000 (10:47 +0100)]
vici: Document how we pronounce the vici protocol and plugin

5 years agoswanctl: Describe what happens when a FQDN is specified in local|remote_addrs
Tobias Brunner [Fri, 10 Mar 2017 15:03:14 +0000 (16:03 +0100)]
swanctl: Describe what happens when a FQDN is specified in local|remote_addrs

5 years agoman: Describe what happens when a FQDN is specified in left or right
Noel Kuntze [Thu, 9 Mar 2017 16:47:49 +0000 (17:47 +0100)]
man: Describe what happens when a FQDN is specified in left or right

5 years agoikev1: First do PSK lookups based on identities then fallback to IPs
Tobias Brunner [Mon, 13 Mar 2017 15:37:12 +0000 (16:37 +0100)]
ikev1: First do PSK lookups based on identities then fallback to IPs

This provides a solution for configs where there is e.g. a catch-all %any
PSK, while more specific PSKs would be found by the identities of configs
that e.g. use FQDNs as local/remote addresses.

Fixes #2223.

5 years agotesting: Fix URL for kernel sources
Tobias Brunner [Mon, 20 Mar 2017 09:13:33 +0000 (10:13 +0100)]
testing: Fix URL for kernel sources

5 years agoike-sa-manager: Remove superfluous assignment
Thomas Egerer [Wed, 15 Mar 2017 15:24:49 +0000 (16:24 +0100)]
ike-sa-manager: Remove superfluous assignment

Memory is allocated with calloc, hence set to zero, thus assigning the
numerical value 0 is not required.

Signed-off-by: Thomas Egerer <>
5 years agoike: Log remote IP when deleting half-open IKE_SAs
Tobias Brunner [Wed, 15 Mar 2017 14:56:39 +0000 (15:56 +0100)]
ike: Log remote IP when deleting half-open IKE_SAs

5 years agocoverage: Exclude test suites and /usr from coverage report
Tobias Brunner [Wed, 15 Mar 2017 14:54:38 +0000 (15:54 +0100)]
coverage: Exclude test suites and /usr from coverage report

5 years agotravis: Create coverage report via
Tobias Brunner [Wed, 15 Mar 2017 09:19:02 +0000 (10:19 +0100)]
travis: Create coverage report via

5 years agoVersion bump to 5.5.2dr7 5.5.2dr7
Andreas Steffen [Mon, 6 Mar 2017 19:21:40 +0000 (20:21 +0100)]
Version bump to 5.5.2dr7

5 years agoaikpub2: Removed aikpub2 tool
Andreas Steffen [Mon, 6 Mar 2017 05:16:10 +0000 (06:16 +0100)]
aikpub2: Removed aikpub2 tool

The aikpub2 tool has been replaced by pki --pub|--req --keyid hex ..
where keyid indicates the TPM 2.0 private key object handle. Thus
either the public key in PKCS#1 format can be extracted or a PKCS#10
certificate request signed by the TPM private key can be generated.

5 years agopki: Add key object handle of smartcard or TPM private key as an argument to pki...
Andreas Steffen [Mon, 6 Mar 2017 11:53:48 +0000 (12:53 +0100)]
pki: Add key object handle of smartcard or TPM private key as an argument to pki --keyid

5 years agoutils: chunk_from_hex() skips optional 0x prefix
Andreas Steffen [Mon, 6 Mar 2017 10:51:10 +0000 (11:51 +0100)]
utils: chunk_from_hex() skips optional 0x prefix

5 years agopki: Edited keyid parameter use in various pki man pages and usage outputs
Andreas Steffen [Sun, 5 Mar 2017 10:19:39 +0000 (11:19 +0100)]
pki: Edited keyid parameter use in various pki man pages and usage outputs

5 years agoquick-mode: Correctly prepare NAT-OA payloads as responder
Tobias Brunner [Fri, 3 Mar 2017 16:11:04 +0000 (17:11 +0100)]
quick-mode: Correctly prepare NAT-OA payloads as responder

The initiator's address was sent back twice previously.

Fixes #2268.

5 years agoVersion bump to 5.5.2dr6 5.5.2dr6
Andreas Steffen [Fri, 3 Mar 2017 08:34:50 +0000 (09:34 +0100)]
Version bump to 5.5.2dr6

5 years agoAdd keyid of smartcard or TPM private key as an argument to pki --req
Andreas Steffen [Thu, 2 Mar 2017 19:30:24 +0000 (20:30 +0100)]
Add keyid of smartcard or TPM private key as an argument to pki --req

5 years agotesting: load-testconfig script loads config from source dir
Tobias Brunner [Thu, 2 Mar 2017 08:50:34 +0000 (09:50 +0100)]
testing: load-testconfig script loads config from source dir

It now does replace the IPs too. This way it's easier to play around
with a config (otherwise a do-tests run was required to build the
config files in the build dir).

5 years agolibipsec: Enforce a minimum of 256 for SPIs
Tobias Brunner [Thu, 2 Mar 2017 10:51:27 +0000 (11:51 +0100)]
libipsec: Enforce a minimum of 256 for SPIs

RFC 4303 reserves the SPIs between 1 and 255 for future use.  This also
avoids an overflow and a division by zero if spi_min is 0 and spi_max is

5 years agolibipsec: Fix min/max SPI
Tobias Brunner [Thu, 2 Mar 2017 09:11:32 +0000 (10:11 +0100)]
libipsec: Fix min/max SPI

5 years agocontroller: Don't listen for CHILD_SA state changes when terminating IKE_SAs
Tobias Brunner [Thu, 2 Mar 2017 07:58:07 +0000 (08:58 +0100)]
controller: Don't listen for CHILD_SA state changes when terminating IKE_SAs

We actually want to wait until the IKE_SA is destroyed, not any of the
CHILD_SAs (even though there might not be that much of a difference
depending on the number of CHILD_SAs).

Fixes #2261.

5 years agokernel: Make range of SPIs for IPsec SAs configurable
Tobias Brunner [Tue, 21 Feb 2017 18:21:01 +0000 (19:21 +0100)]
kernel: Make range of SPIs for IPsec SAs configurable

5 years agosettings: Add support for hex integers (0x prefix) via get_int()
Tobias Brunner [Tue, 21 Feb 2017 18:12:32 +0000 (19:12 +0100)]
settings: Add support for hex integers (0x prefix) via get_int()

5 years agolibipsec: Log a packet's ports and protocol in case of a policy mismatch
Tobias Brunner [Fri, 17 Feb 2017 11:40:36 +0000 (12:40 +0100)]
libipsec: Log a packet's ports and protocol in case of a policy mismatch

5 years agohost: Don't log port if it is zero
Tobias Brunner [Fri, 17 Feb 2017 11:32:04 +0000 (12:32 +0100)]
host: Don't log port if it is zero

5 years agolibipsec: Match IPsec policies against ports of processed packets
Tobias Brunner [Wed, 15 Feb 2017 14:33:38 +0000 (15:33 +0100)]
libipsec: Match IPsec policies against ports of processed packets

Fixes #2252.

5 years agoNEWS: Mention the new addrblock features
Martin Willi [Thu, 2 Mar 2017 07:23:01 +0000 (08:23 +0100)]
NEWS: Mention the new addrblock features

5 years agoaddrblock: Use dynamic TS narrowing instead of rejecting the whole CHILD_SA
Martin Willi [Wed, 22 Feb 2017 09:01:19 +0000 (10:01 +0100)]
addrblock: Use dynamic TS narrowing instead of rejecting the whole CHILD_SA

Previously, the client had to propose no wider selectors than the certificate
permits, otherwise the complete CHILD_SA was rejected. However, with IKEv2
we can dynamically narrow the selectors to what the certificate allows. This
makes client and gateway configurations very simple by just proposing,
narrowed to selectors the client is permitted to route into the network.

5 years agoaddrblock: Support an optional non-strict mode accepting certs without addrblock
Martin Willi [Wed, 22 Feb 2017 08:43:31 +0000 (09:43 +0100)]
addrblock: Support an optional non-strict mode accepting certs without addrblock

This allows a gateway to enforce the addrblock policy on certificates that
actually have the extension only. For (legacy) certificates not having the
extension, traffic selectors are validated/narrowed by other means, most
likely by the configuration.

5 years agochild-cfg: Always apply hosts to traffic selectors if proposing transport mode
Tobias Brunner [Mon, 20 Feb 2017 10:36:30 +0000 (11:36 +0100)]
child-cfg: Always apply hosts to traffic selectors if proposing transport mode

Usually, %dynamic is used as traffic selector for transport mode SAs,
however, if wildcard traps are used then the remote TS will be a subnet.
With strongSwan at the remote end that usually works fine as the local
%dynamic TS narrows the proposed TS appropriately.  But some
implementations reject non-host TS for transport mode SAs.
Another problem could be if several distinct subnets are configured for a
wildcard trap, as we'd then propose unrelated subnets on that transport
mode SA, which might be problematic even for strongSwan (switch to tunnel
mode and duplicate policies).

Closes strongswan/strongswan#61.

5 years agotraffic-selector: Allow calling set_address() for any traffic selector
Tobias Brunner [Mon, 20 Feb 2017 10:28:17 +0000 (11:28 +0100)]
traffic-selector: Allow calling set_address() for any traffic selector

Users may check is_host(), is_dynamic() or includes() before calling this
if restrictions are required (most actually already do).

5 years agoMerge branch 'pki-addrblock'
Martin Willi [Mon, 27 Feb 2017 08:37:51 +0000 (09:37 +0100)]
Merge branch 'pki-addrblock'

Add support to the x509 plugin and pki to generate certificates with the
RFC 3779 addrblock extension.

5 years agopki: Add a note about constructing RFC 3779 compliant certificates to manpage
Martin Willi [Wed, 22 Feb 2017 07:52:00 +0000 (08:52 +0100)]
pki: Add a note about constructing RFC 3779 compliant certificates to manpage

5 years agopki: Support an --addrblock option for issued certificates
Martin Willi [Tue, 21 Feb 2017 14:07:42 +0000 (15:07 +0100)]
pki: Support an --addrblock option for issued certificates

5 years agopki: Support an --addrblock option for self-signed certificates
Martin Willi [Tue, 21 Feb 2017 14:07:03 +0000 (15:07 +0100)]
pki: Support an --addrblock option for self-signed certificates

5 years agopki: Add a helper function parse traffic selectors from CIDR subnets or ranges
Martin Willi [Tue, 21 Feb 2017 14:06:15 +0000 (15:06 +0100)]
pki: Add a helper function parse traffic selectors from CIDR subnets or ranges

5 years agox509: Do not mark generated addrblock extension as critical
Martin Willi [Tue, 21 Feb 2017 15:54:57 +0000 (16:54 +0100)]
x509: Do not mark generated addrblock extension as critical

While RFC 3779 says we SHOULD mark it is critical, this has severe side effects
in practice. The addrblock extension is not widely used nor implemented, and
only a few applications can handle this extension. By marking it critical,
none of these applications can make use of such certificates where included
addrblocks do not matter, such as TLS/HTTPS.

If an application wants to make use of addrblocks, that is usually an explicit
decision. Then the very same application obviously can handle addrblocks, and
there is no need for the extension to be critical. In other words, for local
policy checks it is a local matter to handle the extension, hence making it
critical is usually not of much help.

5 years agox509: Support encoding the RFC 3779 addrblock extension
Martin Willi [Tue, 21 Feb 2017 14:05:17 +0000 (15:05 +0100)]
x509: Support encoding the RFC 3779 addrblock extension

5 years agobuilder: Define a builder part for X.509 RFC 3779 address blocks
Martin Willi [Tue, 21 Feb 2017 14:04:18 +0000 (15:04 +0100)]
builder: Define a builder part for X.509 RFC 3779 address blocks

5 years agoplugin-loader: Fix hashing of registered plugin features
Tobias Brunner [Fri, 24 Feb 2017 17:22:23 +0000 (18:22 +0100)]
plugin-loader: Fix hashing of registered plugin features

This strangely never caused any noticeable issues, but was the reason for
build failures in certain test cases (mostly BLISS) due to missing plugin
features when built with specific options on Travis (was not reproducible

5 years agoVersion bump to 5.5.2dr5 5.5.2dr5
Andreas Steffen [Thu, 23 Feb 2017 16:31:11 +0000 (17:31 +0100)]
Version bump to 5.5.2dr5

5 years agoUse of TPM 2.0 private keys for signatures via tpm plugin
Andreas Steffen [Thu, 16 Feb 2017 07:39:47 +0000 (08:39 +0100)]
Use of TPM 2.0 private keys for signatures via tpm plugin

5 years agoImplement signatures with private keys bound to TPM 2.0
Andreas Steffen [Mon, 13 Feb 2017 18:06:18 +0000 (19:06 +0100)]
Implement signatures with private keys bound to TPM 2.0

5 years agoandroid: New release after fixing potential ANR issue
Tobias Brunner [Mon, 20 Feb 2017 15:34:43 +0000 (16:34 +0100)]
android: New release after fixing potential ANR issue

5 years agoandroid: Send network change events from a separate thread via JNI
Tobias Brunner [Wed, 15 Feb 2017 15:08:35 +0000 (16:08 +0100)]
android: Send network change events from a separate thread via JNI

Doing this from the main UI thread (which delivers the broadcast) might
cause an ANR if there is a delay (e.g. while acquiring a mutex in the
native parts). There might also have been a race condition during
termination previously because Unregister() was not synchronized so there
might have been dangling events that got delivered while or after the mutex
in the native parts was destroyed.

5 years agoikev1: Respond to DPDs for rekeyed IKE_SAs
Tobias Brunner [Mon, 13 Feb 2017 10:54:53 +0000 (11:54 +0100)]
ikev1: Respond to DPDs for rekeyed IKE_SAs

Some devices always use the oldest IKE_SA to send DPDs and will delete
all IKE_SAs when there is no response. If uniqueness is not enforced
rekeyed IKE_SAs might not get deleted until they expire so we should
respond to DPDs.

References #2090.

5 years agoike-sa: Optionally try to migrate to the best path on routing priority changes
Martin Willi [Mon, 10 Oct 2016 13:59:52 +0000 (15:59 +0200)]
ike-sa: Optionally try to migrate to the best path on routing priority changes

When multihomed, a setup might prefer to dynamically stay on the cheapest
available path by using MOBIKE migrations. If the cheapest path goes away and
comes back, we currently stay on the more expensive path to reduce noise and
prevent potential migration issues. This is usually just fine for links not
generating real cost.

If we have more expensive links in the setup, it can be desirable to always
migrate to the cheapest link available. By setting charon.prefer_best_path,
charon tries to migrate to the path using the highest priority link, allowing
an external application to update routes to indirectly control MOBIKE behavior.
This option has no effect if MOBIKE is unavailable.

5 years agoikev2: Ignore roam events without MOBIKE but static local address
Tobias Brunner [Tue, 15 Nov 2016 14:42:33 +0000 (15:42 +0100)]
ikev2: Ignore roam events without MOBIKE but static local address

Disabling MOBIKE and statically configuring a local address should be
enough indication that the user doesn't want to roam to a different
address.  There might not be any routes that indicate we can use the
current address but it might still work (e.g. if the address is on an
interface that is not referenced in any routes and the address itself
is neither).  This way we avoid switching to another address for routes
that might be available on the system.

We currently don't make much use of COND_STALE anyway when MOBIKE is not
enabled, e.g. to avoid sending DPDs if the connection is seemingly down.
With MOBIKE enabled we don't exactly check that state but we do don't
send DPDs if there is no route/source address available.

5 years agoike-cfg: Add helper function to determine if a given IP address was configured
Tobias Brunner [Wed, 1 Feb 2017 17:00:57 +0000 (18:00 +0100)]
ike-cfg: Add helper function to determine if a given IP address was configured

5 years agoMerge branch 'vici-updates'
Tobias Brunner [Thu, 16 Feb 2017 18:24:17 +0000 (19:24 +0100)]
Merge branch 'vici-updates'

Adds several new features for the VICI interface and swanctl.

5 years agoNEWS: VICI updates
Tobias Brunner [Wed, 15 Feb 2017 16:49:06 +0000 (17:49 +0100)]
NEWS: VICI updates

5 years agovici: Only log messages if there actually is a listener
Tobias Brunner [Mon, 13 Feb 2017 16:38:49 +0000 (17:38 +0100)]
vici: Only log messages if there actually is a listener

5 years agovici: Let has_event_listeners() actually check if clients are registered
Tobias Brunner [Wed, 8 Feb 2017 14:20:58 +0000 (15:20 +0100)]
vici: Let has_event_listeners() actually check if clients are registered

Fixes: 8d96f90a7983 ("vici: Add function to test if an event should be

5 years agovici: Add support for mediation extension
Tobias Brunner [Tue, 7 Feb 2017 11:04:30 +0000 (12:04 +0100)]
vici: Add support for mediation extension

5 years agopeer-cfg: Store mediated_by as name and not peer-cfg reference
Tobias Brunner [Tue, 7 Feb 2017 10:30:49 +0000 (11:30 +0100)]
peer-cfg: Store mediated_by as name and not peer-cfg reference

This way updates to the mediation config are respected and the order in
which configs are configured/loaded does not matter.

The SQL plugin currently maintains the strong relationship between
mediated and mediation connection (we could theoretically change that to a
string too).

5 years agovici: Include uniqueness policy in list-conns
Tobias Brunner [Wed, 1 Feb 2017 10:02:22 +0000 (11:02 +0100)]
vici: Include uniqueness policy in list-conns

5 years agoswanctl: Add --rekey command
Tobias Brunner [Tue, 24 Jan 2017 15:34:32 +0000 (16:34 +0100)]
swanctl: Add --rekey command

5 years agovici: Add command to initiate SA rekeying
Tobias Brunner [Tue, 24 Jan 2017 15:26:48 +0000 (16:26 +0100)]
vici: Add command to initiate SA rekeying

5 years agovici: Use unique names for CHILD_SAs in the list-sas command
Tobias Brunner [Fri, 9 Dec 2016 13:45:41 +0000 (14:45 +0100)]
vici: Use unique names for CHILD_SAs in the list-sas command

The original name is returned in the new "name" attribute.

This fixes an issue with bindings that map VICI messages to
dictionaries.  For instance, in roadwarrior scenarios where every
CHILD_SA has the same name only the information of the last CHILD_SA
would end up in the dictionary for that name.

5 years agoswanctl: Allow specifying pubkeys directly via 0x/0s prefix
Tobias Brunner [Wed, 18 Jan 2017 16:46:27 +0000 (17:46 +0100)]
swanctl: Allow specifying pubkeys directly via 0x/0s prefix

5 years agovici: Add support to load CA certificates from tokens and paths in authority sections
Tobias Brunner [Wed, 30 Nov 2016 14:41:18 +0000 (15:41 +0100)]
vici: Add support to load CA certificates from tokens and paths in authority sections

5 years agovici: Add support to load certificates from file paths
Tobias Brunner [Wed, 30 Nov 2016 14:09:04 +0000 (15:09 +0100)]
vici: Add support to load certificates from file paths

Probably not that useful via swanctl.conf but could be when used via VICI.

5 years agovici: Add support to load certificates from tokens
Tobias Brunner [Wed, 30 Nov 2016 11:44:51 +0000 (12:44 +0100)]
vici: Add support to load certificates from tokens

5 years agoswanctl: Add `token` secrets for keys on tokens/smartcards
Tobias Brunner [Fri, 18 Nov 2016 15:40:34 +0000 (16:40 +0100)]
swanctl: Add `token` secrets for keys on tokens/smartcards

5 years agovici: Add command to load a private key from a token
Tobias Brunner [Fri, 18 Nov 2016 14:01:18 +0000 (15:01 +0100)]
vici: Add command to load a private key from a token

PINs are stored in a "hidden" credential set, so that its shared
secrets are not exposed via VICI.  Since they are not explicitly loaded as
shared secrets via VICI a client might consider them as removed secrets and
remove them.

5 years agovici: List namespace/peer-cfg name with policies and allow filtering
Tobias Brunner [Mon, 13 Feb 2017 17:18:58 +0000 (18:18 +0100)]
vici: List namespace/peer-cfg name with policies and allow filtering

The two names are also transmitted in separate keys.

5 years agoswanctl: Pass optional connection name to --initiate/install/uninstall
Tobias Brunner [Tue, 11 Oct 2016 16:27:29 +0000 (18:27 +0200)]
swanctl: Pass optional connection name to --initiate/install/uninstall

5 years agovici: Explicitly use peer name when uninstalling trap and shunt policies
Tobias Brunner [Wed, 16 Nov 2016 17:13:59 +0000 (18:13 +0100)]
vici: Explicitly use peer name when uninstalling trap and shunt policies

Also adds an `ike` parameter to the `uninstall` command.

5 years agostroke: Use peer name as namespace for shunt policies
Tobias Brunner [Wed, 8 Feb 2017 15:13:32 +0000 (16:13 +0100)]
stroke: Use peer name as namespace for shunt policies

The same goes for the start-action-job.  When unrouting, we search for
the first policy with a matching child-cfg.