Lennart Poettering [Tue, 4 Dec 2018 18:50:11 +0000 (19:50 +0100)]
resolved: remove redundant code
Lennart Poettering [Tue, 4 Dec 2018 18:29:15 +0000 (19:29 +0100)]
resolved: add some assert()s
Lennart Poettering [Tue, 4 Dec 2018 18:27:25 +0000 (19:27 +0100)]
resolved: be more careful with types in dns_stream_writev()
Let's not name a variable of type ssize_t "r". We usually use "r" for
return values of API calls that return some kind of error as in int.
This creates a lot of confusion if used differently here, which actually
resulted in connect()'s return value being assigned to this mistyped "r"
by accident.
Let's rename the variable "m" hence, and not use it for connect() return
values.
Lennart Poettering [Tue, 4 Dec 2018 18:26:54 +0000 (19:26 +0100)]
resolved: line split dns_stream_new() function signature
Lennart Poettering [Tue, 4 Dec 2018 18:14:51 +0000 (19:14 +0100)]
resolved: reorder things, to place registration of DnsStream in Manager close to each other
Lennart Poettering [Tue, 4 Dec 2018 18:14:28 +0000 (19:14 +0100)]
resolved: ensure DnsStream.fd is initialized before first error path
Lennart Poettering [Tue, 4 Dec 2018 18:13:55 +0000 (19:13 +0100)]
resolved: simplify on_stream_io() a bit
Evgeny Vereshchagin [Fri, 7 Dec 2018 08:49:51 +0000 (09:49 +0100)]
travis: also compile systemd on Debian Testing
This should help to catch issues like https://github.com/systemd/systemd/issues/11075
Filipe Brandenburger [Wed, 5 Dec 2018 22:13:02 +0000 (14:13 -0800)]
man: fix reference to sd_journal_print(3)
The reference in systemd-journald.service(8) was incorrectly pointing to
man volume 4 instead of 3.
This commit should fix the broken link at
https://www.freedesktop.org/software/systemd/man/systemd-journald.service.html
Yu Watanabe [Fri, 7 Dec 2018 09:11:36 +0000 (18:11 +0900)]
missing: re-add drm related entries
This effectively reverts
dab28f09051445fe370466e767e31652e0804a0f.
Fixes #11075.
Yu Watanabe [Thu, 6 Dec 2018 17:51:56 +0000 (18:51 +0100)]
bash-completion: analyze: support 'security'
Zbigniew Jędrzejewski-Szmek [Fri, 7 Dec 2018 07:03:32 +0000 (08:03 +0100)]
Merge pull request #11072 from yuwata/fix-networkctl-ip6gre
networkctl: fix TYPE field of ip6gre devices
Yu Watanabe [Thu, 6 Dec 2018 16:42:44 +0000 (17:42 +0100)]
missing: add ARPHRD_IP6GRE
This fixes TYPE field of networkctl's output for ip6gre devices.
Yu Watanabe [Thu, 6 Dec 2018 16:42:21 +0000 (17:42 +0100)]
af-list,arphrd-list: also include relevant missing_*.h headers
Yu Watanabe [Fri, 7 Dec 2018 02:48:37 +0000 (11:48 +0900)]
journal: use missing_audit.h to generate audit_type-list.txt
Yu Watanabe [Thu, 6 Dec 2018 16:38:37 +0000 (17:38 +0100)]
cap-list: use missing_capabilty.h to generate list of capabilities
Yu Watanabe [Fri, 7 Dec 2018 01:53:40 +0000 (02:53 +0100)]
Merge pull request #11038 from keszybz/man-timeouts
Small improvements for documentation of timeout options
Zbigniew Jędrzejewski-Szmek [Thu, 6 Dec 2018 14:04:14 +0000 (15:04 +0100)]
systemctl: be nice to users and give hint how to specify "-.mount"
https://bugzilla.redhat.com/show_bug.cgi?id=1656639
Using "--" is a trick that is hard to discover. Let's give users a hint:
$ build/systemctl status -.service
build/systemctl: invalid option -- '.'
Hint: to specify units starting with a dash, use "--":
build/systemctl [OPTIONS...] {COMMAND} -- -.service ...
I use program_invocation_name because that's what getopt seems to use.
"::" is used in the option string so that getopt doesn't complain about
a missing argument in case somebody passes "-." as the argument. After all
"." is not a real option.
Zbigniew Jędrzejewski-Szmek [Thu, 6 Dec 2018 08:56:37 +0000 (09:56 +0100)]
shared/blkid-util: do not include util.h needlessly
Yu Watanabe [Thu, 6 Dec 2018 17:03:01 +0000 (18:03 +0100)]
Revert "pager: close all fds when forking off pager"
This reverts commit
55844aebb60b42f56335eb147dd395269d8fb686.
As `exe_name_pipe` are closed by safe_fork().
Yu Watanabe [Thu, 6 Dec 2018 18:29:42 +0000 (19:29 +0100)]
Merge pull request #11056 from poettering/resolved-ifindex
resolved: request incoming ifindex for DNS UDP packets, too
Yu Watanabe [Thu, 6 Dec 2018 18:16:53 +0000 (19:16 +0100)]
Merge pull request #11066 from yuwata/test-network-tiny-cleanups
test-network: several cleanups
Alexey Bogdanenko [Thu, 6 Dec 2018 14:54:46 +0000 (17:54 +0300)]
test-resolved: fix error code check in assertion
If file "/etc/hosts" does not exist, fopen fails and sets errno to ENOENT
("No such file or directory"). So errno should be compared with ENOENT.
This mistake causes test test-resolved-etc-hosts to fail when run on Debian
image built with mkosi.debian included in the repo. The image does not include
"/etc/hosts" file as it is not created by debootstrap, see debootstrap manpage
https://manpages.debian.org/stretch/debootstrap/debootstrap.8.en.html.
Zbigniew Jędrzejewski-Szmek [Thu, 6 Dec 2018 16:19:21 +0000 (17:19 +0100)]
Merge pull request #10935 from poettering/rlimit-nofile-safe
Merged by hand to resolve a trivial conflict in TODO.
Yu Watanabe [Thu, 6 Dec 2018 15:54:27 +0000 (16:54 +0100)]
Merge pull request #11063 from yuwata/update-missing-v3
missing: split missing.h into small pieces
Yu Watanabe [Thu, 6 Dec 2018 15:46:16 +0000 (16:46 +0100)]
test-network: make search_words_in_file() hit multi words
Yu Watanabe [Thu, 6 Dec 2018 15:02:55 +0000 (16:02 +0100)]
test-network: delete rules before running next test
Yu Watanabe [Thu, 6 Dec 2018 14:33:23 +0000 (15:33 +0100)]
test-network: add a line break after starting networkd
This may be ugly. But otherwise, outputs of several check commands
are shown without new line...
Yu Watanabe [Thu, 6 Dec 2018 14:14:57 +0000 (15:14 +0100)]
test-network: check link before starting dnsmasq
Yu Watanabe [Thu, 6 Dec 2018 14:12:28 +0000 (15:12 +0100)]
test-network: show command outputs
Yu Watanabe [Thu, 6 Dec 2018 13:59:49 +0000 (14:59 +0100)]
test-network: sort imported modules
Yu Watanabe [Thu, 6 Dec 2018 13:57:20 +0000 (14:57 +0100)]
test-network: sort links and units
Yu Watanabe [Thu, 6 Dec 2018 14:29:55 +0000 (15:29 +0100)]
test-network: fix typo in class name
Zbigniew Jędrzejewski-Szmek [Thu, 6 Dec 2018 15:42:21 +0000 (16:42 +0100)]
Merge pull request #10963 from poettering/bus-force-state-change-signal
force PropertiesChanged bus signal on all unit state changes
Paweł Szewczyk [Fri, 30 Nov 2018 11:35:23 +0000 (12:35 +0100)]
fd-util: Fix error handling in safe_fclose
Function fclose_nointr returns negative value on error.
Zbigniew Jędrzejewski-Szmek [Thu, 6 Dec 2018 14:26:05 +0000 (15:26 +0100)]
Merge pull request #10994 from poettering/sd-bus-tweaks
small sd-bus tweaks
Yu Watanabe [Thu, 6 Dec 2018 13:32:52 +0000 (14:32 +0100)]
Merge pull request #10988 from ssahani/IFLA_BRPORT_MCAST_TO_UCAST
networkd: bridge add support to configure multicast_to_unicast
Yu Watanabe [Thu, 6 Dec 2018 13:27:36 +0000 (14:27 +0100)]
Merge pull request #11032 from ssahani/invert-rule
networkd: RPDB rule - add support to configure inverted rule.
Yu Watanabe [Thu, 6 Dec 2018 06:41:52 +0000 (07:41 +0100)]
tree-wide: drop missing.h from headers and use relevant missing_*.h
Yu Watanabe [Thu, 6 Dec 2018 06:30:35 +0000 (07:30 +0100)]
lockfile: drop unnecessary headers from lockfile-util.h
Yu Watanabe [Thu, 6 Dec 2018 06:23:23 +0000 (07:23 +0100)]
nsflsgs: drop missing.h and use missing_sched.h
Yu Watanabe [Thu, 6 Dec 2018 06:20:07 +0000 (07:20 +0100)]
util: drop missing.h from socket-util.h
Yu Watanabe [Thu, 6 Dec 2018 06:19:27 +0000 (07:19 +0100)]
util: drop missing.h from time-util.c
Yu Watanabe [Thu, 6 Dec 2018 05:57:27 +0000 (06:57 +0100)]
missing: move fs or mount related definitions to missing_fs.h
This also fixes errnous definition MS_REC -> MS_SLAVE.
Yu Watanabe [Thu, 6 Dec 2018 05:55:19 +0000 (06:55 +0100)]
missing: drop falloc related definitions
All these values are exposed earlier than linux-3.11.
Let's drop them.
Yu Watanabe [Thu, 6 Dec 2018 06:13:02 +0000 (07:13 +0100)]
missing: move sched.h related definitions to missing_sched.h
Yu Watanabe [Thu, 6 Dec 2018 06:12:26 +0000 (07:12 +0100)]
missing: drop old drm related definitions
These values are exposed earlier than linux-3.11.
Let's use drm/drm.h.
Yu Watanabe [Thu, 6 Dec 2018 05:50:40 +0000 (06:50 +0100)]
missing: drop BPF_XOR as we have linux/bpf_common.h
Yu Watanabe [Thu, 6 Dec 2018 05:50:11 +0000 (06:50 +0100)]
missing: drop DM_DEFERRED_REMOVE as we have dm-ioctl.h
Yu Watanabe [Thu, 6 Dec 2018 05:49:18 +0000 (06:49 +0100)]
missing: move definition of secure_getenv() to missing_stdlib.h
Yu Watanabe [Thu, 6 Dec 2018 05:47:47 +0000 (06:47 +0100)]
missing: move rand related definitions to missing_random.h
Yu Watanabe [Thu, 6 Dec 2018 05:46:33 +0000 (06:46 +0100)]
missing: drop TIOCVHANGUP
This is defined before linux-3.11. Also, its value depnends on
architecture. So, it is not good to define in this form.
Yu Watanabe [Thu, 6 Dec 2018 05:44:47 +0000 (06:44 +0100)]
missing: add memfd related definitions to missing_mman.h
Yu Watanabe [Thu, 6 Dec 2018 05:43:12 +0000 (06:43 +0100)]
missing: move timer related definition to missing_timerfd.h
Yu Watanabe [Thu, 6 Dec 2018 05:41:34 +0000 (06:41 +0100)]
missing: move socket related definitions to missing_socket.h
Yu Watanabe [Thu, 6 Dec 2018 05:38:40 +0000 (06:38 +0100)]
missing: move fcntl related definitions to missing_fcntl.h
Yu Watanabe [Thu, 6 Dec 2018 05:36:30 +0000 (06:36 +0100)]
missing: move resource related definitions to missing_resource.h
Yu Watanabe [Thu, 6 Dec 2018 05:34:04 +0000 (06:34 +0100)]
missing: move audit related definitions to missing_audit.h
Yu Watanabe [Thu, 6 Dec 2018 05:32:41 +0000 (06:32 +0100)]
missing: move syscall related definitions to missing_syscall.h
Yu Watanabe [Thu, 6 Dec 2018 05:30:19 +0000 (06:30 +0100)]
random-util: drop duplicated linux/random.h
Yu Watanabe [Thu, 6 Dec 2018 05:29:22 +0000 (06:29 +0100)]
missing: drop old OOM related definitions
These are exposed earlier than linux-3.11. Let's just include linux/oom.h.
Yu Watanabe [Thu, 6 Dec 2018 05:27:51 +0000 (06:27 +0100)]
missing: move one more network related definition to missing_network.h
Yu Watanabe [Thu, 6 Dec 2018 05:24:20 +0000 (06:24 +0100)]
missing: use integer types used by kernel
By using them, we can decrease included headers except linux/types.h.
Susant Sahani [Thu, 6 Dec 2018 06:24:14 +0000 (11:54 +0530)]
test-network: add test for bridge MulticastToUnicast
Yu Watanabe [Thu, 6 Dec 2018 05:22:50 +0000 (06:22 +0100)]
macro: include errno.h
As several macros touch errno.
Susant Sahani [Thu, 6 Dec 2018 05:30:36 +0000 (11:00 +0530)]
test-network: Add test for invert RPDB rule
Susant Sahani [Mon, 3 Dec 2018 12:55:30 +0000 (18:25 +0530)]
networkd: RPDB rule - add support to configure inverted rule.
Closes #10706
Yu Watanabe [Tue, 4 Dec 2018 09:44:55 +0000 (10:44 +0100)]
missing: move several network related entries to missing_network.h
Yu Watanabe [Wed, 5 Dec 2018 22:13:17 +0000 (23:13 +0100)]
Merge pull request #11021 from ssahani/isatap
networkd: Add support to configure ISATAP tunnel
Lennart Poettering [Wed, 5 Dec 2018 17:42:32 +0000 (18:42 +0100)]
logs-show: use grey color for de-emphasizing journal log output
Lennart Poettering [Wed, 5 Dec 2018 17:38:50 +0000 (18:38 +0100)]
resolvectl: color "comment" output grey
Let's de-emphasize the "comment" output a bit, by making it grey.
Lennart Poettering [Wed, 5 Dec 2018 17:23:48 +0000 (18:23 +0100)]
resolvectl: add some basic highlighting to query output
Let's highlight the actual payload the user requested.
Lennart Poettering [Wed, 5 Dec 2018 16:42:09 +0000 (17:42 +0100)]
resolvectl: when doing reverse lookups, show ifindex as "comment" too in output
Similar as before: don't output ifindex twice on the same address, and
show it as comment only.
Do this for reverse lookup output and all other output too.
Lennart Poettering [Wed, 5 Dec 2018 16:31:35 +0000 (17:31 +0100)]
resolvectl: fix interface output when resolving hostnames→addresses
We already used in_addr_ifindex_to_string() which internally appends the
ifindex to the address with % if necessary. It's simply wrong to attach the
intreface a second time with % then. Also, it breaks stuff that cannot
deal with that. Hence, let's reformat this, and add the ifindex as a
comment to the output, and drop the second % suffix.
Lennart Poettering [Wed, 5 Dec 2018 13:02:06 +0000 (14:02 +0100)]
resolvectl: remove allow_loopback parameter from ifname_mangle()
All our bus calls validate whether the specified device is a loopback
device anyway on the server side. Let's hence simplify the client,
there's no value in optimizing error paths after all. But there is value
in simpler code.
Lennart Poettering [Wed, 5 Dec 2018 12:58:34 +0000 (13:58 +0100)]
resolvectl: for our per-link verbs alternatively accept interface name in -i
it's a bit confusing that we take two interfaces for verbs such as "dns"
or "domain": once after the verb, and once as --interface=. While
there's logic behind it, let's make this least surprising: if either is
specified be happy.
This means "resolvectl -i foo dns" is now equivalent to "resolvectl dns
foo …". Note that this is a tweak only, to minimize surprises. We don't
document this alternative syntax, and shouldn't to keep things simple.
Lennart Poettering [Wed, 5 Dec 2018 12:33:31 +0000 (13:33 +0100)]
resolvectl: rework ifname_mangle()
Let's compare the ifname passed in with what is set already if there is
something set already. Complain in that case. This makes commands such
as "resolvectl -i foo dns bar" less weird, as we'll refuse the duplicate
ifname specifications.
Also, free the old arg_ifname right before assigning the new, instead of
doing so in advance.
Lennart Poettering [Tue, 4 Dec 2018 22:18:50 +0000 (23:18 +0100)]
resolved: request incoming ifindex for DNS UDP packets, too
This information is useful to pass back to the caller, as it tells us
where we found the answer.
(While we are at it, fix the socket level for the RECVERR logic)
Fixes: #9778
Zbigniew Jędrzejewski-Szmek [Wed, 5 Dec 2018 16:19:35 +0000 (17:19 +0100)]
Merge pull request #11053 from poettering/rc-local-tweak
rc-local-generator message tweak
Lennart Poettering [Tue, 4 Dec 2018 17:42:31 +0000 (18:42 +0100)]
rc-local-generator: tweak log message generation
Let's be more explicit, and not use LOG_WARNING log level when there's
nothing really wrong.
Fixes: #11044
Follow-up for: #10955
Lennart Poettering [Tue, 4 Dec 2018 17:41:27 +0000 (18:41 +0100)]
rc-local-generator: small shortification
Evgeny Vereshchagin [Wed, 5 Dec 2018 10:38:53 +0000 (11:38 +0100)]
README: embed a CentOS CI badge into README
Plus be more specific about where the badges point to.
Marc-Antoine Perennou [Wed, 5 Dec 2018 10:28:45 +0000 (11:28 +0100)]
systemctl: drop my copyright
Signed-off-by: Marc-Antoine Perennou <Marc-Antoine@Perennou.com>
Evgeny Vereshchagin [Wed, 5 Dec 2018 06:19:46 +0000 (09:19 +0300)]
Merge pull request #11025 from evverx/clang-asan
travis: also run the tests built with clang under ASan+UBsan
Evgeny Vereshchagin [Wed, 5 Dec 2018 06:05:02 +0000 (09:05 +0300)]
Merge pull request #11051 from poettering/sock-addr-un-len
trick out ubsan
Lennart Poettering [Tue, 4 Dec 2018 17:21:25 +0000 (18:21 +0100)]
socket-util: Let's trick out ubsan when it validate struct sockaddr_un.sun_path[]
Linux is stupid and sometimes returns a "struct sockaddr_un" that is
longer than its fields, as it NUL terminates .sun_path[] even if it has
full length. ubsan detects this, rightfully. Since this is a Linux
misdesign let's trick out ubsan a bit.
Fixes: #11024
Lennart Poettering [Tue, 4 Dec 2018 17:20:48 +0000 (18:20 +0100)]
socket-util: break lines in sockaddr_pretty() signature
It's soo long, let's make this more readable.
Lennart Poettering [Tue, 4 Dec 2018 17:20:12 +0000 (18:20 +0100)]
socket-util: shorten code a bit
Oliver Smith [Tue, 4 Dec 2018 15:05:15 +0000 (16:05 +0100)]
man/daemon.xml: use AM_DISTCHECK_CONFIGURE_FLAGS
Change the man page to recommend setting AM_DISTCHECK_CONFIGURE_FLAGS
instead of DISTCHECK_CONFIGURE_FLAGS. This is the recommended way from
the automake manual, as otherwise the flag can't be overridden by the
user with something like:
DISTCHECK_CONFIGURE_FLAGS=--some-additional-flags make distcheck
References:
https://www.gnu.org/software/automake/manual/html_node/Checking-the-Distribution.html
https://osmocom.org/issues/3718
Vojtech Trefny [Tue, 4 Dec 2018 15:47:36 +0000 (16:47 +0100)]
rules: watch metadata changes on DASD devices
To make sure the change event is emitted and udev db is updated
after metadata changes.
Lennart Poettering [Tue, 4 Dec 2018 15:19:02 +0000 (16:19 +0100)]
Merge pull request #11041 from yuwata/update-missing-v2
missing: separate missing.h more
Lennart Poettering [Tue, 4 Dec 2018 15:15:09 +0000 (16:15 +0100)]
Merge pull request #11042 from yuwata/tiny-coding-style-fixes
Tiny coding style fixes
Chris Down [Tue, 4 Dec 2018 11:42:30 +0000 (11:42 +0000)]
cgroup: Use varname for cgroup_disable documentation
The current use of literal + replaceable is pretty ugly as it usually
ends up with cgroup_disable= rendered in quotes, which looks really
weird, and this doesn't conform with others of a similar type (for
example, the earlier `DefaultDependencies=no` discussion in the same
file.
Zbigniew Jędrzejewski-Szmek [Tue, 4 Dec 2018 13:32:10 +0000 (14:32 +0100)]
Merge pull request #11037 from poettering/json-table
optionally output tables as JSON
Lennart Poettering [Tue, 4 Dec 2018 11:33:03 +0000 (12:33 +0100)]
Merge pull request #11009 from poettering/root-cgroup-again
tweak root cgroup attribute fiddling for cgroupsv1 again
Yu Watanabe [Tue, 4 Dec 2018 07:26:09 +0000 (08:26 +0100)]
util: drop missing.h from util.h
Yu Watanabe [Tue, 4 Dec 2018 08:29:54 +0000 (09:29 +0100)]
tree-wide: add whitespace between type and variable name
AsciiWolf [Mon, 3 Dec 2018 14:52:24 +0000 (15:52 +0100)]
l10n: update Czech Translation
Yu Watanabe [Tue, 4 Dec 2018 08:18:21 +0000 (09:18 +0100)]
ip-protocol-list: sort headers
Yu Watanabe [Tue, 4 Dec 2018 07:08:15 +0000 (08:08 +0100)]
missing: move char{16,32}_t definitions to missing_type.h