diff options
author | Jason A. Donenfeld <Jason@zx2c4.com> | 2019-08-05 16:44:06 +0200 |
---|---|---|
committer | Jason A. Donenfeld <Jason@zx2c4.com> | 2019-08-05 16:44:06 +0200 |
commit | 4eeccd2b59cc09a384f3534df61c9ad8cf0118b7 (patch) | |
tree | 76f164fd11705eb83f349d9ea0d1336aa7b1eaff /src/peer.c | |
parent | compat: do not run bc on clean target (diff) | |
download | wireguard-monolithic-historical-4eeccd2b59cc09a384f3534df61c9ad8cf0118b7.tar.xz wireguard-monolithic-historical-4eeccd2b59cc09a384f3534df61c9ad8cf0118b7.zip |
netlink: skip peers with invalid keys
Diffstat (limited to '')
-rw-r--r-- | src/peer.c | 11 |
1 files changed, 7 insertions, 4 deletions
@@ -22,20 +22,23 @@ struct wg_peer *wg_peer_create(struct wg_device *wg, const u8 preshared_key[NOISE_SYMMETRIC_KEY_LEN]) { struct wg_peer *peer; + int ret = -ENOMEM; lockdep_assert_held(&wg->device_update_lock); if (wg->num_peers >= MAX_PEERS_PER_DEVICE) - return NULL; + return ERR_PTR(ret); peer = kzalloc(sizeof(*peer), GFP_KERNEL); if (unlikely(!peer)) - return NULL; + return ERR_PTR(ret); peer->device = wg; if (!wg_noise_handshake_init(&peer->handshake, &wg->static_identity, - public_key, preshared_key, peer)) + public_key, preshared_key, peer)) { + ret = -EKEYREJECTED; goto err_1; + } if (dst_cache_init(&peer->endpoint_cache, GFP_KERNEL)) goto err_1; if (wg_packet_queue_init(&peer->tx_queue, wg_packet_tx_worker, false, @@ -74,7 +77,7 @@ err_2: dst_cache_destroy(&peer->endpoint_cache); err_1: kfree(peer); - return NULL; + return ERR_PTR(ret); } struct wg_peer *wg_peer_get_maybe_zero(struct wg_peer *peer) |