summaryrefslogtreecommitdiffstats
path: root/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | sd-event: use LIST_IS_EMPTY()Yu Watanabe2022-06-271-3/+3
| |/ / / /
* | | | | gpt: add PARISC UUIDsSam James2022-06-282-0/+7
| | | | | | | | | | | | | | | | | | | | Not doing PARISC64 for now as no userland exists for it yet.
* | | | | seccomp: add PARISC (HPPA support)Sam James2022-06-282-2/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We have to skip the W^X protections as we need executable memory on PARISC for now. Kernel work is in progress (started w/ 5.18). Closes: https://github.com/systemd/systemd/issues/23180
* | | | | basic/missing-syscalls: add PARISC (HPPA support)Sam James2022-06-287-4/+643
|/ / / / | | | | | | | | | | | | Bug: https://github.com/systemd/systemd/issues/23180
* | | | network: grouping elements in network_free()Yu Watanabe2022-06-271-33/+46
| | | |
* | | | Merge pull request #23842 from medhefgo/boot-stdYu Watanabe2022-06-2745-1156/+1098
|\ \ \ \ | | | | | | | | | | boot: Use standard types
| * | | | boot: Use stdboolJan Janssen2022-06-2718-134/+137
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The way the UEFI spec defines BOOLEAN is fully compatible to stdbool, so it is perfectly safe to switch to it. Although any other values than 0/1 are undefined by the spec, we could theoretically have cases where a sloppy firmware hands us a bad BOOLEAN (since gnu-efi/edk2 declare it as uint8_t). So any uses where we pass a pointer to BOOLEAN are left untouched.
| * | | | boot: Use charJan Janssen2022-06-2715-182/+171
| | | | | | | | | | | | | | | | | | | | This also switches to _cleanup_free_. Otherwise no code changes.
| * | | | boot: Use char16_tJan Janssen2022-06-2721-199/+217
| | | | | | | | | | | | | | | | | | | | This also switches to _cleanup_free_. Otherwise no code changes.
| * | | | boot: Use stdint typesJan Janssen2022-06-2723-402/+391
| | | | |
| * | | | boot: Remove use of EFI_ERRORJan Janssen2022-06-2718-187/+183
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The macro is ugly and annoying to use and provides no real benefit. The only reason to use it would be to allow warnings to go through. But any EFI APIs we call do not return warning status codes or we do not check the return value anyway. The only other case would be BS->StartImage, where we already treat anything other than EFI_SUCCESS as an error anyway. This also helps the compiler and code analyzers to better reason about the code. In particular, this can help reduce use of uninitialized variable warnings.
| * | | | boot: Rename remaining EFI_STATUS vars to err for consistencyJan Janssen2022-06-272-29/+29
| | | | |
| * | | | fundamental: Remove types-fundamental.hJan Janssen2022-06-279-83/+30
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This removes the fundamental typedefs in favor of just using standard C types. These are all used internally anyway and also do not do anything special to warrant any redefinition to EFI types. Even for BOOLEAN we can safely use stdbool. The defition from the EFI specification is fully compatible, including making any other values than 0/1 as undefined. The exception is sd_char as those need to be char16_t. The typedef is moved to string-util-fundamental.h instead.
* / | | test-sd-hwdb: adjust the test to actually do anythingZbigniew Jędrzejewski-Szmek2022-06-271-1/+1
|/ / / | | | | | | | | | | | | Without the terminating colon we wouldn't match anything, so the loop over properties was skipped.
* | | sha256: Use stdbool and uintptr_tJan Janssen2022-06-242-19/+6
| | | | | | | | | | | | | | | | | | This also syncs the copyright blurb with current glibc sources. The written by line does not appear in upstream, so it should be okay to remove.
* | | udev: allow to execute longer command lineYu Watanabe2022-06-241-4/+4
|/ / | | | | | | Fixes #23607.
* | boot: Fix calls to ResetSystemJan Janssen2022-06-231-3/+4
| | | | | | | | | | gnu-efi falsely declares a return type for ResetSystem when it should be void. The spec also says that it never returns, so just assert.
* | logind: simplify codeZbigniew Jędrzejewski-Szmek2022-06-231-10/+4
| | | | | | | | Follow-up for 4885d7490b23e08d8444e5a68927ce9ce8727e5a.
* | Merge pull request #23802 from yuwata/core-watchdog-follow-upsZbigniew Jędrzejewski-Szmek2022-06-231-6/+3
|\ \ | | | | | | core: cleanups for watchdog
| * | core: close watchdog device if watchdog device is unspecified nowYu Watanabe2022-06-221-5/+3
| | | | | | | | | | | | | | | If a watchdog device was specified previously, and unspecified later and PID1 is reloaded, then we need to close the device.
| * | core: drop unnecessary freeYu Watanabe2022-06-221-1/+0
| | | | | | | | | | | | | | | | | | As reset_arguments() frees the string. Follow-up for 919ea64f69f710840c1bc93f0f7cb7c51aae45d0.
* | | logind-session-dbus: allow to set display name via dbusDavid Tardon2022-06-224-0/+54
| | | | | | | | | | | | | | | | | | | | | | | | Currently, the only way to set display name of a graphical session is to pass it to CreateSession(). But modern display managers like gdm start the display server as part of the user session, which means that the display name isn't known yet when the session is being created. Hence, let's make it possible to set it later.
* | | bootspec: Add PE file name to log messagesJan Janssen2022-06-221-18/+19
| | | | | | | | | | | | Fixes: #23783
* | | Merge pull request #23774 from yuwata/netlabel-nftset-follow-upsYu Watanabe2022-06-2231-1575/+14
|\ \ \ | | | | | | | | network, core: revert NFTSet and NetLabel features
| * | | Revert "networkd: NetLabel integration"Yu Watanabe2022-06-2214-383/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts PR #23269 and its follow-up commit. Especially, 2299b1cae32c1fb8911da0ce26efced68032f4f8 (partially), and 3cf63830acdef9d8afdc9ef1cf25aa7e85a5e4d5. The PR was merged without final approval, and has several issues: - The NetLabel for static addresses are not assigned, as labels are stored in the Address objects managed by Network, instead of Link. - If NetLabel is specified for a static address, then the address section will be invalid and the address will not be configured, - It should be implemented with Request object, - There is no test about the feature.
| * | | Revert NFTSet featureYu Watanabe2022-06-2222-1192/+7
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts PR #22587 and its follow-up commit. More specifically, 2299b1cae32c1fb8911da0ce26efced68032f4f8 (partially), e176f855278d5098d3fecc5aa24ba702147d42e0, ceb46a31a01b3d3d1d6095d857e29ea214a2776b, and 51bb9076ab8c050bebb64db5035852385accda35. The PR was merged without final approval, and has several issues: - OSS fuzz reported issues in the conf parser, - It calls synchrnous netlink call, it should not be especially in PID1, - The importance of NFTSet for CGroup and DynamicUser may be questionable, at least, there was no justification PID1 should support it. - For networkd, it should be implemented with Request object, - There is no test for the feature. Fixes #23711. Fixes #23717. Fixes #23719. Fixes #23720. Fixes #23721. Fixes #23759.
* | | Merge pull request #23806 from keszybz/udevadm-info-pagerZbigniew Jędrzejewski-Szmek2022-06-221-2/+12
|\ \ \ | |/ / |/| | Pager for udevadm info
| * | udevadm info: use pager for all output typesZbigniew Jędrzejewski-Szmek2022-06-221-1/+4
| | | | | | | | | | | | | | | Before we used the pager for --tree, but not e.g. for --export-db. I don't see any particular reason to limit this.
| * | udevadm info: implement --no-pagerZbigniew Jędrzejewski-Szmek2022-06-221-2/+9
| | |
* | | resolve: mDNS transaction max attempts fixVishal Chillara Srinivas2022-06-222-9/+29
| | | | | | | | | | | | | | | | | | | | | Maximum attempts to send mDNS requests is one except for probe requests, which should be attempted thrice. Implemented fix to account for the difference between regular queries and probe requests, and prevent even regular queries from being attempted thrice. See RFC 6762 Section 8.1
* | | core: Fix memory leaksŁukasz Stelmach2022-06-221-2/+2
| | | | | | | | | | | | | | | | | | arg_early_core_pattern and arg_watchdog_device hold pointers to memory allocated with strdup() (inside path_make_absolute_cwd). The memory needs to be freed in reset_arguments() during reload rather than forgotten.
* | | Add sys/file.h for LOCK_Pavel Zhukov2022-06-215-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | Fixes build with musl: | ../git/src/shared/dissect-image.c: In function 'mount_image_privately_interactively': | ../git/src/shared/dissect-image.c:2986:34: error: 'LOCK_SH' undeclared (first use in this function) | 2986 | r = loop_device_flock(d, LOCK_SH); | | ^~~~~~~
* | | pkgconfig,rpm: expose vars for user-tmpfiles.d locationZbigniew Jędrzejewski-Szmek2022-06-212-0/+3
| | | | | | | | | | | | Fixes https://bugzilla.redhat.com/show_bug.cgi?id=2098553.
* | | locale: drop unnecessary allocationYu Watanabe2022-06-181-4/+0
| | | | | | | | | | | | | | | | | | Fixes a bug introduced by 3d36b5d7e7b191fca7c5c65dbab94d99cf5f0230. Fixes #23777.
* | | networkctl: fix output of "status": replace "Queue Length" by "Number of Queues"Marc Kleine-Budde2022-06-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit 0307afc681e1 ("networkctl: add support to display Transmit/Recieve queue length (#12633)") added the display of the number of RX and TX Queues to the output of `networkctl status $DEV`. However the row description says "Queue Length". This patch fixes the output by replacing "Queue Length" by "Number of Queues". Fixes: 0307afc681e1 ("networkctl: add support to display Transmit/Recieve queue length (#12633)")
* | | add program name to log linePhilipp Gortan2022-06-181-1/+1
| | |
* | | Merge pull request #23756 from yuwata/analyze-seccomp-filterLuca Boccassi2022-06-176-26/+72
|\ \ \ | | | | | | | | analyze: fix seccomp filter handling in security command
| * | | seccomp-util: make @known include @obsoleteYu Watanabe2022-06-171-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | @known is generated from syscall-list.txt, which generated from kernel headers. So, some syscalls in @obsolete may not be listed in syscall-list.txt.
| * | | analyze-security: always save syscall nameYu Watanabe2022-06-171-17/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts dd51e725df9aec2847482131ef601e0215b371a0 and fixes bugs introduced by 1624114d74f55ad9791b7624b08d89d2339a68b3. Previously, - On online scan, the syscall filter was a string Hashmap, but it might contain syscall name with errno or error action. Hence, we need to drop the errno or error action in the string. - On offline scan, the syscall filter was a Hashmap of syscall ID, so hashmap_contains() with syscall name did not work. We need to convert syscall IDs to syscall names. - If hashmap_contains() in syscall_names_in_filter() is true, then the syscall is allowed when the list is an allow-list, and vice versa. Hence, the condition in syscall_names_in_filter() was errnously inverted by dd51e725df9aec2847482131ef601e0215b371a0. This makes syscalls are always stored with its name, instead of ID, and also correct the condition. Fixes #23663.
| * | | set: introduce set_put_strndup()Yu Watanabe2022-06-173-8/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Note, if `n != SIZE_MAX`, we cannot check the existence of the specified string in the set without duplicating the string. And, set_consume() also checks the existence of the string. Hence, it is not necessary to call set_contains() if `n != SIZE_MAX`.
| * | | core/dbus-execute: drop unnecessary flagYu Watanabe2022-06-171-1/+1
| | | | | | | | | | | | | | | | | | | | The code block is called only when the list was empty, and the newly requested list is allow-list. Hence, invert_flag is always zero here.
| * | | core/dbus-execute: do not append denied syscalls in allow-listYu Watanabe2022-06-171-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Follow-up for 68acc1afbe5cec50da1ffdc411dadda504e4caf5. Before the commit, SystemCallFilter bus property provides only allowed syscalls if ExecContext.syscall_filter is an allow-list, and vice versa. After the commit, if the list is allow-list, it contains allowed syscalls with value `-1`, and denied syscalls with non-negative values. To keep the backward compatibility, denied syscalls must be dropped in SystemCallFilter bus property.
* | | | meson: Reorder cflags for sd-bootJan Janssen2022-06-171-19/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, -Wall and -Wextra override previously passed flags like -Wno-unused-parameter. This reorders them to be passed before any optional flags. -Wsign-compare is part of -Wextra and therefore dropped. -nostdlib is a link-stage flag and dropped as it is already part of efi_ldflags.
* | | | nspawn: fix UID map stringYu Watanabe2022-06-161-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | We send/recv the set of payload uid, host uid, payload gid, host gid. Hence, the index must be incremented with 4, instead of 2. Fixes #23664.
* | | | RFC 6762 section 7.1: a Multicast DNS querier SHOULD NOT include records in theVishal Chillara Srinivas2022-06-161-0/+8
| | | | | | | | | | | | | | | | Known-Answer list whose remaining TTL is less than half of their original TTL
* | | | resolve: mark mDNS RRs in the Additional Records section as cacheableVishal Chillara Srinivas2022-06-161-2/+6
|/ / / | | | | | | | | | | | | RFC 6762 does not restrict caching of records in the Additional section. Caching can improve resolve efficiency.
* | | Merge pull request #23733 from sshedi/cid-1469711Yu Watanabe2022-06-152-7/+6
|\ \ \ | | | | | | | | homework: initialize we_detached
| * | | sd-journal: check retval of sd_id128_from_string callShreenidhi Shedi2022-06-141-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: CID#1469712 CID 1469712 (#1 of 1): Unused value (UNUSED_VALUE) returned_value: Assigning value from sd_id128_from_string(word + 2, &boot_id) to r here, but that stored value is overwritten before it can be used.
| * | | homework: initialize we_detachedShreenidhi Shedi2022-06-141-7/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: CID#1469711 CID 1469711 (#1 of 1): Uninitialized scalar variable (UNINIT) 14. uninit_use: Using uninitialized value we_detached.
* | | | Merge pull request #23709 from yuwata/test-hostname-localeYu Watanabe2022-06-1514-360/+444
|\ \ \ \ | |/ / / |/| | | test: import hostname and locale tests from debian/ubuntu test suite