diff options
author | taesub kim <taesub.kim@samsung.com> | 2017-09-28 07:07:34 +0000 |
---|---|---|
committer | Gerrit Code Review <gerrit@review.ap-northeast-2.compute.internal> | 2017-09-28 07:07:34 +0000 |
commit | 625af662ee726069d2872f2a56e78daef6473f0b (patch) | |
tree | bb75e08647509ed136177694049ffc9a15e523ab | |
parent | 5739f84fc89b70cd3349254005b7a1886c5f6dd3 (diff) | |
parent | 47f0926b6ceba3a2e2102c9c25ef421df020e88f (diff) | |
download | connman-625af662ee726069d2872f2a56e78daef6473f0b.tar.gz connman-625af662ee726069d2872f2a56e78daef6473f0b.tar.bz2 connman-625af662ee726069d2872f2a56e78daef6473f0b.zip |
Merge "[CID-26975, 36747] Fix memory leak and dereference after null check" into tizen_4.0
-rw-r--r-- | vpn/plugins/ipsec.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/vpn/plugins/ipsec.c b/vpn/plugins/ipsec.c index 7a2675e0..fa9dc7af 100644 --- a/vpn/plugins/ipsec.c +++ b/vpn/plugins/ipsec.c @@ -714,7 +714,7 @@ static int ipsec_load_shared_psk(struct vpn_provider *provider) if (!provider) { connman_error("invalid provider"); - ret = -EINVAL; + return -EINVAL; } data = vpn_provider_get_string(provider, "IPsec.IKEData"); @@ -840,8 +840,10 @@ static int ipsec_load_key(struct vpn_provider *provider) return 0; sect = vici_create_section(NULL); - if (!sect) + if (!sect) { + g_free(data); return -ENOMEM; + } vici_add_kv(sect, "type", type, NULL); vici_add_kv(sect, "data", data, NULL); |