BSD: Address some coverity issues

script: Rework assert to pass coverity.

It does work differently for non debug builds, but we don't care
because we reset the value anyway.
This way maybe more descriptive and does avoid a write.

print_option: restore dhcpcd-7 behaviour on invalid env option

By not printing it.

print_option: try and terminate the env option, even on error.

DragonFlyBSD: No longer need to poll for IPv6 address flag changes

control: Add CTL_FREE_LIST to control memory management.

While here, don't try to avoid a malloc and copy of the data
because the script env now just has one buffer.

Fix control queue free argument.

script: Send the actual env len, not the buffer size to listeners.

script: optimise variable

print_option: terminate parameters with no value

script: assert that the env strings are correctly terminated

DHCP: If IP checksum is zero for the message, make it all ones.

This is required by RFC 768.

DragonFly: Now supports sending NA messages from userland

While here, fix NA to the right NetBSD version as well.

Fix prior.

BSD: warn if kernel does not support route(4) filtering

Merge branch 'master' of github.com:rsmarples/dhcpcd

BSD: Use read instead of recvmsg

Makes for a smaller binary.

FreeBSD: warn that we cannot detect overflow on the route socket.

Cast away an alignment error.

Linux: Fix compile on 3.x

DHCP: Rework checksuming so that the packet isn't touched.

Other than setting udp->uh_sum to zero which we need to do to
calculate the checksum.
Also, the UDP checksum needs to include a pseudo IP header
without options and mostly blank. Instead of changing the packet,
just checksum a blank object we've filled in with the needed
data from the given IP object and use this to start the UDP
checksum calculation with.

While here, improve the checksum function so it more matches the
in_cksum function as noted in RFC 1071 4.1 using 16 byte words.

routes: Fix a NULL dereference error for global static routes

No idea why you would want them, but just in case .....

Release dhcpcd-8.0.2

IP: Work with IP headers with options.

BSD: Warn when kernel does not support sending ND6 advertisements

ND6: Disable advertising on OS other than NetBSD and Linux

It's only used for IP6 address sharing to prefer an interface,
but this only works on NetBSD. It might work on Linux too.
For all other OS advertising either fails to work (OpenBSD)
or erroneously logs duplicate address messages (all other BSD's).

DragonFlyBSD: Fix compile

DHCP6: Change state to REQEST when any IA has NOBINDING status

Just to be more RFC conformant.

Fix a memory NULL deference when freeing static routes at exit.

This is less effort than changing the options API to work with
an interface object rather than an interface name.

Solaris: Avoid re-adding IPv6 addresses to preserve the prefix route

Really, kernels should just stop doing this, it's pointless.

Rework prior again so we log the address in debug.

Fix compile issue with prior.

IP: If no address lifetime support don't re-add the address

Unless we need to. If we do, there is a chance that the OS
will scrub the subnet route and re-add it. We want to avoid
this as it's likely in use.

DHCP: Avoid duplicate read of packet

If the BPF socket is open, discard any read from UDP as it
would be duplicated.

configure: Fix test for open_memstream

glibc-2.19 needs the return value testing....

script: ensure that tmp files are removed

fopen creates a new stream which will use a new fd.
Why using fdopen, the new stream will use the same fd, so when
it's closed the tmp file will really unlink.

Update comment

Mention 29-lookup-hostname rather than 10-wpa_supplicant

As the wpa_supplicant hook script is obsolete when using
wpa_suppliant-2.6 or newer build with CONFIG_MATCH_IFACE.

Fix define for prior

Remove ND6 Advertisement from SMALL builds.

This is quite a saving and is only used for IP6 address sharing
which only works on NetBSD.

NetBSD: Allow dhcpcd to be built without ARP support

Because NetBSD has RFC 5227 support in the kernel, we only need
ARP to announce addresses and we only need to do that when
address sharing which shouldn't be a concern for small
installations where size is paramount.

Release dhcpcd-8.0.1

Fix compile on NetBSD with ARP but not ARPING

Fix compile on Linux without ARPING

Free script buffers before exiting from a fork.

Fix build without INET.

Fix build without ARP and/or ARPING

Fix compile without IPv4LL

IPv4LL: Work once more on NetBSD

Fix SMALL build.

Release dhcpcd-8.0.0

BSD: Work around another coverity false positive.

Typo

Linux: Mark another coverity false positive.

inet: Don't lose the host route needed for the router

Linux: Work around a coverity issue

sa points to a struct sockaddr inside a union where the
biggest member os sockaddr_in6.
We cast sa to char *, and then write to the address offset
directly which will go beyond a sockaddr if we're writing
a sockaddr_in6 - but this is fine, the union caters for it.

dhcpcd: Don't leak resources if we can't find a newly added interface

dhcpcd: Don't leak memory when routing socket overflows

Change prior to be non-fatal

Linux: Care about the return of setsockopt

DHCP: On ARP duplication, ensure the interface pointer is valid

logerr: Care about fprintf return codes

Abort early if an error and return the overall result if we ever
need it.

OpenBSD: Fix compile.

Cast time_t to long long to satisfy printf.

There is no standard primitive for it.

Remove fcntl.h from here.

Use rebindtime rather than renewaltime.

DHCP: Always add the leased address

This is needed so that the lifetimes are refreshed.
While here, set the pltime to the renewal time.

DHCP6: Assert auto sla is only 32-bits

It already is due to the use of fls32 above, but this assertion
makes clang static analyser happy.

options: Fix a static analysis error

Can't actually happen because the string doesn't change and the
result is already checked above. However, it's pretty harmless to
add.

Fix prior

options: Fix a potential memory leak

Fix compile issue with prior

DHCP: Set address vltime and pltime to the length of the lease

The only OS which supports this is Linux, but more importantly it
will report the address as "dynamic" so that users can distinguish
it from statically added addresses.

arp: Fix a crash when dhcpcd forks or an interface removed

Because we try and free the ARP state when dhcpcd forks or an
interface is removed, the loop to work out the next state to free
could return invalid memory.
So test we have an ARP state each time we iterate and free.

route: Compare AF_UNSPEC family when cleaning old routes

hooks: Ensure header is empty before populating it.

Thanks to Andreas

10-wpa_supplicant: Correct stop diagnostic

Merge branch 'memstream'

Add SPDX identifiers to all dhcpcd source files.
See https://spdx.org/ for details.

Merge pull request #7 from sashko/master

Fix typo in configure

Fix typo in configure

DHCP6: Fix INFORM support

script: enable if_ssid again

script: Fix a memory error

ND: Ensure that ltime is always set

Remove now unused variables.

Merge branch 'master' into memstream

route: Build list of static routes and routers correctly.

They they need to be inserted by rt_proto_add().

script: Write variables to a FILE

Each variable is NULL terminated inside the file just like the
control stream which saves us from having to flatten it when
writing to the control stream.
Once written, create env pointers to the start of each string
just after the NULL terminator.

This also means that we just need to free two buffers when
dhcpcd exits (FILE buffer and env buffer) rather than each variable
individually.

If open_memstream(3) is not supported by libc then dhcpcd will
write to a file in /tmp instead.

RA: expire whole RA on carrier up

If preserving IP, we need to expire the whole RA now that we care
about RDNSS and DNSSL lifetimes. Cheat by setting the aquired time
to something very low so it all expires.

arp: announce address from most preferred interface

Now we no longer sort interfaces, this is needed.

Merge branch 'master' into rbtree

dhcpcd: don't expire RA on carrier down when when preserving IP

RA: expire RDNSS and DNSSL entries

This allows us to remember N fully expired RA's which works around
an obscure issue where a received RA has no lifetime or any
prefixes with lifetimes but does have an instruction to start DHCP6.

It was harmless but filled the log with spam and now there is no
log spam!

Merge pull request #6 from DanielG/master

30-hostname: Fix typo: hsort -> hshort

30-hostname: Fix typo: hsort -> hshort

DHCP6: Keep running regardless of RA state

We should keep DHCP6 running even if routers expire or link changes.
This is noted in RFC3315 18.1.2:
   If the client receives no responses before the message transmission
   process terminates, as described in section 14, the client SHOULD
   continue to use any IP addresses, using the last known
   lifetimes for those addresses, and SHOULD continue to use any other
   previously obtained configuration parameters.

Fix a gcc warning about signed overflow

route: Compile on older BSD platforms

route: Use order from message / config

We can't just make the order up as that is prone to error.
So apply ordering as from the source.