diff options
| author | Tobias Brunner <tobias@strongswan.org> | 2008-11-11 09:56:47 +0000 |
|---|---|---|
| committer | Tobias Brunner <tobias@strongswan.org> | 2008-11-11 09:56:47 +0000 |
| commit | e7c27b4f3f2538045278dc1c20dbfa2da9c271e3 (patch) | |
| tree | 8e8ef9c54abfb1f359275e9adb4bcc6339f88999 /src/charon/plugins/kernel_netlink/kernel_netlink_ipsec.c | |
| parent | ea625fabf9ecde1acaa953f2ba0c1fae1af22c35 (diff) | |
| download | strongswan-e7c27b4f3f2538045278dc1c20dbfa2da9c271e3.tar.bz2 strongswan-e7c27b4f3f2538045278dc1c20dbfa2da9c271e3.tar.xz | |
fixing a memory leak
Diffstat (limited to 'src/charon/plugins/kernel_netlink/kernel_netlink_ipsec.c')
| -rw-r--r-- | src/charon/plugins/kernel_netlink/kernel_netlink_ipsec.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/src/charon/plugins/kernel_netlink/kernel_netlink_ipsec.c b/src/charon/plugins/kernel_netlink/kernel_netlink_ipsec.c index d0c9b721b..6832222a4 100644 --- a/src/charon/plugins/kernel_netlink/kernel_netlink_ipsec.c +++ b/src/charon/plugins/kernel_netlink/kernel_netlink_ipsec.c @@ -1756,6 +1756,7 @@ static bool add_bypass_policies() { int fd, family, port; enumerator_t *sockets; + bool status = TRUE; /* we open an AF_KEY socket to autoload the af_key module. Otherwise * setsockopt(IPSEC_POLICY) won't work. */ @@ -1797,17 +1798,20 @@ static bool add_bypass_policies() { DBG1(DBG_KNL, "unable to set IPSEC_POLICY on socket: %s", strerror(errno)); - return FALSE; + status = FALSE; + break; } policy.sadb_x_policy_dir = IPSEC_DIR_INBOUND; if (setsockopt(fd, sol, ipsec_policy, &policy, sizeof(policy)) < 0) { DBG1(DBG_KNL, "unable to set IPSEC_POLICY on socket: %s", strerror(errno)); - return FALSE; + status = FALSE; + break; } } - return TRUE; + sockets->destroy(sockets); + return status; } /* |