diff options
| author | Thomas Egerer <thomas.egerer@secunet.com> | 2015-10-06 11:02:45 +0200 |
|---|---|---|
| committer | Tobias Brunner <tobias@strongswan.org> | 2015-11-10 15:40:14 +0100 |
| commit | db61c37690b5bd1c9d157ad7a91d3a6a0b8684bc (patch) | |
| tree | 39ea8590eff8751c55e050f1920211f27c851cd7 /src/libhydra/kernel/kernel_net.c | |
| parent | e8f2c13f9a80c53f4fae15119c5d2cdafb08726b (diff) | |
| download | strongswan-db61c37690b5bd1c9d157ad7a91d3a6a0b8684bc.tar.bz2 strongswan-db61c37690b5bd1c9d157ad7a91d3a6a0b8684bc.tar.xz | |
kernel-interface: Return bool for kernel interface registration
If the (un)registering of a kernel interface (net or ipsec) fails, the
plugin loader will never know, since the appropriate functions always
returns TRUE. By making the (un)register functions return a boolean
value, the loader can detect a failure during initializing the kernel
interface and abort charon startup if desired.
Diffstat (limited to 'src/libhydra/kernel/kernel_net.c')
| -rw-r--r-- | src/libhydra/kernel/kernel_net.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/src/libhydra/kernel/kernel_net.c b/src/libhydra/kernel/kernel_net.c index 0841ed803..07d8b2999 100644 --- a/src/libhydra/kernel/kernel_net.c +++ b/src/libhydra/kernel/kernel_net.c @@ -25,13 +25,14 @@ bool kernel_net_register(plugin_t *plugin, plugin_feature_t *feature, { if (reg) { - hydra->kernel_interface->add_net_interface(hydra->kernel_interface, + return hydra->kernel_interface->add_net_interface( + hydra->kernel_interface, (kernel_net_constructor_t)data); } else { - hydra->kernel_interface->remove_net_interface(hydra->kernel_interface, + return hydra->kernel_interface->remove_net_interface( + hydra->kernel_interface, (kernel_net_constructor_t)data); } - return TRUE; } |