diff options
| author | Vladislav Grishenko <themiron@mail.ru> | 2018-10-25 05:11:32 +0500 |
|---|---|---|
| committer | Vladislav Grishenko <themiron@mail.ru> | 2018-10-25 05:16:55 +0500 |
| commit | 603800151415c69ceb3c9d986ef0c3487817cb67 (patch) | |
| tree | 657aee87964ca4501a416db40ce3c66d9529128f /accel-pppd/extra/ipv6pool.c | |
| parent | 938bad250baa7bd6f0761e70ce68a2de063c528c (diff) | |
| download | accel-ppp-603800151415c69ceb3c9d986ef0c3487817cb67.tar.gz accel-ppp-603800151415c69ceb3c9d986ef0c3487817cb67.zip | |
fix gateway address setup fail due memory corruption
log:
libnetlink: RTNETLINK answers: Invalid argument
ppp0: f7bb00a79ef667d2: failed to set IPv4 address: Invalid argument
ipaddr_add_peer() called only with mask equeal 0 or 32, but w/o
zero-allocated structs it contans garbage in some cases.
so, instead ipaddr_add() was called with wrong mask value.
also, init chap-secrets mask for the same reason.
Diffstat (limited to 'accel-pppd/extra/ipv6pool.c')
| -rw-r--r-- | accel-pppd/extra/ipv6pool.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/accel-pppd/extra/ipv6pool.c b/accel-pppd/extra/ipv6pool.c index 6fa5cc53..4f29a280 100644 --- a/accel-pppd/extra/ipv6pool.c +++ b/accel-pppd/extra/ipv6pool.c @@ -83,6 +83,7 @@ static void generate_ippool(struct in6_addr *addr, int mask, int prefix_len) while (in6_addr_cmp(&ip, &end) <= 0) { it = malloc(sizeof(*it)); + memset(it, 0, sizeof(*it)); it->it.owner = &ipdb; INIT_LIST_HEAD(&it->it.addr_list); a = malloc(sizeof(*a)); @@ -119,6 +120,7 @@ static void generate_dppool(struct in6_addr *addr, int mask, int prefix_len) while (in6_addr_cmp(&ip, &end) <= 0) { it = malloc(sizeof(*it)); + memset(it, 0, sizeof(*it)); it->it.owner = &ipdb; INIT_LIST_HEAD(&it->it.prefix_list); a = malloc(sizeof(*a)); |