Merge pull request #43 from bwhacks/security-fixes

Security/regression fixes

Fix end pointer passed from dhcpv6_handle_reply() to dhcpv6_parse_ia()

The end of the IA option is odata + olen; there's no need to add anything.

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>

Fix regression in entry_to_env()

I broke entry_to_env() by incrementing the wrong variable in commit
a6bbd1d7f5c2 ("Fix potential buffer overflow in entry_to_env").

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>

Merge pull request #42 from bwhacks/logic-fixes

Logic fixes

Merge pull request #41 from bwhacks/security-fixes

Security fixes

Fix iteration after expiring a list entry

After removing an entry, the next entry will be at the same offset as
the entry we just removed.  Also the total length will have changed.

Update the length when we remove an entry, and advance the offset only
when we don't.

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>

Fix script environment variable setting for empty lists

When setting an environment variable as a space-separated list, and
the list is empty, we must not delete the '=' before the value.

In practice putenv() is likely to discard the invalid string, leaving
the variable unset, but this is not guaranteed.

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>

Add missing option length checks in dhcpv6_handle_advert

These might be redundant with checks elsewhere but it's better to be
safe.

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>

Fix potential log forgery via status string

We should not include any control characters from the server status
message when logging it; in particular if we include '\n' this could
result in additional arbitrary log lines.  In dhcpv6_log_status_code,
replace all control characters with '?'.

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>

Check for unsupported PD exclusion configuration in dhcpv6_parse_ia

We currently only support PD exclusions that only affect bits 64-95 of
the address, so we require:

    32 <= PD prefix length < exclusion prefix length <= 64

The first inequality was not validated, and this could result in a
buffer overflow when generating the next request message.

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>

Fix memory leak in dhcpv6_add_server_cand in case odhcp6c_insert_state fails

If we fail to store information from the new server, the associated
NA and PD options will never be freed.  An attacker could use this
for denial-of-service.

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>

Change odhcp6c_insert_state to return a success/failure indicator

Some callers will need to free resources on failure.

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>

Fix possible stack buffer overflow in s46_to_env when copying IPv6 prefixes

An 8-bit prefix-length field can be as large as 255, but values larger
than 128 will result in a buffer overflow when copying to in6.

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>

Fix off-by-one in buffer length in int_to_env

We need to allow for '=', negative sign, 10 digits and the null
terminator, adding up to 13 bytes not 12.

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>

Fix potential buffer overflow in entry_to_env

It appears that an entry of type ENTRY_PREFIX with iaid != 1 and an
exclusion can expand to a string of length up to 154 bytes, whereas we
allocate only 144 bytes per entry.

Also, in case of truncation, snprintf() returns the length of the
un-truncated output so we must not use this to increment buf_len.

Finally some of the lengths given to snprintf() are unnecessarily
generous.  Reduce them so we don't have to increase the allocated
length per entry further.

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>

Avoid copying buffer after dn_expand() fails

If dn_expand() returns an error we could copy from an uninitialised
output buffer or append the previous domain name again.

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>

Merge pull request #40 from dedeckeh/bugfixes

script: Launch script with correct action if last script call is terminated

script: Launch script with correct action if last script call is terminated

Actions launched as resume will be used in a next script_call
if the new action is not marked as resume even when the previous
script run was already terminated.
This behavior is particular visible when a RA is received as
the script will run with action bound and not ra-updated
resulting into a wan6 interface down/up transition

odhcp6c: sync and accumulate RA & DHCPv6 events correctly

Merge pull request #37 from themiron/master

Avoid solicit for zero-length prefix

Merge branch 'upstream'

dhcpv6: remove dead code

Avoid solicit for zero-length prefix

dhcpv6: clear CUSTOM_OPTS in a more sane manner

Signed-off-by: Steven Barth <steven@midlink.org>

Fix LW4over6 parameter handling

Merge pull request #33 from themiron/master

Avoid of waiting for Advertise in stateless-only mode

Avoid of waiting for Advertise in stateless-only mode

Start with Information-request when configured not to ask
IA_NA/IA_PD. It allows to complete the exchange using only
two messages, instead of four, and fixes infinite Advertise
waiting loop with servers that just ignore Solicit messages.

Fixup INF_MAX_RT for RFC 3315 as well

Signed-off-by: Steven Barth <steven@midlink.org>

Fix SOL_MAX_RT default value to match RFC 3315

Signed-off-by: Steven Barth <steven@midlink.org>

example: actually chmod new resolv.conf after overwriting

Signed-off-by: Steven Barth <steven@midlink.org>

ra: avoid saving an empty search domain entry

Signed-off-by: Steven Barth <steven@midlink.org>

scan-code fixes

Improve handling of DNS search domains

Signed-off-by: Steven Barth <steven@midlink.org>

Fix odhcp6c_find_entry

Signed-off-by: Steven Barth <steven@midlink.org>

Remove obsolete prefix class support

Signed-off-by: Steven Barth <steven@midlink.org>

Sanitize RA values less

Pass ICMP configuration to user-space instead of directly applying it

Update README

script: don't resolve AFTR since DNS won't be applied yet anyway

Signed-off-by: Steven Barth <steven@midlink.org>

properly handle return codes

Signed-off-by: John Crispin <blogic@openwrt.org>

ra: use proper struct msghdr initializer to fix build error on musl

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

make route metric more compatible with linux defaults

Get rid of getifaddrs for interface ID detection

Signed-off-by: Steven Barth <steven@midlink.org>

Merge pull request #30 from themiron/master

Fix handling of DHCPv6 replies containing unrequested IA_NA/IA_PD options

Fix handling of DHCPv6 replies containing unrequested IA_NA/IA_PD options

example: prevent script from running in parallel to avoid races

set default information refresh time to 86400 as per RFC 4242

README: adjust for a bit of feature creep

Merge pull request #28 from janakj/master

Minor help string clarification for client-ID

Minor help string clarification for client-ID

Document that the value of -c must be a 16-bit type (network byte order) followed by a client-ID value.
For example, to use a UUID based client-ID (type 4, RFC 6355) one could use the following cmdline option:
  -c0004<128_bit_uuid_in_hex>

Avoid sending empty DHCPv6 release messages

Don't apply excess filter to DHCPv6 and unify odhcp6c_update_entry

More compatibility with non RFC-compliant servers

Signed-off-by: Steven Barth <steven@midlink.org>

Cleanup pidfile-gerneration

Be less picky when erroring out due to strange IA status-codes

Fix initial delay of SOLICIT message

Signed-off-by: Steven Barth <steven@midlink.org>

Fix msghdr initialization on musl

Be less verbose by default and add -v flag

update cer-id definition

softwires: avoid unnecessary allocations

Merge pull request #24 from themiron/master

Fix parsing empty IA_NA, IA_PD and invalid IA_ADDR options

Fix parsing empty IA_NA, IA_PD and invalid IA_ADDR options

Merge pull request #23 from dedeckeh/bugfixes

odhcp6c: Set bound state true before script_call in statefull mode

odhcp6c: Set bound state true before script_call in statefull mode

script: don't leak IA_PD / IA_NA if not yet bound

Ensure signal-safety of signal handlers

Export DHCPv6 server address to env

Send RS alternatingly with and without source MAC

Enable softwire-support by default

Also test for correct IA-ID for IA_NA

ia_na: use big-endian 1 as IAID

Remove BFD from master-branch (kept in BFD-branch)

Send router solicitations with source link-layer address

Filter FQDN as well

Improve filter

filter DNS-domain from passthru

Add support for DHCPv6 option passthru

Add MAPE / MAPT / LW4O6 to Readme

softwire: fix DMR parsing

Fix compiler warning

MAP: export type value in rules

Fix MAP parameter parsing

Ignore multiple MAP-E instances for now and output DMR / BR for every rule

Initial support for MAP & LW4O6 provisioning

Fix fallout

Revert "Revert to old behaviour regarding information requests"

This reverts commit c98181c4a48c57e405effd1dc9046aaaee6d480f.

Merge pull request #19 from dedeckeh/bugfixes

Add user-class option in help text and minor clean up

Add user-class option in help text and minor clean up

Correctly clear CER

Don't disable looping (meh)

Add initial support for CER-ID

Stop rebinding when all IAs are lost

Fix building with clang 3.4

Fix HMAC-MD5 verify in reconfigure

fix integer overflow after 50 days (thx Hauke Mehrtens)

ra: don't set nd_ra_{reachable,retransmit} to 0

Set default for min-update-interval to 30

Merge pull request #18 from kaspar030/rebase

add option to specify minimum interval for accepting RA or DHCP updates.

add option to specify minimum interval for accepting RA or DHCP updates.

This was fixed to 60s. Add '-m <seconds>' option to change default time.

Fix invalid use of open()