summaryrefslogtreecommitdiffstats
path: root/coccinelle/empty-if.cocci (unfollow)
Commit message (Collapse)AuthorFilesLines
2017-03-16check for _POSIX_C_SOURCE instead of __USE_POSIX*Matija Skala1-1/+1
check for _GNU_SOURCE as well as sd_event_child_handler_t needs to be defined correctly while compiling systemd
2017-03-16assigning stdout and stderr is not allowedMatija Skala1-2/+2
2017-03-16man: Fix a simple grammar error in systemd.service.xml (#5594)AlexanderKurtz1-1/+1
2017-03-15fix includesMatija Skala2-1/+1
linux/sockios.h is needed for the SIOCGSTAMPNS macro xlocale.h is included indirectly in glibc and doesn't even exist in other libcs
2017-03-15build-sys: make RPM macros installation path configurable (#5564)steelman2-1/+13
--with-rpmmacrosdir=no disables installation of the macros.
2017-03-14man: Document when pam_systemd sets XDG_RUNTIME_DIR (#5570)Adrian Heine né Lang1-2/+3
https://github.com/systemd/systemd/blob/f97b34a6/src/login/pam_systemd.c#L439
2017-03-14machinectl: don't return 1 in case we couldn't figure out container IP ↵Michal Sekletar1-0/+1
addresses (#5587) This is in spirit very similar to commit 4b2419165ce409ee55ce96a926302f89685f2293. Fixes: #5581
2017-03-14Never call unmap with MAP_FAILED. (#5590)Tobias Stoeckmann1-2/+2
When mmap is called, the code in correctly checks for p == MAP_FAILED. But the resource cleanup at the end of busname_peek_message checks for p == NULL, and if that's not true, munmap is called. Therefore in error case, munmap is called with a MAP_FAILED argument which can result in unexpected behaviour depending on sz's value. Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>
2017-03-13journal: prevent integer overflow while validating header (#5569)Tobias Stoeckmann1-3/+9
It is possible to overflow uint64_t while validating the header of a journal file. To prevent this, the addition itself is checked to be within the limits of UINT64_MAX first. To keep this readable, I have introduced two stack variables which hold the converted values during validation.
2017-03-13headers: check that __INCLUDE_LEVEL__ is defined before using it (#5575)Zbigniew Jędrzejewski-Szmek1-2/+2
That macro is a gcc extension, and while widely supported, not ubiquitous. In particular the coverity scanner is having trouble with it.
2017-03-13README: include small graphs of open issues and pull requests (#5576)Zbigniew Jędrzejewski-Szmek1-0/+2
2017-03-13resolve: add missing space in output message (#5574)Yu Watanabe1-2/+2
2017-03-12build-sys: add helper command to sign tarballsZbigniew Jędrzejewski-Szmek1-0/+3
2017-03-12build-sys: simplify git tar invocationZbigniew Jędrzejewski-Szmek1-1/+1
git is nowadays nice enough to compress based on the output file suffix, let's make use of that.
2017-03-12build-sys: make more targets .PHONYZbigniew Jędrzejewski-Szmek1-0/+3
2017-03-10Fix typo in function name (#5565)Michal Sekletar2-3/+3
2017-03-09hwdb: Add new quirk type to documentationBastien Nocera1-1/+2
Missed in b698b5c.
2017-03-09hwdb: Add property for keyboards without LEDsBastien Nocera2-0/+15
It is useful for desktop environments to be able to show Caps-Lock or Num-Lock status changes as an on-screen display when using a keyboard that doesn't have LEDs for this. Closes: #5559
2017-03-09hwdb: Better documentation for keyboard quirksBastien Nocera1-5/+21
Split up the device matching explanation, and add documentation for the fixed layout quirk.
2017-03-08build-sys: only install legacy *.pkla files with old polkit versions (#5555)Martin Pitt2-1/+9
The legacy *.pkla files are not required when running against polkit ≥ 106, and we want to avoid shipping files in /var if possible (but pkla files can only be in /etc/ or /var). Only install the *.pkla files if we detect an old polkit version during configure. Don't install them if polkit isn't installed during build, as distributions other than Debian-based ones have moved to the new polkit long ago. Fixes #5523
2017-03-08man: fix typo (#5556)Torstein Husebø1-1/+1
2017-03-07docs: use https in the mailing list urlAsciiWolf1-1/+1
2017-03-07docs: add a note about reporting security vulns (#5541)Zbigniew Jędrzejewski-Szmek1-0/+4
We *do* have the occasional security issue, where it would be nice to have non-public disclosure and time to fix the issue before it's fully public. Our github infrastracture does not make it easy to report vulnerabilities in confidential manner, so let's leverage the distro mechanisms for that. I think we're better off with this solution than leaving it up to individual reporters to discover some mechanism on their own.
2017-03-07rules: set ID_BUS=bluetooth for any device with id/bustype attr of 0x0005 ↵Peter Hutterer1-0/+2
(#5539) Not all bluetooth devices come through the bluetooth subsystem and those that don't currently lack the ID_BUS=bluetooth env. This again fails to apply udev rules and/or hwdb entries that rely on the bluetooth bustype to be set. Fix this by checking the attribute id/bustype on the device instead of just the subsystem. Fixes #4566
2017-03-07sd-bus: drop kdbus-related docs (#5533)AsciiWolf4-577/+9
2017-03-07cgtop: use PRIu64 to print uint64_t (#5544)Thomas H. P. Andersen1-1/+1
Commit 59f448cf replaced usage of off_t with uint64_t. Change the format string to use PRIu64 to match it.
2017-03-05coredump: fix handling of premature-eof data for --backtraceZbigniew Jędrzejewski-Szmek2-2/+3
We'd fail with an assert in journal_importer_process_data(), because that function requires the caller to handle EOF themselves.
2017-03-05import: bump image size safety limit for machinectl pull (#5535)Dax Kelson1-1/+1
We currenly use 40GB images in our environment
2017-03-03hwdb: quirks for accelerometers in HP laptopsThomas Hindoe Paaboel Andersen1-0/+7
This patch adds quirks for the two laptops I could test on (8540w and 8560w). The accelerometer is configured in the kernel to report values according to the base of the laptop. We want the values according to the screen instead. It is likely (but untested) to match all HP laptops with the lis3lv02d accelerometer on this list: https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/drivers/platform/x86/hp_accel.c#n207
2017-03-03rules: allow quirks for platform input accelerometersThomas Hindoe Paaboel Andersen1-0/+4
2017-03-03test: skip instead of fail if crypto kmods are not availableMartin Pitt2-6/+19
Package build machines may have module loading disabled, thus AF_ALG sockets are not available. Skip the tests that cover those (khash and id128) instead of failing them in this case. Fixes #5524
2017-03-02Avoid strict DM interface version dependencies (#5519)Michael Biebl5-3/+359
Compiling against the dm-ioctl.h header as provided by the Linux kernel will embed the DM interface version number. Running an older kernel can result in an error like this on shutdown: Could not detach DM dm-11: ioctl mismatch, kernel(4.34.4), user(4.35.4) Work around this by shipping a local copy of dm-ioctl.h. We need at least the version from 3.13 for DM_DEFERRED_REMOVE [1], so bump the requirements in README accordingly. [1] https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=2c140a246dc0bc085b98eddde978060fcec1080c Fixes: #5492
2017-03-02build-sys: do not require everything to be rebuild to push man pages (#5521)Zbigniew Jędrzejewski-Szmek1-1/+1
When the doc-sync target was created, we didn't have man target yet.
2017-03-02basic/architecture: Add sub-architecture types for SuperHJohn Paul Adrian Glaubitz1-1/+17
On SuperH, there are multiple sub-architectures defined with different values for LIB_ARCH_TUPLE. The different sub- architectures can be detected by checking whether __SH1__, __SH2__, __SH3__, __SH4__, and so on are defined.
2017-03-02basic/architecture: Properly set LIB_ARCH_TUPLE for powerpcspeJohn Paul Adrian Glaubitz1-1/+5
On powerpc, there are two possible ABIs and hence values for LIB_ARCH_TUPLE. The convential type with an FPU and the embedded variant, called powerpcspe, which does not have a convential FPU but a special Signal Processing Engine (SPE). The latter can be detected by checking whether __NO_FPRS__ is defined.
2017-03-02basic/architecture: Properly set LIB_ARCH_TUPLE for x32John Paul Adrian Glaubitz1-1/+5
On x86_64, there are two possible ABIs and hence values for LIB_ARCH_TUPLE. The convential 64-bit type and the 32-bit variant, called x32. The latter can be detected by checking whether __ILP32__ is defined.
2017-03-02Revert "environment.d: fix date"Zbigniew Jędrzejewski-Szmek1-1/+1
This reverts commit 46987593ccb1c78f3889ea91a0c94f9cc810d43c. Pushed to the wrong repo, sorry.
2017-03-02environment.d: fix dateZbigniew Jędrzejewski-Szmek1-1/+1
2017-03-02NEWS: fix word (#5514)Lucas Werkmeister1-1/+1
2017-03-02hwdb: Endless ELT-NL3 accelerometer supportDaniel Drake1-0/+6
The Endless ELT-NL3 laptop has an accelerometer but its values are not being interpreted correctly, meaning that it's very easy to end up with the screen rotated inappropriately. Add an appropriate mount matrix to correct the values. P: /devices/LNXSYSTM:00/LNXSYBUS:00/ACCE0001:00 E: DEVPATH=/devices/LNXSYSTM:00/LNXSYBUS:00/ACCE0001:00 E: DRIVER=cmpc_accel_v4 E: ID_VENDOR_FROM_DATABASE=Accton Technology Corporation E: MODALIAS=acpi:ACCE0001: E: SUBSYSTEM=acpi P: /devices/LNXSYSTM:00/LNXSYBUS:00/ACCE0001:00/input/input8 E: ABS=7 E: ACCEL_MOUNT_MATRIX=0, 1, 0; 0, 0, -1; -1, 0, 0 E: DEVPATH=/devices/LNXSYSTM:00/LNXSYBUS:00/ACCE0001:00/input/input8 E: EV=9 E: ID_FOR_SEAT=input-acpi-ACCE0001_00 E: ID_INPUT=1 E: ID_INPUT_ACCELEROMETER=1 E: ID_PATH=acpi-ACCE0001:00 E: ID_PATH_TAG=acpi-ACCE0001_00 E: IIO_SENSOR_PROXY_TYPE=input-accel E: MODALIAS=input:b0000v0000p0000e0000-e0,3,kra0,1,2,mlsfw E: NAME="cmpc_accel_v4" E: PRODUCT=0/0/0/0 E: PROP=0 E: SUBSYSTEM=input E: SYSTEMD_WANTS=iio-sensor-proxy.service E: TAGS=:seat:systemd: P: /devices/LNXSYSTM:00/LNXSYBUS:00/ACCE0001:00/input/input8/event7 N: input/event7 E: ACCEL_MOUNT_MATRIX=0, 1, 0; 0, 0, -1; -1, 0, 0 E: DEVNAME=/dev/input/event7 E: DEVPATH=/devices/LNXSYSTM:00/LNXSYBUS:00/ACCE0001:00/input/input8/event7 E: ID_INPUT=1 E: ID_INPUT_ACCELEROMETER=1 E: ID_PATH=acpi-ACCE0001:00 E: ID_PATH_TAG=acpi-ACCE0001_00 E: IIO_SENSOR_PROXY_TYPE=input-accel E: MAJOR=13 E: MINOR=71 E: SUBSYSTEM=input E: SYSTEMD_WANTS=iio-sensor-proxy.service E: TAGS=:power-switch:systemd:
2017-03-02udev: Allow quirks for ACPI input accelerometersDaniel Drake1-0/+4
The existing accelerometer rules only support IIO devices, however iio-sensor-proxy can also work with accelerometers made available through the input (evdev) subsystem. In this case I am working with an accelerometer input device backed by an ACPI driver for which the hierarchy is: - ACCE0001 (ACPI device) -> input8 -> event7 We want the mount matrix (from hwdb) to be applied to both input8 and event7. However, to match in 60-sensor.hwdb, we need to be working with the modalias of the parent device (ACCE0001), and it is tricky to access that when processing the input8 device which has it's own modalias. Instead of working directly with modalias, this ACPI-specific rule uses the "hid" attribute to reconstruct the ACPI modalias. Since input and event devices do not provide a hid attribute we will always get this from the ACPI parent. The modalias is constructed according to the definition in the kernel's Documentation/acpi/namespace.txt and create_pnp_modalias(). We will only use the first _CID/_HID value available, i.e. in some cases we will only reconstruct the first part of the modalias, but that should be enough granularity for our needs.
2017-03-01NEWS: 'systemd' is always spelt with a lowercase 's'v233Lennart Poettering1-1/+1
2017-03-01NEWS: add note about 'make install-tests' (#5512)Zbigniew Jędrzejewski-Szmek1-0/+4
2017-03-01NEWS: reorder entries by subject, fix some typos and descriptions (#5511)Zbigniew Jędrzejewski-Szmek1-107/+105
This doesn't add anything major, but moves some stuff around. In particular changes which might require updates to the build environment (new kernel requirements, cgroup stuff, dbus, etc) are moved to the top, where it's most likely that people will read them. In particular cgroup hierarchy changes are moved to the top because they're most likely to be problematic. Various items are grouped by subject where it's easy. The description of list-jobs --after/--before was reversed.
2017-03-01udev: Use parent bus id for virtio disk builtin path-id (#5500)Viktor Mihajlovski2-4/+5
The builtin path id for virtio block devices has been changed to use the bus id without a prefix "virtio-pci" to be compatible with all virtio transport types. In order to not break existing setups, the by-path symlinks for virtio block devices on the PCI bus are reintroduced by udev rules. The virtio-pci symlinks are considered to be deprecated and should be replaced by the native PCI symlinks. Example output for a virtio disk in PCI slot 7: $ ls /dev/disk/by-path pci-0000:00:07.0 pci-0000:00:07.0-part1 virtio-pci-0000:00:07.0 virtio-pci-0000:00:07.0-part1 See also [1] https://lists.freedesktop.org/archives/systemd-devel/2017-February/038326.html [2] https://lists.freedesktop.org/archives/systemd-devel/2017-March/038397.html This reverts f073b1b but keeps the same symlinks for compatibility.
2017-03-01man: escape: fix --suffix examples (#5508)Lucas Werkmeister1-2/+2
systemd-escape does not permit the suffix to begin with a dot.
2017-03-01Revert "util: Fine tune running_in_chroot() a bit" (#5506)Lennart Poettering1-20/+5
This reverts commit 295ee9845c801300298d01256eb5a9e3ff117ae0. Let'd revert this for now, see #5446 for discussions. We want systemd-detect-virt --chroot to return true for all chroot-like stuff, for example mock environments which have use a mount namespace. The downside of this revert that systemctl will not work from our own namespaced services, anything with RootDirectory=/RootImage= set.
2017-03-01man: systemd: fix description of SIGRTMIN+26,27,28 (#5505)Lucas Werkmeister1-1/+1
2017-03-01coredump: define a macro for a "short bus call timeout"Lennart Poettering1-1/+3
I think it would be a good idea to move such fixed, picked values out of the main sources into the head of a file, to make sure they are ultimately tunables.
2017-03-01coredump: introduce is_journald_crash() and is_pid1_crash() helpersLennart Poettering1-6/+17
We check these a number of times, hence let's unify these checks here. This also allows us to make the PID 1 check more elaborate as we can check both the PID and the cgroup. Checking the PID has the benefit that we'll also cover cases where PID 1 might still be in the root cgroup, and the cgroup check has the benefit that we also cover crashes in forked off crasher processes (the way we actually do it in systemd)