]> git.ipfire.org Git - thirdparty/systemd.git/log
thirdparty/systemd.git
5 years agoMerge pull request #10825 from keszybz/advertize-followup
Lennart Poettering [Mon, 19 Nov 2018 10:05:18 +0000 (11:05 +0100)] 
Merge pull request #10825 from keszybz/advertize-followup

udev link advertizing followup

5 years agoMerge pull request #10507 from cdown/cpu_acct
Lennart Poettering [Mon, 19 Nov 2018 09:57:48 +0000 (10:57 +0100)] 
Merge pull request #10507 from cdown/cpu_acct

cgroup v2: Don't require CPU controller for CPU accounting in 4.15+

5 years agotest: kill all processes launched by test-execute before exiting
Evgeny Vereshchagin [Sat, 17 Nov 2018 19:36:31 +0000 (20:36 +0100)] 
test: kill all processes launched by test-execute before exiting

As was shown in https://github.com/systemd/systemd/issues/10696#issuecomment-439613204,
currently `meson` waits for 1080 seconds (which is three times the global timeout) for the
test to fail completely even though it takes just two minutes for it to really fail. This
happens because the test itself leaves the services it has launched behind, which, in turn, makes
meson think that the test is still in progress. KILL_ALL with SIGKILL should make the issue
go away.

5 years agocatalog: update Polish translation
Piotr Drąg [Sun, 18 Nov 2018 18:04:53 +0000 (19:04 +0100)] 
catalog: update Polish translation

5 years agotest: let the shell find "touch" instead of hard-coded "/bin/touch"
Xi Ruoyao [Sun, 18 Nov 2018 18:51:25 +0000 (02:51 +0800)] 
test: let the shell find "touch" instead of hard-coded "/bin/touch"

5 years agobasic/missing: drop _ETHTOOL_LINK_MODE_MAX 10825/head
Zbigniew Jędrzejewski-Szmek [Sun, 18 Nov 2018 09:13:32 +0000 (10:13 +0100)] 
basic/missing: drop _ETHTOOL_LINK_MODE_MAX

It was only used in one place, where we don't actually need it, and
it is too easy to forget to update it when adding new items to the table.
Let's just drop it.

5 years agoudev: use an "inline" array instead of allocating for advertise mode
Zbigniew Jędrzejewski-Szmek [Sun, 18 Nov 2018 09:12:14 +0000 (10:12 +0100)] 
udev: use an "inline" array instead of allocating for advertise mode

The code is a bit shorter and we don't allocate the uint32_t[127] array.

5 years agobasic/util: import memeqzero from casync
Zbigniew Jędrzejewski-Szmek [Sun, 18 Nov 2018 09:42:39 +0000 (10:42 +0100)] 
basic/util: import memeqzero from casync

5 years agocgroup v2: DefaultCPUAccounting=yes if CPU controller isn't required 10507/head
Chris Down [Thu, 25 Oct 2018 13:03:58 +0000 (14:03 +0100)] 
cgroup v2: DefaultCPUAccounting=yes if CPU controller isn't required

We now don't enable the CPU controller just for CPU accounting if we are
on 4.15+ and using pure unified hierarchy, as this is provided
externally to the CPU controller. This makes CPUAccounting=yes
essentially free, so enabling it by default when it's cheap seems like a
good idea.

5 years agocgtop: Still try to get CPU statistics if controller-free
Chris Down [Sat, 17 Nov 2018 11:47:44 +0000 (11:47 +0000)] 
cgtop: Still try to get CPU statistics if controller-free

If CPU accounting is cheap, no controller necessarily needs to be
enabled here for us to be able to read statistics.

5 years agocgroup v2: Don't require CPU controller for CPU accounting in 4.15+
Chris Down [Sat, 17 Nov 2018 11:19:07 +0000 (11:19 +0000)] 
cgroup v2: Don't require CPU controller for CPU accounting in 4.15+

systemd only uses functions that are as of Linux 4.15+ provided
externally to the CPU controller (currently usage_usec), so if we have a
new enough kernel, we don't need to set CGROUP_MASK_CPU for
CPUAccounting=true as the CPU controller does not need to necessarily be
enabled in this case.

Part of this patch is modelled on an earlier patch by Ryutaroh Matsumoto
(see PR #9665).

5 years agotest-util: modernize
Zbigniew Jędrzejewski-Szmek [Sun, 18 Nov 2018 09:46:30 +0000 (10:46 +0100)] 
test-util: modernize

5 years agoMerge pull request #10819 from yuwata/fix-10807
Zbigniew Jędrzejewski-Szmek [Sun, 18 Nov 2018 09:27:29 +0000 (10:27 +0100)] 
Merge pull request #10819 from yuwata/fix-10807

ethtool: drop NetDevAdvertise

5 years agoethtool: drop NetDevAdvertise 10819/head
Yu Watanabe [Sat, 17 Nov 2018 14:08:22 +0000 (23:08 +0900)] 
ethtool: drop NetDevAdvertise

Fixes #10807.

5 years agomissing: add ETHTOOL_LINK_MODE_* and struct ethtool_link_settings
Yu Watanabe [Sat, 17 Nov 2018 14:42:27 +0000 (23:42 +0900)] 
missing: add ETHTOOL_LINK_MODE_* and struct ethtool_link_settings

5 years agoMerge pull request #10817 from evverx/audit-fuzzer
Zbigniew Jędrzejewski-Szmek [Sat, 17 Nov 2018 17:26:54 +0000 (18:26 +0100)] 
Merge pull request #10817 from evverx/audit-fuzzer

Add a fuzzer for process_audit_string

5 years agotest: also tests enumerating subsystems
Yu Watanabe [Sat, 17 Nov 2018 15:42:33 +0000 (00:42 +0900)] 
test: also tests enumerating subsystems

5 years agotests: introduce dummy_server_init and use it in all journald fuzzers 10817/head
Evgeny Vereshchagin [Sat, 17 Nov 2018 12:01:09 +0000 (13:01 +0100)] 
tests: introduce dummy_server_init and use it in all journald fuzzers

5 years agotravis: make the ASan & UBsan job a little bit stricter
Evgeny Vereshchagin [Fri, 16 Nov 2018 14:49:02 +0000 (15:49 +0100)] 
travis: make the ASan & UBsan job a little bit stricter

5 years agojournald: check whether sscanf has changed the value corresponding to %n
Evgeny Vereshchagin [Fri, 16 Nov 2018 22:32:31 +0000 (23:32 +0100)] 
journald: check whether sscanf has changed the value corresponding to %n

It's possible for sscanf to receive strings containing all three fields
and not matching the template at the same time. When this happens the
value of k doesn't change, which basically means that process_audit_string
tries to access memory randomly. Sometimes it works and sometimes it doesn't :-)

See also https://bugzilla.redhat.com/show_bug.cgi?id=1059314.

5 years agoconf-parse: use strjoina() where appropriate
Lennart Poettering [Fri, 16 Nov 2018 17:08:29 +0000 (18:08 +0100)] 
conf-parse: use strjoina() where appropriate

5 years agoconf-parse: drop unused prototype
Lennart Poettering [Fri, 16 Nov 2018 17:05:46 +0000 (18:05 +0100)] 
conf-parse: drop unused prototype

5 years agomacro: remove double evaluation in FLAGS_SET()
Lennart Poettering [Fri, 16 Nov 2018 20:04:38 +0000 (21:04 +0100)] 
macro: remove double evaluation in FLAGS_SET()

5 years agorc-local-generator: add comment explaining the background of the generator
Lennart Poettering [Fri, 16 Nov 2018 21:00:35 +0000 (22:00 +0100)] 
rc-local-generator: add comment explaining the background of the generator

This is not obvious, hence it deserves some form of documentation.
However, it's also ultimately an implementation detail, hence let's not
add this to the man page, but as a code comment, that is visible right
at the top of source file.

Fixes: #10675
5 years agotests: add a fuzzer for process_audit_string
Evgeny Vereshchagin [Fri, 16 Nov 2018 20:23:56 +0000 (21:23 +0100)] 
tests: add a fuzzer for process_audit_string

5 years agofix build with -Defi=false
asavah [Fri, 16 Nov 2018 19:02:00 +0000 (21:02 +0200)] 
fix build with -Defi=false

5 years agoMerge pull request #10805 from poettering/migrate-boot-loader-interface
Lennart Poettering [Fri, 16 Nov 2018 16:55:08 +0000 (17:55 +0100)] 
Merge pull request #10805 from poettering/migrate-boot-loader-interface

migrate boot loader interface doc from wiki into markdown (split out of #10495)

5 years agoMerge pull request #10751 from yuwata/sd-event-util
Lennart Poettering [Fri, 16 Nov 2018 16:54:43 +0000 (17:54 +0100)] 
Merge pull request #10751 from yuwata/sd-event-util

Reuse sd_event_source object for timer event

5 years agoMerge pull request #10770 from poettering/unit-done-log
Lennart Poettering [Fri, 16 Nov 2018 16:54:19 +0000 (17:54 +0100)] 
Merge pull request #10770 from poettering/unit-done-log

improvements to structure log events from PID1

5 years agoMerge pull request #10777 from poettering/seccomp-filter-others
Lennart Poettering [Fri, 16 Nov 2018 16:53:50 +0000 (17:53 +0100)] 
Merge pull request #10777 from poettering/seccomp-filter-others

list syscalls supported by the local kernel but not in any syscall groups explicitly in "systemd-analyze syscall-filter"

5 years agoMerge pull request #10785 from poettering/cgroup-join-removal
Lennart Poettering [Fri, 16 Nov 2018 16:53:26 +0000 (17:53 +0100)] 
Merge pull request #10785 from poettering/cgroup-join-removal

remove JoinControllers= setting

5 years agoMerge pull request #10802 from poettering/hide-only-on
Lennart Poettering [Fri, 16 Nov 2018 16:53:01 +0000 (17:53 +0100)] 
Merge pull request #10802 from poettering/hide-only-on

man: let's deprecate PermissionsStartOnly=

5 years agoMerge pull request #10804 from poettering/sd-boot-updates
Lennart Poettering [Fri, 16 Nov 2018 16:52:37 +0000 (17:52 +0100)] 
Merge pull request #10804 from poettering/sd-boot-updates

various sd-boot/EFI fixes (split out from #10495)

5 years agoMerge pull request #10806 from poettering/logind-many-fixes
Lennart Poettering [Fri, 16 Nov 2018 16:52:11 +0000 (17:52 +0100)] 
Merge pull request #10806 from poettering/logind-many-fixes

various smaller fixes and improvements to logind (split out of #10495)

5 years agodocs: tweak index.md generation and run it again 10805/head
Lennart Poettering [Tue, 23 Oct 2018 14:29:28 +0000 (16:29 +0200)] 
docs: tweak index.md generation and run it again

5 years agodocs: migrate boot loader interface from fdo wiki to git
Lennart Poettering [Tue, 23 Oct 2018 14:28:30 +0000 (16:28 +0200)] 
docs: migrate boot loader interface from fdo wiki to git

This imports
https://www.freedesktop.org/wiki/Software/systemd/BootLoaderInterface/
into our sources, and extends it substantially with various variables
now supported.

5 years agodocs/TRANSIENT-SETTINGS: drop PermissionsStartOnly= from 10802/head
Zbigniew Jędrzejewski-Szmek [Fri, 16 Nov 2018 15:20:31 +0000 (16:20 +0100)] 
docs/TRANSIENT-SETTINGS: drop PermissionsStartOnly= from

5 years agoanalyze: add some minimal ANSI highlighting of syscall-filter output 10777/head
Lennart Poettering [Wed, 14 Nov 2018 18:59:27 +0000 (19:59 +0100)] 
analyze: add some minimal ANSI highlighting of syscall-filter output

5 years agoseccomp: add some missing syscalls to filter sets
Lennart Poettering [Wed, 14 Nov 2018 18:53:49 +0000 (19:53 +0100)] 
seccomp: add some missing syscalls to filter sets

5 years agoanalyze: show list of syscalls the kernel supports but not included in any group...
Lennart Poettering [Wed, 14 Nov 2018 18:40:45 +0000 (19:40 +0100)] 
analyze: show list of syscalls the kernel supports but not included in any group in the output

This is quite useful for making sure our list is complete.

5 years agococci: simplify some if checks
Lennart Poettering [Fri, 16 Nov 2018 13:42:14 +0000 (14:42 +0100)] 
cocci: simplify some if checks

5 years agologind: fix determination of CanHibernate() and other calls 10806/head
Henry Tung [Tue, 23 Oct 2018 12:09:13 +0000 (14:09 +0200)] 
logind: fix determination of CanHibernate() and other calls

As suggest here:

https://github.com/systemd/systemd/issues/10250#issuecomment-426788301

Fixes: #10250
(This suggested patch has been converted into a proper commit by me,
Lennart, with attribution to the original author, @henryptung)

5 years agologind: imply right to reboot when user has right to set reboot-to-firmware flag
Lennart Poettering [Mon, 22 Oct 2018 18:00:31 +0000 (20:00 +0200)] 
logind: imply right to reboot when user has right to set reboot-to-firmware flag

Let's reduce the number of actions the user has to acquire.

5 years agoefivars: let's add some validation of boot menu entry name syntax
Lennart Poettering [Mon, 22 Oct 2018 17:59:45 +0000 (19:59 +0200)] 
efivars: let's add some validation of boot menu entry name syntax

5 years agoefivars: add new helper efi_set_variable_string()
Lennart Poettering [Mon, 22 Oct 2018 17:58:26 +0000 (19:58 +0200)] 
efivars: add new helper efi_set_variable_string()

Let's make it easier to parse an UTF-16 string properly.

5 years agologind: log correct error msg
Lennart Poettering [Mon, 22 Oct 2018 16:06:18 +0000 (18:06 +0200)] 
logind: log correct error msg

5 years agologind: sort includes
Lennart Poettering [Mon, 22 Oct 2018 16:06:06 +0000 (18:06 +0200)] 
logind: sort includes

5 years agobootctl: display loader features in a pretty way.
Lennart Poettering [Mon, 22 Oct 2018 14:52:57 +0000 (16:52 +0200)] 
bootctl: display loader features in a pretty way.

5 years agoutil: force time multiplications to happen in longs
Lennart Poettering [Mon, 22 Oct 2018 14:51:29 +0000 (16:51 +0200)] 
util: force time multiplications to happen in longs

5 years agolocale-util: add special glyph support for check marks/cross marks
Lennart Poettering [Mon, 22 Oct 2018 14:37:10 +0000 (16:37 +0200)] 
locale-util: add special glyph support for check marks/cross marks

5 years agoefi: NUL terminate strings read with efivar_get() 10804/head
Lennart Poettering [Tue, 23 Oct 2018 11:44:23 +0000 (13:44 +0200)] 
efi: NUL terminate strings read with efivar_get()

let's be more careful and NUL terminate everything we read from EFI
variables, in case it isn't already.

5 years agosd-boot: make sure special menu items also work if menu is skipped
Lennart Poettering [Tue, 23 Oct 2018 11:43:10 +0000 (13:43 +0200)] 
sd-boot: make sure special menu items also work if menu is skipped

While it doesn't really make much sense to set "auto-reboot-to-firmware"
as oneshot boot item, let's still support it properly, by also
dispatching such a menu item if selected.

5 years agosd-boot: use structured initialization
Lennart Poettering [Tue, 23 Oct 2018 11:42:58 +0000 (13:42 +0200)] 
sd-boot: use structured initialization

5 years agosd-boot: change name of automatic entry for rebooting into firmware
Lennart Poettering [Mon, 22 Oct 2018 19:51:47 +0000 (21:51 +0200)] 
sd-boot: change name of automatic entry for rebooting into firmware

Let's stick to one nomenclature. In userspace we usually call this
"reboot to firmware setup", hence use the same name in sd-boot too.

This name was previously only relevant internally, but since the
addition of the LoaderEntries EFI var is exposed to userspace, hence
let's get this right with the first release adding this.

5 years agosd-boot: add new EFI variable exposing feature set of boot loader
Lennart Poettering [Mon, 22 Oct 2018 14:51:46 +0000 (16:51 +0200)] 
sd-boot: add new EFI variable exposing feature set of boot loader

We keep adding new features, let's advertise to the host OS what these
are in a new variable LoaderFeatures.

It works a bit like OsIndicationsSupported, but is about Loader
features.

5 years agoefi: make sure parse_boolean() does something useful on a NULL parameter
Lennart Poettering [Mon, 22 Oct 2018 14:45:32 +0000 (16:45 +0200)] 
efi: make sure parse_boolean() does something useful on a NULL parameter

5 years agoefi: add 'const' to various util.h API parameters
Lennart Poettering [Mon, 22 Oct 2018 14:43:53 +0000 (16:43 +0200)] 
efi: add 'const' to various util.h API parameters

5 years agosd-boot: introduce a one-time override for the boot menu timeout
Lennart Poettering [Mon, 22 Oct 2018 11:40:51 +0000 (13:40 +0200)] 
sd-boot: introduce a one-time override for the boot menu timeout

This is useful to allow userspace to request a "boot into boot menu"
feature.

5 years agosd-boot: when converting menu timeout for UINTN to INTN saturate
Lennart Poettering [Mon, 22 Oct 2018 11:35:06 +0000 (13:35 +0200)] 
sd-boot: when converting menu timeout for UINTN to INTN saturate

Let's be a bit more careful and handle overly long timeouts in a
slightly more sensible way.

5 years agoefi: add simple macros for MAX values of EFI's UINTN/INTN types
Lennart Poettering [Mon, 22 Oct 2018 11:32:59 +0000 (13:32 +0200)] 
efi: add simple macros for MAX values of EFI's UINTN/INTN types

5 years agosd-boot: timeout_sec is unsigned hence show it with %u
Lennart Poettering [Mon, 22 Oct 2018 11:31:41 +0000 (13:31 +0200)] 
sd-boot: timeout_sec is unsigned hence show it with %u

5 years agologind: also expose bool prop on bus that declares whether we are on external power
Lennart Poettering [Mon, 22 Oct 2018 11:02:50 +0000 (13:02 +0200)] 
logind: also expose bool prop on bus that declares whether we are on external power

The three core variables that affect idleness handling are whether we
are docked, whether we are on AC power and whether the lid is closed,
hence let's also expose the third variable on the bus, to make things
nicely debuggable.

5 years agologind: make sure that if we can't figure out whether we are on AC we presume we are
Lennart Poettering [Mon, 22 Oct 2018 11:02:07 +0000 (13:02 +0200)] 
logind: make sure that if we can't figure out whether we are on AC we presume we are

It's safer to assume that we are on AC than to assume we are on battery,
if we can't figure it out.

5 years agologind: don't claim that RebootToFirmwareSetup was constant
Lennart Poettering [Mon, 22 Oct 2018 10:47:44 +0000 (12:47 +0200)] 
logind: don't claim that RebootToFirmwareSetup was constant

It's not, after all, that's what SetRebootToFirmware() is about.

(I was wondering for a moment whether to make this EMITS_CHANGES, but
decided against it, given that the flag actually can be changed
externally to logind too, and we couldn't send out notifications for
that.)

5 years agologind: expose a boolean on the bus explaining whether logind thinks the lid is closed
Lennart Poettering [Mon, 22 Oct 2018 10:41:34 +0000 (12:41 +0200)] 
logind: expose a boolean on the bus explaining whether logind thinks the lid is closed

This copies how we already expose a "Docked" boolean the reflects
whether we are in a docking station or not.

5 years agojob: simplify status message extraction 10770/head
Lennart Poettering [Fri, 16 Nov 2018 14:28:26 +0000 (15:28 +0100)] 
job: simplify status message extraction

As @keszybz points out these fields are always here, there's no point in
checking if they are NULL or not.

5 years agojob: when a job was skipped due to a failed condition, log about it
Lennart Poettering [Wed, 14 Nov 2018 10:08:16 +0000 (11:08 +0100)] 
job: when a job was skipped due to a failed condition, log about it

Previously we'd neither show console status output nor log output. Let's
fix that, and still log something.

5 years agocore: make unit_start() return a distinguishable error code in case conditions didn...
Lennart Poettering [Wed, 14 Nov 2018 10:38:51 +0000 (11:38 +0100)] 
core: make unit_start() return a distinguishable error code in case conditions didn't hold

Ideally we'd even propagate this all the way to the client, by having a
separate JobType enum value for this. But it's hard to add this without
breaking compat, hence for now let's at least internally propagate this
case differently from the case "already on it".

This is then used to call job_finish_and_invalidate() slightly
differently, with the already= parameter false, as in the failed
condition case no message was likely produced so far.

5 years agojob: add two explanatory comments
Lennart Poettering [Wed, 14 Nov 2018 10:01:28 +0000 (11:01 +0100)] 
job: add two explanatory comments

5 years agojob: let's remove one comparison and reduce indentation level by one
Lennart Poettering [Wed, 14 Nov 2018 10:01:14 +0000 (11:01 +0100)] 
job: let's remove one comparison and reduce indentation level by one

5 years agocore: log a recognizable message when a unit succeeds, too
Lennart Poettering [Tue, 13 Nov 2018 22:28:09 +0000 (23:28 +0100)] 
core: log a recognizable message when a unit succeeds, too

We already are doing it on failure, let's do it on success, too.

Fixes: #10265
5 years agocore: make log messages about unit processes exiting recognizable
Lennart Poettering [Tue, 13 Nov 2018 21:10:38 +0000 (22:10 +0100)] 
core: make log messages about unit processes exiting recognizable

5 years agocore: make log messages about units entering a 'failed' state recognizable
Lennart Poettering [Tue, 13 Nov 2018 20:25:22 +0000 (21:25 +0100)] 
core: make log messages about units entering a 'failed' state recognizable

Let's make this recognizable, and carry result information in a
structure fashion.

5 years agosd-messages: add comment and reorder a bit
Lennart Poettering [Tue, 13 Nov 2018 19:40:38 +0000 (20:40 +0100)] 
sd-messages: add comment and reorder a bit

The message SD_MESSAGE_UNIT_FAILED is closely related to
SD_MESSAGE_UNIT_STARTED as it is generated when a start job failed
instead of completed successfully, Hence they should be placed together.

Otherwise one might get the impression that the message was about
failing units, which it really is not.

5 years agocatalog: update job begin/done messages
Lennart Poettering [Tue, 13 Nov 2018 19:36:51 +0000 (20:36 +0100)] 
catalog: update job begin/done messages

These texts have been slightly misleading previously, as they talked
about units, not jobs, but are actually generated for jobs, not units.
This difference matters as units can change state without a job
requesting that.

Also, the message be02cf6855d2428ba40df7e9d022f03d was particularly
wrong, as it claimed the unit failed, while it actually is the start job
that failed, which is a major difference, as jobs can fail without the
unit actually being placed in a failed state. Let's move this message a
bit up, closed to 39f53479d3a045ac8e11786248231fbf (i.e. the message
seen when a start job finished successfully).

5 years agojob: also include job ID in log messages when we begin with a job
Lennart Poettering [Tue, 13 Nov 2018 19:16:45 +0000 (20:16 +0100)] 
job: also include job ID in log messages when we begin with a job

5 years agocore: move unit_status_emit_starting_stopping_reloading() and related calls to job.c
Lennart Poettering [Tue, 13 Nov 2018 18:57:43 +0000 (19:57 +0100)] 
core: move unit_status_emit_starting_stopping_reloading() and related calls to job.c

This call is only used by job.c and very specific to job handling.
Moreover the very similar logic of job_emit_status_message() is already
in job.c.

Hence, let's clean this up, and move both sets of functions to job.c,
and rename them a bit so that they express precisely what they do:

1. unit_status_emit_starting_stopping_reloading() →
   job_emit_begin_status_message()
2. job_emit_status_message() → job_emit_done_status_message()

The first call is after all what we call when we begin with the
execution of a job, and the second call what we call when we are done
wiht it.

Just some moving and renaming, not other changes, and hence no change in
behaviour.

5 years agojob: add log message when we can't enable the job run event source
Lennart Poettering [Tue, 13 Nov 2018 18:40:02 +0000 (19:40 +0100)] 
job: add log message when we can't enable the job run event source

5 years agojob: add comment for EAGAIN job run case
Lennart Poettering [Tue, 13 Nov 2018 18:39:04 +0000 (19:39 +0100)] 
job: add comment for EAGAIN job run case

5 years agojob: minor coding style tweaks
Lennart Poettering [Tue, 13 Nov 2018 18:34:30 +0000 (19:34 +0100)] 
job: minor coding style tweaks

5 years agojob: include JOB_ID field in log message about jobs
Lennart Poettering [Tue, 13 Nov 2018 18:28:54 +0000 (19:28 +0100)] 
job: include JOB_ID field in log message about jobs

5 years agotests: run fuzzers four times in a row (#10794)
Evgeny Vereshchagin [Fri, 16 Nov 2018 14:09:24 +0000 (17:09 +0300)] 
tests: run fuzzers four times in a row (#10794)

This should help to catch issues that are easily detectable by
bad_build_check like the one being fixed in https://github.com/systemd/systemd/pull/10793,
which would totally break the build tomorrow if I hadn't run
`helper.py check_build` manually.

5 years agosd-ndisc: do not unref() event sources when update or disable them 10751/head
Yu Watanabe [Tue, 13 Nov 2018 05:40:02 +0000 (14:40 +0900)] 
sd-ndisc: do not unref() event sources when update or disable them

5 years agosd-ndisc: use structured initializer at one more place
Yu Watanabe [Tue, 13 Nov 2018 05:33:41 +0000 (14:33 +0900)] 
sd-ndisc: use structured initializer at one more place

5 years agosd-radv: do not unref() event sources when update or disable them
Yu Watanabe [Tue, 13 Nov 2018 05:32:12 +0000 (14:32 +0900)] 
sd-radv: do not unref() event sources when update or disable them

5 years agosd-radv: use structured initializer at one more place
Yu Watanabe [Tue, 13 Nov 2018 05:28:16 +0000 (14:28 +0900)] 
sd-radv: use structured initializer at one more place

5 years agolldp: do not unref() event sources when update or disable them
Yu Watanabe [Tue, 13 Nov 2018 05:26:33 +0000 (14:26 +0900)] 
lldp: do not unref() event sources when update or disable them

5 years agolldp: use structured initializer at one more place
Yu Watanabe [Tue, 13 Nov 2018 05:26:00 +0000 (14:26 +0900)] 
lldp: use structured initializer at one more place

5 years agoipv4acd: do not unref() event sources when update or disable them
Yu Watanabe [Tue, 13 Nov 2018 05:18:41 +0000 (14:18 +0900)] 
ipv4acd: do not unref() event sources when update or disable them

5 years agoipv4acd: use structured initializer at one more place
Yu Watanabe [Tue, 13 Nov 2018 05:17:20 +0000 (14:17 +0900)] 
ipv4acd: use structured initializer at one more place

5 years agodhcp6-client: do not unref() event sources when update or disable them
Yu Watanabe [Tue, 13 Nov 2018 05:05:07 +0000 (14:05 +0900)] 
dhcp6-client: do not unref() event sources when update or disable them

5 years agodhcp6-client: use structured initializer at one more place
Yu Watanabe [Tue, 13 Nov 2018 05:04:37 +0000 (14:04 +0900)] 
dhcp6-client: use structured initializer at one more place

5 years agosd-dhcp6: fix crash by unrefing event sources before re-adding them
Benjamin Berg [Thu, 15 Nov 2018 22:09:43 +0000 (07:09 +0900)] 
sd-dhcp6: fix crash by unrefing event sources before re-adding them

In certain cases the timeouts may not have been unref'ed before they
need to be re-added. Add the appropriate unref calls to ensure we don't
register the timeout multiple times.

This fixes possible cases where timeouts are triggered multiple times
and even on destroyed DHCPv6 clients.

https://gitlab.freedesktop.org/NetworkManager/NetworkManager/issues/73

Fixes #10749.

5 years agodhcp-client: do not unref() event sources when update or disable them
Yu Watanabe [Tue, 13 Nov 2018 04:50:08 +0000 (13:50 +0900)] 
dhcp-client: do not unref() event sources when update or disable them

5 years agodhcp-client: use structured initializer at one more place
Yu Watanabe [Tue, 13 Nov 2018 04:49:12 +0000 (13:49 +0900)] 
dhcp-client: use structured initializer at one more place

5 years agoudevd: use event_source_disable()
Yu Watanabe [Tue, 13 Nov 2018 04:37:17 +0000 (13:37 +0900)] 
udevd: use event_source_disable()

5 years agoudevd: use event_reset_time() to update kill_workers_event
Yu Watanabe [Tue, 13 Nov 2018 04:16:41 +0000 (13:16 +0900)] 
udevd: use event_reset_time() to update kill_workers_event

5 years agosd-event: also introduce event_source_is_enabled()
Yu Watanabe [Thu, 15 Nov 2018 22:02:35 +0000 (07:02 +0900)] 
sd-event: also introduce event_source_is_enabled()

5 years agosd-event: introduce event_source_disable()
Yu Watanabe [Tue, 13 Nov 2018 04:34:44 +0000 (13:34 +0900)] 
sd-event: introduce event_source_disable()

5 years agosd-event: introduce event_reset_time()
Yu Watanabe [Fri, 16 Nov 2018 08:32:20 +0000 (17:32 +0900)] 
sd-event: introduce event_reset_time()