update DNSSEC TODO

resolved: properly propagate query candidate error

We already properly propagate errors from transactions to queries. Make sure that errors that happened during handling
of query candidates are propagated to the query, too.

resolved: replace DNS_TRANSACTION_RESOURCES by DNS_TRANSACTION_ERRNO

Whenever we encounter an OS error we did not expect, we so far put the transaction into DNS_TRANSACTION_RESOURCES
state. Rename this state to DNS_TRANSACTION_ERRNO, and save + propagate the actual system error to the caller. This
should make error messages triggered by system errors much more readable by the user.

catalog: add DNSSEC log messages to message catalog

catalog: fix line width to 79 chars

Line breaks default to 119 characters for systemd sources now, configured through the .vimrc and .dir-local.el files.
However, for the catalog files we really should stick to 79 chars, as they are regularly shown on terminal screens.

resolved: log recognizably about DNSSEC downgrades

If we downgrade from DNSSEC to non-DNSSEC mode, let's log about this in a recognizable way (i.e. with a message ID),
after all, this is of major importance.

resolved: synthesize RRs for data from /etc/hosts

This way the difference between lookups via NSS and our native bus API should become minimal.

resolved: when synthesizing RR responses, own the name fully

When we synthesize A/AAAA for domains like "localhost", then make sure we generate ENODATA if the user asks for RR
types such a RP to be solved on the name. Previously, we'd pass the error back in that case that was generated from the
usual lookup procedure.

resolved: properly handle LLMNR/TCP connection errors

The LLMNR spec suggests to do do reverse address lookups by doing direct LLMNR/TCP connections to the indicated
address, instead of doing any LLMNR multicast queries. When we do this and the peer doesn't actually implement LLMNR
this will result in a TCP connection error, which we need to handle. In contrast to most LLMNR lookups this will give
us a quick response on whether we can find a suitable name. Report this as new transaction state, since this should
mostly be treated like an NXDOMAIN rcode, except that it's not one.

core: normalize error handling a bit, in setup_pam()

Assign errno-style errors to a variable called "r" when they happen, the same way we do this in most other calls. It's
bad enough that the error handling part of the function deals with two different error variables (pam_code and r) now,
but before this fix it was even three!

resolved: fix rcode formatting string length

Since we honour the edns rcode extension we need more than 4 bits to format it. To avoid further confusion, derive the
right length from the type.

github: extend README.md a bit

Link up more stuff on the GitHub landing page.

github: add a CONTRIBUTING.md file that github shows when a PR or issue is filed

github links up CONTRIBUTING.md if it exists from the issue/PR filing form. Let's add one, in the hope that people have
a look before filing a PR.

README: drop link to systemd-commits ML

Since we moved to github the commits ML is basically dead, hence don't advertise it anymore.

CODING_STYLE: make sure line break recommendation matches edit configuration

In the .vimrc and .dir-locals.el we suggest a line width of 119. We should recommend the same in CODING_STYLE.

resolve: fix wording in resolver statistics output

Now that we count both negative and positive validation results, we shouldn't claim we just counted RRsets.

resolved: log each time we increase the DNSSEC verdict counters

Also, don't consider RRs that aren't primary to the lookups we do as relevant to the lookups.

resolved: never store NSEC/NSEC3 RRs from the upper zone of a zone cut in cache

When using NSEC/NSEC3 RRs from the cache to derive existance of arbitrary RRs, we should not get confused by the fact
that NSEC/NSEC3 RRs exist twice at zone cuts: once in the parent zone, and once in the child zone. For most RR types we
should only consult the latter since that's where the beef is. However, for DS lookups we have to check the former.

This change makes sure we never cache NSEC/NSEC3 RRs from any parent zone of a zone-cut. It also makes sure that when
we look for a DS RR in the cache we never consider any cached NSEC RR, as those are now always from the child zone.

resolved: if we detect a message with incomplete DNSSEC data, consider this an invalid packet event

resolved: also collect statistics about negative DNSSEC proofs

We already maintain statistics about positive DNSSEC proofs, and count them up by 1 for each validated RRset. Now,
update the same counters each time we validated a negative query, so that the statistics are the combined result of all
validation checks, both positive and negative.

resolve: use different bitmap checking rules when we find an exact NSEC3 match, or just a covering enclosure

If we are looking for a DS RR we need to check the NSEC3 bitmap of the parent zone's NSEC3 RR, not the one from the
child. For any other RR we need to look at the child's however, hence enforce this with the bitmaps.

Note that not coverign checks only the lower zone's NSEC3 bitmaps matter, hence the existing check is fine.

resolve: minor strings improvements

man: document systemd-resolve(8)

This also links up the new manpage from systemd-resolved.service(8), and makes a couple of unrelated additions.

emacs: also configure nxml parameters in .dir-locals.el

We configure the parameters we use for the docbook XML.

resolve: rename "systemd-resolve-host" tool to simply "systemd-resolve"

The tool resolves way more than just hosts, hence give it a more generic name. This should be safe, as the tool is
currently undocumented. Before we add documentation for it, let's get the name right.

This also moves the C source into src/resolve/ (from src/resolve-host/), since the old name is a misnomer now. Also,
since it links directly to many of the C files of resolved it really belongs into resolved's directory anyway.

resolved: get rid of unnecessary if check

DNS_ANSWER_FOREACH() can deal with NULL answers anyway, let's simplify our code here.

resolved: split out RR synthesis logic into its own C file

Also, while we are at it, set the "authenticated" bit for everything we synthesize

resolved: move dns_type_to_af() to dns-type.c

resolve: generate a nice clean error when clients try to resolve a name when the network is down

resolved: when checking whether a link is relevant, check kernel operstate

This mimics what networkd is doing to detect a carrier.

resolved: fix typo

Noticed by @vcaputo

resolved: fix RR key reduction logic

Fixes #2380.

logind,machined: bump TasksMax=

Issue #2388 suggests the current TasksMax= setting for user processes is to low. Bump it to 12K. Also, bump the
container TasksMax= from 8K to 16K, so that it remains higher than the one for user processes.

(Compare: the kernel default limit for processes system-wide is 32K).

Fixes #2388

Merge pull request #2428 from msekletar/nspawn-alloca-fix

nspawn: fix memory leak

nspawn: fix memory leak

Merge pull request #2425 from yuwata/journal-remote-v4

ZJS: remove unnecessary oom check after strdupa().

Merge pull request #2423 from keszybz/udevd-crash-in-cleanup

udevd: do not clean up fds in main

journal-remote: fix broken --getter option

This commit fixes the following broken --getter option:
when systemd-journal-remote is called with --getter option,
it causes the error meesage "Zero sources specified" and
the getter command will not be called.

journal-remote: add an example to man

journal-remote: output file name is determined by the remote hostname

When --url option is specified, e.g. --url='http://some.host:19531/entries'
retrieved remote journal entries will be stored to
/var/log/journal/remote/remote-some.host.journal

journal-remote: make --url option support arbitrary url

Currently, --url option supports the only form like http(s)://some.host:19531.
This commit adds support to call systemd-journal-remote as follwos:
systemd-journal-remote --url='http://some.host:19531'
systemd-journal-remote --url='http://some.host:19531/'
systemd-journal-remote --url='http://some.host:19531/entries'
systemd-journal-remote --url='http://some.host:19531/entries?boot&follow'
The first three example result the same and retrieve all entries.
The last example retrieves only current boot entries and wait new events.

Merge pull request #2318 from vcaputo/coalesce-ftruncates-redux

journal: coalesce ftruncate()s in 250ms windows

udevd: do not clean up fds in main

fds will also be closed during manager cleanup in run, leading
to an error when we try to close them again. It is now possible
to "leak" the fds on error, but it's an unlikely event and we
will exit immediately anyway.

Fixes #2418.

Merge pull request #2410 from dobyrch/master

systemctl: Allow 'edit' and 'cat' on unloaded units

Merge pull request #2412 from fbuihuu/device-fixes

Device fixes

device: make sure to not ignore re-plugged device

systemd automatically mounts device unless 'noauto' is part of the
mount options. This can happen during boot if the device is plugged at
that time or later when the system is already running (the latter case
is not documented AFAICS).

After the systemd booted, I plugged my USB device which had an entry
in /etc/fstab with the default options and systemd automatically
mounted it.

However I noticed that if I unplugged and re-plugged the device the
automatic mounting of the device didn't work anymore: systemd didn't
notice that the device was re-plugged.

This was due to the device unit which was not recycled by the GC
during the unplug event because in the case of automounting, the mount
unit still referenced it. When the device was re-plugged, the old
device unit was reused but it still had the old sysfs path (amongst
other useful information).

Systemd was confused by the stalled sysfs path and decided to ignore
the plug event.

This patch fixes this issue by simply not doing the sanity checking on
the sysfs path if the device is in unplugged state.

Merge pull request #2413 from msekletar/update-todo-reload

Remove TODO entry

Remove TODO entry

Feature was introduced by 06af2a0

Merge pull request #2405 from zonque/sysusers

sysusers: use GID_FMT and UID_FMT instead of %d

Merge pull request #2331 from yuwata/journal-remote-unit-v2

journal-remote: add SupplementaryGroups to systemd-journal-upload.service

Merge pull request #2332 from yuwata/journal-remote-tmpfiles-v2

journal-remote: tmpfiles.d/journal-remote.conf (v2)

Merge pull request #2080 from chaloulo/split-mode-host-remove-port-from-journal-filename

journal-remote: split-mode=host, remove port from journal filename

Merge pull request #2409 from snakeroot/dropin-doc-2

man: describe precedence of drop-in .conf files over unit files

Merge pull request #2411 from ssahani/activate4

systemd-activate: Add support for datagram socket

man: add man for systemd-activate datagram syntax

systemd-activate: Add support for datagram sockets

core: Add flexible way to provide socket type
the socket type should be a diffrent argumet
in make_socket_fd . In this way we can set the socket
type like SOCK_STREAM SOCK_DGRAM in the address.

journal-remote: modify make_socket_fd

systemctl: Allow 'edit' and 'cat' on unloaded units

Don't fail if the unit has a LoadError; otherwise `systemctl edit` cannot be
used to correct the error (e.g. multiple "ExecStart=" lines).

Remove file changed warning so cat output isn't interspersed with log messages.

Fixes #829

man: describe precedence of drop-in .conf files over unit files

sysusers: use GID_FMT and UID_FMT instead of %d

Merge pull request #1359 from jengelh/ue

sysusers: help useless error message

Merge pull request #2071 from chaloulo/journal-upload-miss-logs

journal-upload : Ignore journal event when already in uploading state.

Merge pull request #2401 from lnykryn/sysv-split-deps-v4

v4: sysv-generator: do not join dependencies on one line, split them

sysv-generator: do not join dependencies on one line, split them

If there is a lot of initscripts and dependencies between them we might
end generating After= (and similar) lines which are longer then LINE_MAX
and thus rejected by parser in systemd.

Fixes #2099

Merge pull request #2393 from evverx/ignore-test-dnssec-complex

.gitignore: add test-dnssec-complex

Merge pull request #2371 from evverx/add-valgrind-helper-for-daemon-reexec

core: add valgrind helper for daemon-reexec

Merge pull request #2391 from keszybz/coverity

Coverity inspired fixes

Merge pull request #2341 from nmartensen/fstab-generator-fixes

Fstab-generator fixes

.gitignore: add test-dnssec-complex

This is a follow-up for 412577e

core: add valgrind helper for daemon-reexec

Inspired by https://github.com/systemd/systemd/issues/2187#issuecomment-165587140

systemd: remove dead code

We only go to fail label if pam_pid <= 0.

CID #1306746.

Add assert in barrier code

This function is not supposed to fail, it only returns a boolean.
So add an assert in case anyone calls it without proper initialization
of *b.

timesyncd: use (void) to mark ignored value

CID #1325772.

test-date: check return value

CID #1327432.

test-cgroup-mask: check return value

CID #1339830.

libsystemd-network: avoid double free on error

This could happen if the remote sent us a badly formatted
option.

CID #1317206.

libsystemd-network: use assert_se

It cannot fail.

CID #1320623.

networkd: use (void) to mark ignored values

Null link or netdev are handled fine.

CID #1338084.

Merge pull request #2389 from bengal/dhcp-api-cleanup-v3

Improve libsystemd-networkd DHCP API (v3)

test-tmpfiles: actually test that the file is temporary

CID #1341451.

Merge pull request #2329 from ssahani/tunnel

networkd: tunnel add support to configure address "any"

Merge pull request #2337 from dhxgit/patch-1

Fix IPv6PrivacyExtension (networkd-ndisc.c)

Merge pull request #2054 from keszybz/nss-link-less-2

Nss link less 2

Merge pull request #2267 from gdamjan/dont-drop-critical

networkd: link - do not drop config for critical interfaces

dhcp: make DHCP6_OPTION_* enum public

libsystemd-network provides the public function
sd_dhcp6_client_set_request_option() to enable the request of a given
DHCP option. However the enum defining such options is defined in the
internal header dhcp6-protocol.h. Move the enum definition to the
public header sd-dhcp6-client.h and properly namespace values.

dhcp: make DHCP_OPTION_* enum public

libsystemd-network provides the public function
sd_dhcp_client_set_request_option() to enable the request of a given
DHCP option. However the enum defining such options is defined in the
internal header dhcp-protocol.h. Move the enum definition to the
public header sd-dhcp-client.h and properly namespace values.

dhcp: export routes as opaque objects

At the moment sd_dhcp_lease_get_routes() returns an array of structs
which are not defined in public headers. Instead, change the function
to return an array of pointers to opaque sd_dhcp_route objects.

Merge pull request #1607 from keszybz/lz4-remove-v1

Remove the old version of the lz4 stream compressor

Merge pull request #2085 from fbuihuu/more-use-of-check-load-state

core: use bus_unit_check_load_state() in transaction_add_job_and_depe…

Merge pull request #2222 from snakeroot/eventsplat

hwdb: remove references to udevadm info /dev/input/event*

Merge pull request #2387 from keszybz/mhd-offset

journal-gatewayd: fix offset

journal-gatewayd: fix offset

I was checking something when writing the patch and
committed this by mistake.

Merge pull request #2385 from zonque/bootctl

bootctl: use DRAW_TREE_RIGHT rather than hard-coded UTF-8 character

Merge pull request #2381 from jsynacek/journalctl-colors-v4

basic/terminal-util: introduce SYSTEMD_COLORS environment variable

bootctl: use DRAW_TREE_RIGHT rather than hard-coded UTF-8 character

Fixes #2384

Merge pull request #2372 from poettering/dnssec17

resolved bus API improvements

basic/terminal-util: introduce SYSTEMD_COLORS environment variable

... to determine if color output should be enabled. If the variable is not set,
fall back to using on_tty(). Also, rewrite existing code to use
colors_enabled() where appropriate.

resolved: rework DNSSECSupported property

Not only report whether the server actually supports DNSSEC, but also first check whether DNSSEC is actually enabled
for it in our local configuration.

Also, export a per-link DNSSECSupported property in addition to the existing manager-wide property.

resolved: add SetLinkXYZ() method counterparts on the Link object

So far, we exposed SetLinkXYZ() on the Manager interface, to set a couple of link properties. This adds similar calls
SetXYZ() on the Link interface, and makes sure the former is little more than a shortcut to the latter.

SetLinkXYZ() has the benefit of not requiring a GetLink() round trip for setting these properties, while the method
actually belongs to the Link objects, and this change corrects that.

sd-resolve: use UINT64_C() macros where appropriate

networkd: sd_bus_path_decode() returns 0, if the prefix doesn't match