summaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAge
...
* fsckd: make sure we free the connection event source before we close the ↵Lennart Poettering2015-03-09
| | | | connection fd
* fsckd: don't allow unbounded numbers of clientsLennart Poettering2015-03-09
|
* fsckd: free client event source before we close its fdLennart Poettering2015-03-09
|
* fsck: rename functions to reflect some object orientationLennart Poettering2015-03-09
| | | | | | | | Let's clean up the function naming scheme and put the object they operate on first in the name, the way OO programming usually does it. Also, let's make sure can properly destroy half-initialized Manager objects.
* fsck: don't read invalid dataLennart Poettering2015-03-09
|
* fsckd: internaly check if a client already was cancelledLennart Poettering2015-03-09
|
* fsck: simplify client destruction logicLennart Poettering2015-03-09
|
* fsckd: rework plymouth connection managementLennart Poettering2015-03-09
| | | | | | | | | | | - the even source should not be freed before the fd for it is closed - read() returns an ssize_t and we need to handle it as such - properly handle errors from read() - reuse on_plymouth_disconnect() whenever we disconnect from plymouth, and rename it plymouth_disconnect hence()
* fscd: fix error handlingLennart Poettering2015-03-09
|
* fsck: no need for a temporary variableLennart Poettering2015-03-09
|
* fsckd: fix error handling when sending cancel request to fsck clientLennart Poettering2015-03-09
|
* fsck: unify exit path for connect_plymouth()Lennart Poettering2015-03-09
|
* fsck: use only a single exit code ternary operatorLennart Poettering2015-03-09
|
* fsck: simplificationLennart Poettering2015-03-09
|
* fsckd: the error code is actually returned in 'fd'Lennart Poettering2015-03-09
| | | | Also, we don't use {} for single-line if-blocks.
* fsckd: simplify code a bitLennart Poettering2015-03-09
|
* fsckd: make use of safe_close()'s return valueLennart Poettering2015-03-09
|
* build-sys: add one more Makefile symlinkLennart Poettering2015-03-09
|
* importd: add API for exporting container/VM imagesLennart Poettering2015-03-09
| | | | Also, expose it in machinectl.
* udev: use inttypes.h types wherever appropriateLennart Poettering2015-03-09
|
* tree-wide: use _packed_ macro instead of raw gcc __attribute__Lennart Poettering2015-03-09
|
* udevd: close race in udev settleTom Gundersen2015-03-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | The udev-settle guarantees that udevd is no longer processing any of the events casued by udev-trigger. The way this works is that it sends a synchronous PING to udevd after udev-trigger has ran, and when that returns it knows that udevd has started processing the events from udev-trigger. udev-settle will then wait for the event queue to empty before returning. However, there was a race here, as we would only update the /run state at the beginning of the event loop, before reading out new events and before processing the ping. That means that if the first uevent arrived in the same event-loop iteration as the PING, we would return the ping before updating the queue state in /run (which would happen on the next iteration). The race window here is tiny (as the /run state would probably get updated before udev-settle got a chance to read /run), but still a possibility. Fix the problem by updating the /run state as the last step before returning the PING. We must still update it at the beginning of the loop as well, otherwise we risk being stuck in poll() with a stale state in /run. Reported-by: Daniel Drake <drake@endlessm.com>
* missing.h: add NDA_*Michael Olbrich2015-03-09
| | | | | This is necessary to build with older kernel headers. NDA_VLAN was introduced in v3.9 and NDA_PORT, NDA_VNI and NDA_IFINDEX in v3.10
* Fix typosTorstein Husebø2015-03-09
|
* sd-journal: return error when we cannot open a fileZbigniew Jędrzejewski-Szmek2015-03-08
| | | | | Lack of this caused journalctl not to display a hint about missing groups properly when the user lacks permissions.
* journalctl: update hint now that we set ACL everywhereZbigniew Jędrzejewski-Szmek2015-03-08
|
* bus: fix leak in error pathZbigniew Jędrzejewski-Szmek2015-03-07
| | | | CID #1271349.
* systemctl: remove dead checkZbigniew Jędrzejewski-Szmek2015-03-07
| | | | | | r could never be less than zero. CID #1271350.
* core/load-fragment: safe_close() protects errnoZbigniew Jędrzejewski-Szmek2015-03-07
|
* libsystemd-terminal: use at most LOG_ERR for XKB errorsZbigniew Jędrzejewski-Szmek2015-03-07
| | | | | | | XKB errors aren't *that* important. Coverity complained that the same action is taken in multiple branches, which is semi-valid, so is fixed too (CID #1256582).
* login: fix copy-pasto in error pathZbigniew Jędrzejewski-Szmek2015-03-07
| | | | CID #1256583.
* nspawn: fix use-after-free and leak in error pathsZbigniew Jędrzejewski-Szmek2015-03-07
| | | | CID #1257765.
* core/dbus-manager: remove dead checkZbigniew Jędrzejewski-Szmek2015-03-07
| | | | CID #1257766.
* bus-util: remove stray errno assignmentZbigniew Jędrzejewski-Szmek2015-03-07
|
* networkctl: avoid leak if a field was specified twiceZbigniew Jędrzejewski-Szmek2015-03-07
| | | | | | | The input data would have to be borked, so this is unlikely to happen, but since we have a nice helper function to do it properly... why not? CID #1261390.
* machine: do not rely on asprintf setting arg on errorZbigniew Jędrzejewski-Szmek2015-03-07
| | | | | Strictly speaking, the output variable is undefined if asprintf fails. We use the return value not the arg everywhere, and should we do here.
* v4l_id: use standard option parsing loopZbigniew Jędrzejewski-Szmek2015-03-07
| | | | | | | Not terribly important, but the loop wasn't an actual loop, making coverity unhappy. CID #1261725.
* shared/machine-pool: remove unnecessary checkZbigniew Jędrzejewski-Szmek2015-03-07
| | | | CID #128739.
* sysusers: do not reject users with already present /etc/shadow entriesIvan Shapovalov2015-03-07
| | | | | | This is needed to interoperate firstboot and sysusers. The former one is started first, and it writes only /etc/shadow when it is told to set the root password. It's better to relax checks here than to duplicate functionality in firstboot.
* firstboot: set all spwd fields to -1 for consistency with sysusersIvan Shapovalov2015-03-07
|
* core: do not spawn jobs or touch other units during coldpluggingIvan Shapovalov2015-03-07
| | | | | | | | | | | | | Because the order of coldplugging is not defined, we can reference a not-yet-coldplugged unit and read its state while it has not yet been set to a meaningful value. This way, already active units may get started again. We fix this by deferring such actions until all units have been at least somehow coldplugged. Fixes https://bugs.freedesktop.org/show_bug.cgi?id=88401
* adjust for time spent in timedated even without dbus timestampShawn Landden2015-03-07
| | | | | | | | it is trivial to fall back to our own timestamp v2: use now() v3: remove useless if () v4: add comment
* vconsole: match on vtcon events, not fbcon onesJan Engelhardt2015-03-06
| | | | | | | | | | | | | | | | I observe that upon loading of framebuffer drivers, I do not get the desired system font, but the kernel-level defaults (usually lib/fonts/font_8x16.c, but your mileage may vary depending on kernel config and boot options). The fbcon driver may be loaded at a time way before the first framebuffer device is active, such that the vconsole setup helper runs too early. The existing rule is non-fitting. The going live of the fbcon kernel component does not indicate the proper time at which to load the visuals, which really ought to be done when a new vtcon object comes into existence. (The font table is a per-vtcon property.)
* login: make hold-off timeout configurableDavid Herrmann2015-03-06
| | | | | | | | | | | | | | | | | This introduces 'HoldoffTimeoutSec' to logind.conf to make IGNORE_LID_SWITCH_{SUSPEND,STARTUP}_USEC configurable. Background: If an external monitor is connected, or if the system is docked, we want to ignore LID events. This is required to support setups where a laptop is used with external peripherals while the LID is closed. However, this requires us to probe all hot-plugged devices before reacting to LID events. But with modern buses like USB, the standards do not impose any timeout on the slots, so we have no chance to know whether a given slot is used or not. Hence, after resume and startup, we have to wait a fixed timeout to give the kernel a chance to probe devices. Our timeout has always been generous enough to support even the slowest devices. However, a lot of people didn't use these features and wanted to disable the hold-off timer. Now we provide a knob to do that.
* machinectl: minor --help text improvementsLennart Poettering2015-03-05
|
* tmpfiles: Fix handling of duplicate linesMartin Pitt2015-03-05
| | | | | | | | Commit 3f93da987 accidentally dropped the "return 0" after detection of a duplicate line. Put it back, to get back the documented and intended "first match wins" behaviour. https://launchpad.net/bugs/1428540
* nspawn: Map all seccomp filters to capabilitiesJay Faulkner2015-03-04
| | | | | | | | | | | | | | This change makes it so all seccomp filters are mapped to the appropriate capability and are only added if that capability was not requested when running the container. This unbreaks the remaining use cases broken by the addition of seccomp filters without respecting requested capabilities. Co-Authored-By: Clif Houck <me@clifhouck.com> [zj: - adapt to our coding style, make struct anonymous]
* build-sys: generate CLEANFILES from EXTRA_DISTZbigniew Jędrzejewski-Szmek2015-03-04
| | | | | | | | | Everything that is generated can be assumed to belong to CLEANFILES, which means that the original file has to be in EXTRA_DIST. Simplify the rules by generating as in $subject. We have less lists to adjust manually, and 'make clean' actually removes more stuff that before.
* Allow up to 4096 simultaneous connectionsHannes Reinecke2015-03-04
| | | | | | | | | On large system we hit the limit on 512 simultaneous dbus connections, resulting in tons of annoying messages: Too many concurrent connections, refusing This patch raises the limit to 4096.
* Remove the cap on epoll eventsHannes Reinecke2015-03-04
| | | | | | | Currently the code will silently blank out events if there are more then 512 epoll events, causing them never to be handled at all. This patch removes the cap on the number of events for epoll_wait, thereby avoiding this issue.