options: Fix allocating the script option

When passing PARSE_STRING_NULL we expect to store the string
NULL terminated. As such, allocate space for it an ensure we have
space for it.

Release dhcpcd-8.1.3

DHCP: dl is always >0 at this point, so remove check.

Found by the LGTM bot.

compat: Add a guard to _strtoi.h

Not really needed but LGTM wants it which blows.
Should really take this upstream.....

Linux: Move RTA_OK and RTA_NEXT from a while to a for loop.

DHCP: Ensure we have a lease to extract options from.

DHCP6: Remove now unused vars

DHCP6: Add disabled code to allow packet replay

I get bored of adding similar code from time to time just to
aid debugging, so let's just add it in.

Someone else might find it useful too.

DHCP6: Rework delegation deprecation

Split ipv6_addaddrs out so ipv6_doaddr can operate on a single address.
Call this when deprecating delegated addresses to avoid calling
ipv6_addaddrs.

This allows a more simple ipv6_addaddrs that doesn't need to test
which address collection we are deleting from and removes DHCPv6
specific code from the generic IPv6 module.

DHCP6: Fix deprecating a delegated prefix

Be sure to remove the address using the right TAILQ member.
While here, simplify the initialisation of pd_pfxs.

options: Suboption 3 of NTP Server is a FQDN

As such it should be domain and not ip6address.

DHCP: Don't warn about expiry of deprecated addresses

IPv6ND: Fix reachable test

dhcpcd: Only report SSID when we have a carrier

DHCP6: Fix prior to log an error

DHCP6: Improve API for privsep

DHCP: Improve API for privsep

Linux: remove XX debug

Linux: Support kernels without PR_SET_MM_MAP

Linux: prefer ms RA times

The old sysctls have apparently been deprecated since 2.6.12.

Spelling fixes

Release dhcpcd-8.1.2

dhcpcd: report SSID connected to when we gain carrier

Helps debugging without the need for explicit debug.

RA: Set mtu to zero and then pull in the MTU option if any

Incase the router sets it back to unspecified.

OpenBSD: Support LINK_STATE_HALF_DUPLEX and LINK_STATE_FULL_DUPLEX

For if_msghdr messages from route(4).

IPv6ND: If a router is reachable again, don't solicit another.

RA: Warn if advertised MTU is greater than link MTU

BSD: Try and set linkmtu for the interface just to be nice.

If it fails, zero it out and try again - only the linkmtu should
fail really. This is fine as we fix it against the route itself.

OpenBSD: Fix prior - OS cannot set any RA bits such as hop limit.

BSD: strlcpy > strncpy

Solaris: Fix compile with prior

Linux: fix compile for prior

INET6: Apply hoplimit, reachable and retrans timer values from RA.

Only tested thus far on NetBSD.
While here, fix prior to not lose the first sorted route.

INET6: Sort routers when reachability changes

If unreachable, try and install a better default router.

script: Ensure pid is initialised.

DHCP: When rebinding ensure we have a DHCP ARP state

So we can defend the address.

Linux: rename _open_link_socket to if_linksocket

Linux: improve the netlink API more

By putting the callback arg with the callback function.

build: Improve detection of dlopen(3) in libdl

dev: remove pointless cast

INET: Fix corruption of IPv4 address flags when renewing

What a mistaka to maka!

Linux: always open a netlink socket when writing netlink messages

While here, bring some function names into the same style.

build: Improve import-src target

Incase a supposed source file file does not exist.

and more pretty

build: make prior output more pretty

build: Check compiler actually supports address sanitisation.

BSD: Disable kernel RTADV sysctl earlier

Just makes life a bit easier.

build: Enforce address sanitisaton when debugging.

dhcpcd: Set ctx.ifaces to NULL after freeing

Because they it might be used trying to find an interface to assign
our option routes to when freed if never used.

eloop: Fix a segfault if we free a NULL eloop

control: when closing, remove from eloop.

script: Improve API

DHCP6: Simplify opening the UDP port for all interfaces.

DHCP: Rework closing BPF and seeing if we need to open UDP

Just easier to read.

DHCP: Simplify opening the UDP port for all interfaces.

ARP: Add interface to arp_state

Simplifies the codes slightly and allows a backreference if
we need to pass the ARP state as a context.

Fix prior on other os

Solaris: Fix prior by receving IP6_PKTINFO

Split *_recv() into *_recv() and *_recvmsg() functions

The *_recvmsg() function has the whole packet in the first and
only iov.

Serves little purpose now, but will be used in the future.

Fix prior

Whitespace.

BSD: Use IP_RECVIF

IP_RECVIF is supported on all BSD platforms as well as Solaris,
so it's more widely available than IP_PKTINFO.

This allows us to ensure that all platforms can retrieve the
receving interface from UDP network packets and thus make the code
paths a lot easier.

eloop: Add eloop_clear function

This will allow forking processes for future privsep work.

INET: Fix a potential memory leak

When someone deletes the address from under us.

Linux: validate RTM_NEWADDR/RTM_DELADDR for AF_INET as well.

INET: If we fail to add an address that already exists, don't free it

Should not happen in production.....

Linux: Validate RTM_DELADDR/RTM_NEWADDR messages for IPv6

To ensure that if messages lag, they can be ignored.
How to do similar without a heavy getifaddrs call for IPv4?

Fix building on systems with musl (#10)

musl has its own definition of struct ethhdr, so only include
netinet/if_ether.h on systems with GLIBC.  For the ARPHDR constants, we
must include linux/if_arp.h instead.

dhcpcd: Run the STOPPED hook reason for the interface on timeout

If not in master mode.

Merge pull request #9 from vegerot/patch-1

Added documentation for undocumented option

DHCP: Fix prior to not waste space

Added documentation for undocumented option

The dbdir option is used to specify the location of the DHCPd database.  The default /var/db is not FHS compliant so some may wish to change this to

Typo, thanks to Arfrever.

Relese dhcpcd-8.1.1

FreeBSD: Fix test for setproctitle

DHCP: Fix strict aliasing of checksumming the pseudo header

in_cksum relies on accessing the data by uint16_t blocks.

Because the underlying object we send to in_cksum is struct ip,
we need to use union to allow access by uint16_t so strict aliasing
rules are not broken.

compat: Fix UB in arc4random.

DHCP: Fix aliasing issues accessing the UDP header

Not been a problem before, but it seems to be now.
Copy out the UDP header rather than accessing it directly.
Adjust the code to zero out the UDP checksum in the packet
to compensate ..... and put the checksum back afterwards.

compat: Go back to linux specific setproctitle

Solaris does not work with argv stamping and this is much cleaner
anyway.

Revert "compat: Use more portable setproctitle from nginx"

This reverts commit cbcb352f10de5ba5185860f20e148509081f9d2f.

Revert "compat: Fix setproctitle on Solaris"

This reverts commit c085a2836f8439bc3e9d01a85d399afb8ad1dc12.

compat: Fix setproctitle on Solaris

compat: Use more portable setproctitle from nginx

Hopefully works on older Linux and Solaris.

dhcp: Allow for trailing FCS in the packet.

dhcp: Fix incoming verification and checksum routines

While here remove unused get_option_raw define.

compat: Add setproctitle from LXC

inet6: Fix a potential crash learning addresses

Interface maybe active but without options.
In this case, check the global state of IPv6.

Release dhcpcd-8.1.0

BPF: Move validation logic from BPF to consumers

Even though we program the BPF filter should we trust it?
On Linux at least there is a window between opening the socket,
binding the interface and setting the filter where we receive data.
This data is NOT checked OR flushed and IS returned when reading.
We have no way of flushing it other than reading these packets!
But we don't know if they passed the filter or not ..... so we need
to validate each and every packet that comes through ourselves as well.
Even if Linux does fix this sorry state, who is to say other kernels
don't have bugs causing a similar effect?

As such, let's strive to keep the filters just for pattern matching
to avoid waking dhcpcd up.

Minor optimisation and fix to prior.

BSD: Add IFT_TUNNEL

Requires explicit support but shouldn't log a code.

DHCP6: Remove 0 sla and 0 prefix means whole prefix.

If we get a /48 we can still use /64 with a sla of 0.
It's all valid, my bad.

man: some virtual interfaces ....

For example dummy on Linux still works by default.
There could be others....

man: Note the default ignoring of virtual interfaces.

BSD: Ignore bridge interfaces

Linux: Ignore TAP and Bridge interfaces by default

We can check these places:
  /sys/class/net/tap0/tun_flags
  /sys/class/net/br0/bridge

BSD: Ignore the firewire interface as well.

BSD: Ignore interface groups as we would the interface name

Incase someone renames tap0 to foo4.

if: Ignore TAP interfaces by default

TAP interfaces are virtual ethernet and not really distinguishable
from real interfaces other than the interface name.

On BSD the interfaces are always called tapN.

DHCP6: Work better with infinite addresses

And as such just log infinite lease rather than an arbitary
number.