summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* tree-wide: remove spurious spaceThomas Hindoe Paaboel Andersen2015-06-0811-20/+20
|
* Merge pull request #106 from teg/man-mac-randomLennart Poettering2015-06-081-1/+3
|\ | | | | man: systemd.link - explain random MAC addresses
| * man: systemd.link - explain random MAC addressesTom Gundersen2015-06-081-1/+3
| | | | | | | | | | Two of the bits in the MAC address are set unconditioanlly, and the rest is randomized, make this clear in the documentation (as it currently read as if it was all random).
* | Merge pull request #101 from zonque/kmodLennart Poettering2015-06-081-9/+11
|\ \ | |/ |/| kmod-setup: split warn flags
| * kmod-setup: split warn flagsDaniel Mack2015-06-081-9/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Traditionally, we used to warn about ipv6 being a module or being unavailable. This was changed in b4aa82f16 ("kmod-setup: don't warn when ipv6 can't be loaded") in a way that neither of the two conditions will cause a log message. Now, while running a setup without any IPv6 is completely fine and shouldn't cause any warning, we should still warn about ipv6 being a module instead of built-in. To achieve this, split the boolean warn flag into two: one for a feature not being built-in but shipped as a module, and one to print an error when a module is entirely unavailable. We will, however, still warn if kmod returns anything else than -ENOENT in the attempt of loading the module, and at the very least, turn the message into a debug log.
* | Merge pull request #96 from haraldh/set_consumeLennart Poettering2015-06-081-6/+11
|\ \ | |/ |/| util:bind_remount_recursive() fix "use after free"
| * util:bind_remount_recursive() fix "use after free"Harald Hoyer2015-06-081-6/+11
|/ | | | | | | | set_consume(done, x) consumes x with free(x) but mount(…, x, …) uses it afterwards. coverity CID 1299006
* Merge pull request #92 from zonque/READMELennart Poettering2015-06-081-1/+2
|\ | | | | README.md: add Coverity scan status badge
| * README.md: add Coverity scan status badgeDaniel Mack2015-06-081-1/+2
|/
* Merge pull request #79 from zonque/fdo-87475Kay Sievers2015-06-051-1/+1
|\ | | | | kmod-setup: don't warn when ipv6 can't be loaded (FDO bug #87475)
| * kmod-setup: don't warn when ipv6 can't be loadedDaniel Mack2015-06-051-1/+1
| | | | | | | | | | | | | | | | | | Not having IPv6 is a valid setup. Let's not print a warning in that case. Addresses: https://bugs.freedesktop.org/show_bug.cgi?id=87475
* | Merge pull request #82 from gmacario/bootchart-fix-typoDaniel Mack2015-06-051-1/+1
|\ \ | | | | | | systemd-bootchart: Trivial typo fix in warning
| * | systemd-bootchart: Trivial typo fix in warningGianpaolo Macario2015-06-051-1/+1
|/ / | | | | Signed-off-by: Gianpaolo Macario <gmacario@gmail.com>
* | Merge pull request #80 from zonque/README.mdHarald Hoyer2015-06-051-0/+8
|\ \ | | | | | | Add README.md
| * | Add README.mdDaniel Mack2015-06-051-0/+8
| |/ | | | | | | | | | | A README.md allows us to include a badge for the current build status of Semaphore CI. Keep other information in this file minimal, and avoid information duplication but point readers to the official README.
* | bus: don't force send-masks on kdbus busesDavid Herrmann2015-06-051-9/+0
| | | | | | | | | | | | | | | | | | | | Right now we always pass KDBUS_ITEM_ATTACH_FLAGS_RECV to KDBUS_CMD_BUS_MAKE, effectively forcing every bus connection to do the same during KDBUS_CMD_HELLO. This used to be a workaround to make sure all metadata is always present. However, we refrained from that approach and intend to make all metadata collection solely rely on /proc access restrictions. Therefore, there is no need to force the send-flags mask on newly created buses.
* | logind: Fix user_elect_display() to be more stablePhilip Withnall2015-06-051-36/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The previous implementation of user_elect_display() could easily end up overwriting the user’s valid graphical session with a new TTY session. For example, consider the situation where there is one session: c1, type = SESSION_X11, !stopping, class = SESSION_USER it is initially elected as the user’s display (i.e. u->display = c1). If another session is started, on a different VT, the sessions_by_user list becomes: c1, type = SESSION_X11, !stopping, class = SESSION_USER c2, type = SESSION_TTY, !stopping, class = SESSION_USER In the previous code, graphical = c1 and text = c2, as expected. However, neither graphical nor text fulfil the conditions for setting u->display = graphical (because neither is better than u->display), so the code falls through to check the text variable. The conditions for this match, as u->display->type != SESSION_TTY (it’s actually SESSION_X11). Hence u->display is set to c2, which is incorrect, because session c1 is still valid. Refactor user_elect_display() to use a more explicit filter and pre-order comparison over the sessions. This can be demonstrated to be stable and only ever ‘upgrade’ the session to a more graphical one. https://bugs.freedesktop.org/show_bug.cgi?id=90769
* | Merge pull request #63 from arvidjaar/issue/50Daniel Mack2015-06-051-2/+8
|\ \ | | | | | | fstab-generator: cescape device name in root-fsck service
| * | fstab-generator: cescape device name in root-fsck serviceAndrei Borzenkov2015-06-031-2/+8
| | | | | | | | | | | | | | | | | | | | | We unescape ExecStart line when parsing it, so escape device name before adding it to unit file. fixes #50
* | | Merge pull request #76 from crrodriguez/masterDaniel Mack2015-06-051-6/+2
|\ \ \ | |_|/ |/| | core: Remove "old kernel" warning if PR_SET_CHILD_SUBREAPER fails
| * | core: Remove "old kernel" warning if PR_SET_CHILD_SUBREAPER failsCristian Rodríguez2015-06-051-6/+2
|/ / | | | | | | This made sense when systemd ran on older kernels, nowdays not so much.
* | Merge pull request #74 from ↵Daniel Mack2015-06-050-0/+0
|\ \ | | | | | | | | | | | | systemd-mailing-devs/1432753344-31461-1-git-send-email-jonathan.boulle@coreos.com fix extraneous space in equality check
| * | fix extraneous space in equality checkJonathan Boulle2015-05-271-1/+1
| | |
* | | bus: update kdbus.hDavid Herrmann2015-06-041-4/+4
| | | | | | | | | | | | Sync with upstream.
* | | Merge pull request #72 from teg/event-fork-unrefDavid Herrmann2015-06-041-0/+6
|\ \ \ | | | | | | | | sd-event: don't touch fd's accross forks
| * | | sd-event: don't touch fd's accross forksTom Gundersen2015-06-041-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We protect most of the API from use accross forks, but we still allow both sd_event and sd_event_source objects to be unref'ed. This would cause problems as it would unregister sources from the underlying eventfd, hence also affecting the original instance in the parent process. This fixes the issue by not touching the fds on unref when done accross a fork, but still free the memory. This fixes a regression introduced by "udevd: move main-loop to sd-event": 693d371d30fee where the worker processes were disabling the inotify event source in the main daemon.
* | | | Merge pull request #73 from zonque/mountinfoDavid Herrmann2015-06-041-0/+3
|\ \ \ \ | | | | | | | | | | core/mount: skip incomplete mountinfo entries
| * | | | core/mount: skip incomplete mountinfo entriesDaniel Mack2015-06-041-0/+3
| |/ / / | | | | | | | | | | | | | | | | | | | | Skip /proc/mountinfo entries for which libmount returns a NULL pointer for 'source' or 'target'. This happened on Semaphore CI's build servers when the test suite is run.
* | | | Merge pull request #71 from kloun/patch-1Martin Pitt2015-06-041-3/+3
|\ \ \ \ | | | | | | | | | | small fix ru translation
| * | | | small fix ru translationkloun2015-06-041-3/+3
| | | | |
* | | | | Merge pull request #58 from pwithnall/wip/pwithnall/user-active-on-vt-switchDavid Herrmann2015-06-041-0/+1
|\ \ \ \ \ | |/ / / / |/| | | | logind: Save the user’s state when a session enters SESSION_ACTIVE
| * | | | logind: Save the user’s state when a session enters SESSION_ACTIVEPhilip Withnall2015-06-021-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When (for example) switching from X11 to a new VT and logging in there, creating a new session, the user state file (/run/systemd/users/$uid) is not updated after the session becomes active. The latest time it is saved is when the session is in SESSION_OPENING. This results in a /run/systemd/users/$uid file which contains STATE=online for the current user on the current active VT, which is obviously wrong. As functions like sd_uid_get_state() use this file to get the user’s state, this could result in things like PolicyKit making incorrect decisions about the user’s state. (See https://bugs.freedesktop.org/show_bug.cgi?id=76358.) Fix this by re-saving the state for a session’s user after completing the state_job for that session. https://bugs.freedesktop.org/show_bug.cgi?id=90818
* | | | | Merge pull request #57 from pwithnall/wip/pwithnall/udev-virtualbox-rulesDavid Herrmann2015-06-041-0/+1
|\ \ \ \ \ | | | | | | | | | | | | logind: Add a udev rule to tag all DRM cards with master-of-seat
| * | | | | logind: Add a udev rule to tag all DRM cards with master-of-seatPhilip Withnall2015-06-021-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is needed for generic DRM devices like the VirtualBox vboxvideo driver, which exposes itself as a generic, ID-less DRM device at /dev/dri/card0 (after applying this commit): $ udevadm info --query=all --path \ /sys/devices/pci0000:00/0000:00:02.0/drm/card0 P: /devices/pci0000:00/0000:00:02.0/drm/card0 N: dri/card0 E: DEVNAME=/dev/dri/card0 E: DEVPATH=/devices/pci0000:00/0000:00:02.0/drm/card0 E: DEVTYPE=drm_minor E: ID_FOR_SEAT=drm-pci-0000_00_02_0 E: ID_PATH=pci-0000:00:02.0 E: ID_PATH_TAG=pci-0000_00_02_0 E: MAJOR=226 E: MINOR=0 E: SUBSYSTEM=drm E: TAGS=:master-of-seat:seat:uaccess: E: USEC_INITIALIZED=59893 Without this patch, the capabilities for a seat on a VirtualBox installation of systemd v219 incorrectly show it as non-graphical, even though I can type these commands from an xterm: $ loginctl show-seat seat0 Id=seat0 CanMultiSession=yes CanTTY=yes CanGraphical=no … https://bugs.freedesktop.org/show_bug.cgi?id=90822
* | | | | | Partially revert "ma-setup: simplify"Zbigniew Jędrzejewski-Szmek2015-06-041-2/+13
| |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | copy_bytes() tries to do the write in chunks, but ima kernel code needs every rule to be written in one write. Writing the whole file at once avoids the issue. http://lists.freedesktop.org/archives/systemd-devel/2015-June/032623.html http://sourceforge.net/p/linux-ima/mailman/message/34145236/ https://bugzilla.redhat.com/show_bug.cgi?id=1226948
* | | | | Merge pull request #68 from whot/hwdb-updatesMartin Pitt2015-06-041-0/+10
|\ \ \ \ \ | | | | | | | | | | | | hwdb: add Apple MagicMouse entry
| * | | | | hwdb: add Apple MagicMouse entryPeter Hutterer2015-06-041-0/+10
|/ / / / /
* | | | | Merge pull request #59 from eswierk/masterKay Sievers2015-06-031-1/+1
|\ \ \ \ \ | | | | | | | | | | | | Add /dev/xvd* to 60-persistent-storage whitelist
| * | | | | Add /dev/xvd* to 60-persistent-storage whitelistEd Swierk2015-06-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Without this, systemd-udevd does not create persistent storage symlinks for xen block devices.
* | | | | | Merge pull request #65 from teg/enumerate-accept-nullKay Sievers2015-06-032-8/+31
|\ \ \ \ \ \ | | | | | | | | | | | | | | libudev: enumerate - accept NULL parameters in add_match()
| * | | | | | libudev: enumerate - accept NULL parameters in add_match()Tom Gundersen2015-06-032-8/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This was a regression introduced when moving to sd-device.
* | | | | | | Merge pull request #52 from mbiebl/systemctl-edit-default-editorTom Gundersen2015-06-032-1/+2
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | systemctl: Use /usr/bin/editor if available
| * | | | | | | systemctl: Use /usr/bin/editor if availableMichael Biebl2015-06-032-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the EDITOR environment variable is not set, the Debian policy recommends to use the /usr/bin/editor program as default editor. This file is managed via the dpkg alternatives mechanism and typically used in Debian/Ubuntu and derivatives to configure the default editor. See section 11.4 of the Debian policy [1]. Therefor prefer /usr/bin/editor over specific editors if available. [1] https://www.debian.org/doc/debian-policy/ch-customized-programs.html
* | | | | | | | kdbus: remove attach_flags_mask module parameter settingKay Sievers2015-06-034-33/+0
| |/ / / / / / |/| | | | | |
* | | | | | | test-util: fix a memleakThomas Hindoe Paaboel Andersen2015-06-031-0/+2
| |_|_|_|_|/ |/| | | | |
* | | | | | Merge pull request #60 from Keruspe/masterDaniel Mack2015-06-032-2/+0
|\ \ \ \ \ \ | | | | | | | | | | | | | | build-sys: drop references to gobject-introspection
| * | | | | | build-sys: drop references to gobject-introspectionMarc-Antoine Perennou2015-06-032-2/+0
| | |/ / / / | |/| | | | | | | | | | | | | | | | It has been removed alongside gudev
* | | | | | Merge pull request #55 from filbranden/rootprefix_empty1Daniel Mack2015-06-032-1/+8
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix --with-rootprefix= (empty) with a workaround for now.
| * | | | | | build-sys: Work around --with-rootprefix= (empty) not producing /Filipe Brandenburger2015-06-031-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since we introduced AX_NORMALIZE_PATH, using --with-rootprefix=/ does produce an empty string, but using --with-rootprefix= (empty) now produces "." instead which is wrong. Work around it until we can find a better solution for AX_NORMALIZE_PATH upstream at autoconf-archive. Bug: https://github.com/systemd/systemd/issues/54
| * | | | | | build-sys: Recommend --with-rootprefix=/ for split-usrFilipe Brandenburger2015-06-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since we started using AX_NORMALIZE_PATH, that is a valid supported setup and is more explicit than --with-rootprefix= (empty) which is actually currently broken. Let's advocate for it in the ./configure suggestion from autogen.sh.