update TODO

update TODO

Merge pull request #201 from mbiebl/drop-include_prefix

build-sys: Drop include_prefix

build-sys: Drop include_prefix

Appears to be unused and a leftover from the udev merge.

Merge pull request #144 from teg/udev-spawn-log-less-2

udevd: event - don't log about failures of spawn processes when this …

Merge pull request #200 from kaysievers/wip

build-sys: include libsystemd-journal and libudev in libshared

build-sys: include libsystemd-journal and libudev in libshared

Merge pull request #196 from dvdhrm/bus-map-props

tree-wide: fix memory leaks in users of bus_map_all_properties()

Merge pull request #198 from ivuk/fix_typo_timesyncd_conf

Fix typos in man/timesyncd.conf.xml

Merge pull request #192 from phomes/master

test-netlink-manual: typo fix

Fix typos in man/timesyncd.conf.xml

tree-wide: fix memory leaks in users of bus_map_all_properties()

If you use bus_map_all_properties(), you must be aware that it might
touch output variables even though it may fail. This is, because we parse
many different bus-properties and cannot tell how to clean them up, in
case we fail deep down in the parser.

Fix all callers of bus_map_all_properties() to correctly cleanup any
context structures at all times.

hwdb: add support for Alienware graphics amplifier

Unplugging and plugging in the cable will create various scancodes
on the keyboard controller.

Userspace within X should be able to interact with these to show
interesting messages. Assign them to generic prog1/prog2.

(David: add comment to hwdb explaining that these keycodes are reserved)

man: don't mention '/run' in hwdb.man

We do not support '/run' for hwdb files. Drop it from the man-pages so
people don't accidentally use it.

This was reported by: Peter Hutterer <peter.hutterer@who-t.net>

test-netlink-manual: typo fix

No functional change, but looked weird.

Merge pull request #178 from utezduyar/man-sd_bus_message_get_creds

Improve the documentation of bus credentials by mentioning send-time metadata. This needs more love, we should really clarify metadata details here. However, this is still better than nothing, so it's fine.

Merge pull request #183 from ssahani/net

Improve tun/tap logging by using the new log_*errno*() functions that set 'errno' explicitly. Also fix a bunch of incorrect errno/r confusions.

Merge pull request #191 from kaysievers/resolv

build-sys: merge convenience library libresolve

Merge pull request #189 from teg/rtnl-rename

Rename sd_rtnl to sd_netlink to prepare for further netlink-protocol support. Anything rtnl specific still uses the sd_rtnl prefix, but the generic parts (including the bus and message objects) are now called sd_netlink.

build-sys: merge convenience library libresolve

Merge pull request #187 from kaysievers/libcleanup

build-sys: simplify lib dependencies

build-sys: simplify lib dependencies

networkd: tuntap improve logging

Replaces  strerror() usage with log_netdev_error_errno()

sd-netlink: socket - move some functions from main source file

sd-netlink: message - split up source file

Split netlink-socket.c and rtnl-message.c from netlink-message.c.

sd-netlink: drop the write-queue

AF_NETLINK is not write-buffered, so this was actually never used.

sd-netlink: rename from sd-rtnl

Merge pull request #184 from l10n-tw/master

po: fix zh_TW mailing list again.

Merge pull request #167 from keszybz/line-oriented-ima-setup

ima-setup: write policy one line at a time

po: fix zh_TW mailing list again.

Merge pull request #179 from l10n-tw/master

po: fix zh_TW mailing list.

po: fix zh_TW mailing list.

Merge pull request #164 from l10n-tw/master

Add zh_TW translation.

po,catalog: add zh_TW translation.

man: mention sd_bus_message_get_creds as another way

of retrieving sd_bus_creds even though sd_bus_creds itself
contains very limited information.

selinux: whitespace fixes

Merge pull request #153 from crawford/cciss

rules: re-add cciss rules

Merge pull request #173 from mischief/ipforwarding-3

IPForwarding=kernel v3

Merge pull request #177 from ssahani/net

core: fix CID 996302

core: fix CID 996302

CID 996302:  Error handling issues  (CHECKED_RETURN)

man: document IPForward=kernel option

networkd: create "kernel" setting for IPForwarding

In 5a8bcb674f71a20e95df55319b34c556638378ce, IPForwarding was introduced
to set forwarding flags on interfaces in .network files. networkd sets
forwarding options regardless of the previous setting, even if it was
set by e.g. sysctl. This commit creates a new option for IPForwarding,
"kernel", that preserves the sysctl settings rather than always setting
them.

See https://bugs.freedesktop.org/show_bug.cgi?id=89509 for the initial
bug report.

Merge pull request #171 from teg/rtnl-broadcast-2

sd-rtnl: make joining broadcast groups implicit

sd-rtnl: make joining broadcast groups implicit

Merge pull request #143 from teg/networkd-packets-per-slave-mode

networkd: bond - only set packets_per_slave on balance-rr mode

Merge pull request #156 from filbranden/journal_leading_whitespace

journald: do not strip leading whitespace from messages

Merge pull request #166 from zonque/kmod

kmod-setup: don't print warning on -ENOSYS

kmod-setup: don't print warning on -ENOSYS

-ENOSYS is returned from kmod_module_probe_insert_module() if a module isn't
available, not -ENOENT. Don't spit out a warning in that case unless the
warn_if_unavailable flag is set.

Also factor out the condition into an own variable for better readability.

Merge pull request #170 from teg/rtnl-recv

sd-rtnl: use netlink header rather than ucred in socket_recv_message()

sd-rtnl: use netlink header rather than ucred in socket_recv_message()

Lets us simplify the function and drop SO_PASSCRED.

Thanks to Alexander Larsson and David Herrmann.

ima-setup: write policy one line at a time

ima_write_policy() expects data to be written as one or more
rules, no more than PAGE_SIZE at a time. Easiest way to ensure
that we are not splitting rules is to read and write one line at
a time.

https://bugzilla.redhat.com/show_bug.cgi?id=1226948

Merge pull request #84 from blueyed/zsh-optimize-filter_units_by_property

zsh-completion: optimize _filter_units_by_property

kmod-setup: when we fail to load a kmod, log the error cause

(Also, downgrade message from LOG_ERROR to LOG_WARNING, after all we
don't care much and just proceed)

core: Let two more booleans survive a daemon-reload

Without the boolean bus_name_good services as well as cgroup_realized
for units a unit of Type=dbus and ExecReload sending SIGHUP to $MAINPID
will be terminated if systemd will be daemon reloaded.

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=746151
https://bugs.freedesktop.org/show_bug.cgi?id=78311
https://bugzilla.opensuse.org/show_bug.cgi?id=934077

Merge pull request #158 from ssahani/net

Net

Merge pull request #157 from filbranden/revert_unused_result

build-sys: suppress warnings of unused results

Merge pull request #160 from kaysievers/libsplit

build-sys: split internal basic/ library from shared/

Merge pull request #150 from poettering/journal-no-recv-loop

journald: don't employ inner loop for reading from incoming sockets

build-sys: split internal basic/ library from shared/

basic/      can be used by everything
            cannot use anything outside of basic/

libsystemd/ can use basic/
            cannot use shared/

shared/     can use libsystemd/

sd-network: fix CID 1306198

networkd: improve logging

Replace strerror() usage with log_netdev_error_errno()

networkd: macvlan improve logging

 Replace strerror() usage with log_netdev_error_errno()

build-sys: suppress warnings of unused results

Unfortunately, gcc keeps warning about those even when we use an
explicit (void) cast to indicate we are not interested in the result.

LLVM's clang does not have that issue and works fine with the casts.

GCC bug being tracked at:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66425

Until that GCC bug is fixed (and the version with the fix is in
many/most distributions) or we switch to LLVM as the default compiler,
it looks like we'll have to disable this warning by default...

Tested by building files known to present warnings about unused results
without the suppression, confirmed that the warnings were no longer
present with this patch applied.

This partially reverts commit 00c11bc53a1d37 ("build-sys: don't suppress irrelevant warnings").

journald: do not strip leading whitespace from messages

Keep leading whitespace for compatibility with older syslog
implementations.  Also useful when piping formatted output to the
`logger` command.  Keep removing trailing whitespace.

Tested with `pstree | logger` and checking that the output of
`journalctl | tail` included aligned and formatted output.

Confirmed that all test cases still pass as expected.

Merge pull request #3 from threatgrid/more_cgtop_enhancements

cgtop enhancements for easier machine-readable output

cgtop: IO readings are valid if any data is available, even if unchanged since last tick

Emit "0" rather than "-" if no change in IO values are seen for a process since
last tick, so long as accounting has registered content at all.

Merge pull request #6 from xnox/drop-name

shared: Drop 'name=' prefix from SYSTEMD_CGROUP_CONTROLLER define.

rules: re-add cciss rules

The original commit (1aff206) doesn't explain why these were removed.
This adds them back since they are in fact needed.

Merge pull request #152 from zonque/buildsys

build-sys: make man/systemd.directives.xml depend on man/custom-entit…

Merge pull request #148 from teg/sd-network-race

sd-network: allow the state dir to be created after the monitor

Merge pull request #151 from mischief/dns-shared-3

move dns code from resolve to shared v3

build-sys: make man/systemd.directives.xml depend on man/custom-entities.ent

Currently, the following command sequence fails:

  make distclean
  ./autogen.sh c
  make distcheck

That's because the command invoked to build man/systemd.directives.xml needs
man/custom-entities.ent to function, which itself isn't a dependency.

The $(filter-out $<,$^) logic used to filter out everything from the
prerequisites except for the first word, which doesn't work anymore
now. Use $(SOURCE_XML_FILES) instead.

Include tools/compile-unifont.py in the source tarball

This prevents a build failure when /usr/share/fonts/unifont/unifont.hex
is newer than unifont-glyph-array.bin.

bus: we now support path_namespace=/

Our bloom-filters support root-path matching. Make sure we properly add
the path_namespace= tag.

bus: fix pattern matching

DBus-spec defines two different pattern matchings:

  1) Path and namespace prefix matching. In this case, A matches B either
     if both are equal, or if B is fully included in the namespace of A.
     In other words, A has to be a prefix of B, but end with a separator
     character (or the following character in B must be one).

     This is used for path_namespace= and arg0namespace=

  2) The other pattern matching is used for arg0path= which does a two-way
     matching. That is, A must be a prefix of B, or B a prefix of A.
     Furthermore, the prefix must end with a separator.

Fix the sd-bus helpers to reflect that. The 'simple_' and 'complex_'
prefixes don't make any sense now, but.. eh..

bus: fix test-bus-kerne-bloom.c to match properly

Make sure we actually verify our match-rules are executed properly. Right
now all we test is the bloom-matches, which are non-reliable as they leave
through false-positives.

bus: fix arg0path= two-way matching

DBus spec clearly defines arg0path= to be a two-way matching. That is,
either the matcher or the matchee can be a prefix of the other to match.
This is not possible to implement with bloom-filters. Instead, we'd have
to add a separate filter for each prefix. This is non-trivial, though.
Hence, just skip the match for now and match locally.

bus: fix bloom_add_prefixes() to add all required data

Lets look at an example where we add arg0="/foo/bar/waldo" to a
bloom-filter. The following strings are added:
  "arg0:/foo/bar/waldo"
  "arg0-slash-prefix:/foo/bar"
  "arg0-slash-prefix:/foo"

Two problems arise:

1) If we match on "arg0path=/foo/bar/waldo", the dbus-spec explicitly
states that equal strings are also considered prefixes. However, in the
bloom-match, we can only provide a single match-filter. Therefore, we have
to add "arg0-slash-prefix:/foo/bar/waldo" there, but this never occured in
the bloom-mask of the message.
Hence, this patch makes sure bloom_add_prefixes() adds the full path as
prefix, too.

2) If we match on "arg0path=/foo/", the dbus-spec states that arg0path
does prefix-matching with the trailing slash _included_, unlike
path_namespace= matches, which does *not* include them. This is
inconsistent, but we have to support the specs. Therefore, we must add
prefixes with _and_ without trailing separators.
Hence, this patch makes sure bloom_add_prefixes() adds all prefixes with
the trailing slash included.

The final set of strings added therefore is:
  "arg0:/foo/bar/waldo"
  "arg0-slash-prefix:/foo/bar/waldo"
  "arg0-slash-prefix:/foo/bar/"
  "arg0-slash-prefix:/foo/bar"
  "arg0-slash-prefix:/foo/"
  "arg0-slash-prefix:/foo"
  "arg0-slash-prefix:/"

libsystemd-network: use domain validation instead of hostname validation for dhcp domain option

previously hostname_is_valid was used to validate domain names, which
would silently drop perfectly valid dns names that were longer than a
single dns label.

shared: add convenience function for validating dns names

resolve: move dns routines into shared

journald: don't employ inner loop for reading from incoming sockets

Otherwise, if the socket is constantly busy we will never return to the
event loop, but we really need to to dispatch other (possibly more
high-priority) events too. Hence, return after dispatching one message
to the event handler, and rely on the event loop calling us back
right-away.

Fixes #125

sd-network: allow the state dir to be created after the monitor

We now listen for new subdirs of /run/systemd, and /run/systemd/netif in case
/run/systemd/netif/links does not exist.

Merge pull request #147 from poettering/cmsg

util: introduce CMSG_FOREACH() macro and make use of it everywhere

networkd: bond - only set packets_per_slave on balance-rr mode

Otherwise the creation of the bond fails.

util: introduce CMSG_FOREACH() macro and make use of it everywhere

It's only marginally shorter then the usual for() loop, but certainly
more readable.

Merge pull request #146 from arvidjaar/pr/udev-import-program-exit-code-man

man: clarify that IMPORT{program} is done only for zero exit code

Fix typo

Follow up for 7c918141ed.

man: clarify that IMPORT{program} is done only for zero exit code

udevd: event - don't log about failures of spawn processes when this is expected

PROGRAM and IMPORT{program} uses the exit code of the spawn process to decide if a rule matches or not,
a failing process is hence normal operation and not something we should warn about.

We still warn about other types of failing processes.

Merge pull request #142 from teg/sd-network-unref-NULL

sd-network: allow NULL in sd_network_monitor_unref

Merge pull request #140 from teg/man-udev-timeout

man: udevd - correct default event timeout

sd-network: allow NULL in sd_network_monitor_unref

Match rest of codebase, we always allow unref'ing NULL.

Merge pull request #85 from keszybz/selinux-context

test-copy: test copy_bytes()

man: udevd - correct default event timeout

This was changed from 30 to 180 seconds quite some time ago.

sd-bus: remove ucred parameter from bus_message_from_header() since we don't use it anymore

sd-bus: fix early exit when we lack all data in bus_get_owner_creds_dbus1()

build-sys: upgrade shadow variable warnings to errors

bus-message: remove shadow warning with log_debug_bus_message()