Goffredo Baroncelli [Sun, 12 Apr 2015 18:30:28 +0000 (20:30 +0200)]
tmpfiles: Add +C attrib to the journal files directories
Add the +C file attribute (NOCOW) to the journal directories, so that
the flag is inherited automatically for new journal files created in
them. The journal write pattern is problematic on btrfs file systems as
it results in badly fragmented files when copy-on-write (COW) is used:
the performances decreases substantially over time.
To avoid this issue, this tmpfile.d snippet sets the NOCOW attribute to
the journal files directories, so newly created journal files inherit
the NCOOW attribute that disables copy-on-write.
Be aware that the NOCOW file attribute also disables btrfs checksumming
for these files, and thus prevents btrfs from rebuilding corrupted files
on a RAID filesystem.
In a single disk filesystems (or filesystems without redundancy) it is
safe to use the NOCOW flags without drawbacks, since the journal files
contain their own checksumming.
Lennart Poettering [Mon, 13 Apr 2015 13:23:42 +0000 (15:23 +0200)]
update TODO
Lennart Poettering [Mon, 13 Apr 2015 13:23:37 +0000 (15:23 +0200)]
man: slightly fewer paragraphs can help readability
Lennart Poettering [Mon, 13 Apr 2015 13:23:07 +0000 (15:23 +0200)]
man: fix examples indentation in tmpfiles.d(5)
Lennart Poettering [Mon, 13 Apr 2015 13:22:50 +0000 (15:22 +0200)]
man: add information about more lines to explanation of argument field
Lennart Poettering [Mon, 13 Apr 2015 13:18:09 +0000 (15:18 +0200)]
man: document which tmpfiles line types follow symlinks
Generally, we will not follow symlinks, except for "w".
Avoid documentation for now for fifo, device node, directory lines,
which currently follow symlinks but better shouldn't.
Lennart Poettering [Mon, 13 Apr 2015 13:16:54 +0000 (15:16 +0200)]
tmpfiles: don't follow symlinks when adjusting ACLs, fille attributes, access modes or ownership
Martin Pitt [Mon, 13 Apr 2015 11:34:14 +0000 (06:34 -0500)]
hwdb: Fix wireless switch on Dell Latitude
It does not generate a release event.
https://launchpad.net/bugs/1441849
Zbigniew Jędrzejewski-Szmek [Mon, 13 Apr 2015 02:46:37 +0000 (22:46 -0400)]
journal-gatewayd: use (void) to silence coverity
CID #996297.
Zbigniew Jędrzejewski-Szmek [Mon, 13 Apr 2015 02:43:08 +0000 (22:43 -0400)]
journal: use (void) to silence coverity
This shouldn't really fail and anyway not much we can do about it.
CID #996292, #996294, #996295.
Zbigniew Jędrzejewski-Szmek [Mon, 13 Apr 2015 02:41:20 +0000 (22:41 -0400)]
pam_system: use (void) to silence coverity
CID #996284.
Zbigniew Jędrzejewski-Szmek [Mon, 13 Apr 2015 01:47:58 +0000 (21:47 -0400)]
man: add link to glib in sd_event_run(3)
Piotr Drąg [Sat, 11 Apr 2015 23:45:11 +0000 (01:45 +0200)]
po: update Polish translation
https://bugs.freedesktop.org/show_bug.cgi?id=89989
Zbigniew Jędrzejewski-Szmek [Sun, 12 Apr 2015 20:05:10 +0000 (16:05 -0400)]
sysv-generator: free memory allocated for service stubs
Zbigniew Jędrzejewski-Szmek [Sun, 12 Apr 2015 20:04:51 +0000 (16:04 -0400)]
shared/hashmap: normalize whitespace
Zbigniew Jędrzejewski-Szmek [Sun, 12 Apr 2015 15:52:06 +0000 (11:52 -0400)]
sysv-generator: always log on oom
This code appears to follow the following convention:
- all errors are logged at point of origin
- oom errors abort execution, non-oom errors are logged but
execution continues.
Make sure all ooms result in a log message, and remove warning which could
not be reached. Downgrade non-fatal errors to warnings.
Zbigniew Jędrzejewski-Szmek [Sun, 12 Apr 2015 15:17:22 +0000 (11:17 -0400)]
sysv-generator: split out two nested blocks into functions
No functional change intended. Just splitting this out to make
it easier to edit in the future.
Lennart Poettering [Sun, 12 Apr 2015 18:15:08 +0000 (20:15 +0200)]
dhcp: yes, infiniband has the larger MAC address length, but let the compuler figure that out...
Lennart Poettering [Sun, 12 Apr 2015 18:01:07 +0000 (20:01 +0200)]
efi-boot-generator: need need to proceed if /boot is already a mount point
Tobias Hunger [Fri, 10 Apr 2015 23:52:50 +0000 (01:52 +0200)]
gpt-generator: Find device on a stateless system
A stateless system has a tmpfs as root file system. That obviously
does not have any block device associated with it. So try falling back
to the device of the /usr filesystem if the root filesystem fails.
Simon Farnsworth [Wed, 25 Mar 2015 17:00:09 +0000 (17:00 +0000)]
python-systemd: fix is_socket_inet to cope with ports
Just a couple of trivial oversights.
Tobias Hunger [Sat, 11 Apr 2015 00:13:31 +0000 (02:13 +0200)]
efi-boot-generator: Continue if /boot does not exist
/boot does not exist on a stateless system, so do not get
confused by that.
Zbigniew Jędrzejewski-Szmek [Sat, 11 Apr 2015 23:39:30 +0000 (19:39 -0400)]
bus-util: add articles to explanation messages
We are talking about one member of a group of things (resource limits, signals,
timeouts), without specifying which one. An indenfinite article is in order.
When we are talking about the control process, it's a specific one, so the
definite article is used.
Zbigniew Jędrzejewski-Szmek [Sat, 11 Apr 2015 16:00:27 +0000 (12:00 -0400)]
sysv-generator: free LookupPaths also on error
Followup for
7a03974a6f.
Zbigniew Jędrzejewski-Szmek [Sat, 11 Apr 2015 15:38:34 +0000 (11:38 -0400)]
udev-builtin-keyboard: make error messages more standard
- No need to add "Error, " prefix, we already have that as metadata.
- Also use double quotes for path names, as in most other places.
- Remove stray newline at end of message.
- Downgrade error messages after which we continue to warnings.
Zbigniew Jędrzejewski-Szmek [Sat, 11 Apr 2015 15:05:39 +0000 (11:05 -0400)]
rules: finish incomplete rename
Fixup for
51c0c2869845a058268d54c3111d55d0dd485704.
Zbigniew Jędrzejewski-Szmek [Fri, 10 Apr 2015 22:57:05 +0000 (18:57 -0400)]
tmpfiles: use qsort_safe
Thomas Hindoe Paaboel Andersen [Sat, 11 Apr 2015 11:45:46 +0000 (13:45 +0200)]
sysv-generator: fix mem leaks
David Herrmann [Fri, 10 Apr 2015 15:44:30 +0000 (17:44 +0200)]
bus: implement bus_path_{en,de}code_unique()
Whenever we provide a bus API that allows clients to create and manage
server-side objects, we need to provide a unique name for these objects.
There are two ways to provide them:
1) Let the server choose a name and return it as method reply.
2) Let the client pass its name of choice in the method arguments.
The first method is the easiest one to implement. However, it suffers from
a race condition: If a client creates an object asynchronously, it cannot
destroy that object until it received the method reply. It cannot know the
name of the new object, thus, it cannot destroy it. Furthermore, this
method enforces a round-trip. If the client _depends_ on the method call
to succeed (eg., it would close() the connection if it failed), the client
usually has no reason to wait for the method reply. Instead, the client
can immediately schedule further method calls on the newly created object
(in case the API guarantees in-order method-call handling).
The second method fixes both problems: The client passes an object name
with the method-call. The server uses it to create the object. Therefore,
the client can schedule object destruction even if the object-creation
hasn't finished, yet (again, requiring in-order method-call handling).
Furthermore, the client can schedule further method calls on the newly
created object, before the constructor returned.
There're two problems to solve, though:
1) Object names are usually defined via dbus object paths, which are
usually globally namespaced. Therefore, multiple clients must be able
to choose unique object names without interference.
2) If multiple libraries share the same bus connection, they must be
able to choose unique object names without interference.
The first problem is solved easily by prefixing a name with the
unique-bus-name of a connection. The server side must enforce this and
reject any other name.
The second problem is solved by providing unique suffixes from within
sd-bus. As long as sd-bus always returns a fresh new ID, if requested,
multiple libraries will never interfere. This implementation re-uses
bus->cookie as ID generator, which already provides unique IDs for each
bus connection.
This patch introduces two new helpers:
bus_path_encode_unique(sd_bus *bus,
const char *prefix,
const char *sender_id,
const char *external_id,
char **ret_path);
This creates a new object-path via the template
'/prefix/sender_id/external_id'. That is, it appends two new labels to
the given prefix. If 'sender_id' is NULL, it will use
bus->unique_name, if 'external_id' is NULL, it will allocate a fresh,
unique cookie from bus->cookie.
bus_path_decode_unique(const char *path,
const char *prefix,
char **ret_sender,
char **ret_external);
This reverses what bus_path_encode_unique() did. It parses 'path' from
the template '/prefix/sender/external' and returns both suffix-labels
in 'ret_sender' and 'ret_external'. In case the template does not
match, 0 is returned and both output arguments are set to NULL.
Otherwise, 1 is returned and the output arguments contain the decoded
labels.
Note: Client-side allocated IDs are inspired by the Wayland protocol
(which itself was inspired by X11). Wayland uses those IDs heavily
to avoid round-trips. Clients can create server-side objects and
send method calls without any round-trip and waiting for any object
IDs to be returned. But unlike Wayland, DBus uses gobally namespaced
object names. Therefore, we have to add the extra step by adding the
unique-name of the bus connection.
David Herrmann [Fri, 10 Apr 2015 15:43:04 +0000 (17:43 +0200)]
bus: implement bus_label_unescape_n()
This is like bus_label_unescape() but takes a maximum length instead of
relying on NULL-terminated strings. This is highly useful to unescape
labels that are not at the end of a path.
David Herrmann [Fri, 10 Apr 2015 09:15:12 +0000 (11:15 +0200)]
hashmap: return NULL from destructor
We _always_ return NULL from destructors to allow direct assignments to
the variable holding the object. Especially on hashmaps, which treat NULL
as empty hashmap, this is pretty neat.
Daniel Drake [Mon, 6 Apr 2015 22:03:43 +0000 (16:03 -0600)]
udevd: fix synchronization with settle when handling inotify events
udev uses inotify to implement a scheme where when the user closes
a writable device node, a change uevent is forcefully generated.
In the case of block devices, it actually requests a partition rescan.
This currently can't be synchronized with "udevadm settle", i.e. this
is not reliable in a script:
sfdisk --change-id /dev/sda 1 81
udevadm settle
mount /dev/sda1 /foo
The settle call doesn't synchronize there, so at the same time we try
to mount the device, udevd is busy removing the partition device nodes and
readding them again. The mount call often happens in that moment where the
partition node has been removed but not readded yet.
This exact issue was fixed long ago:
http://git.kernel.org/cgit/linux/hotplug/udev.git/commit/?id=
bb38678e3ccc02bcd970ccde3d8166a40edf92d3
but that fix is no longer valid now that sequence numbers are no longer
used.
Fix this by forcing another mainloop iteration after handling inotify events
before unblocking settle. If the inotify event caused us to generate a
"change" event, we'll pick that up in the following loop iteration, before
we reach the end of the loop where we respond to settle's control message,
unblocking it.
Gavin Li [Tue, 7 Apr 2015 10:30:10 +0000 (03:30 -0700)]
hwdb: add Samsung ATIV Book 6 / 8
This adds support for the keyboard illumination keys and fixes
Fn+F1.
Marc-Antoine Perennou [Wed, 8 Apr 2015 20:33:54 +0000 (22:33 +0200)]
build: allow setting OBJCOPY
Marc-Antoine Perennou [Wed, 8 Apr 2015 20:33:53 +0000 (22:33 +0200)]
configure: allow setting EFI_CC
Marc-Antoine Perennou [Tue, 7 Apr 2015 18:54:04 +0000 (20:54 +0200)]
efi: use EFI_CC
Marc-Antoine Perennou [Tue, 7 Apr 2015 18:54:02 +0000 (20:54 +0200)]
factory: install to datadir
Nir Soffer [Wed, 8 Apr 2015 01:04:16 +0000 (04:04 +0300)]
udev: restore udevadm settle timeout
Commit
9ea28c55a2 (udev: remove seqnum API and all assumptions about
seqnums) introduced a regresion, ignoring the timeout option when
waiting until the event queue is empty.
Previously, if the udev event queue was not empty when the timeout was
expired, udevadm settle was returning with exit code 1. To check if the
queue is empty, you could invoke udevadm settle with timeout=0. This
patch restores the previous behavior.
(David: fixed timeout==0 handling and dropped redundant assignment)
Matthew Garrett [Thu, 9 Apr 2015 01:58:09 +0000 (18:58 -0700)]
rules: fix tests for removable state
We only care about whether our direct parent is removable, not whether any
further points up the tree are - the kernel will take care of policy for
those itself. This enables autosuspend on devices where the root hub reports
that its removable state is unknown.
Łukasz Stelmach [Fri, 10 Apr 2015 17:39:17 +0000 (19:39 +0200)]
README: glibc version 2.16 is required for IP_UNICAST_IF
https://sourceware.org/git/?p=glibc.git;a=commitdiff;h=
be08eda5
https://bugs.gentoo.org/show_bug.cgi?id=546194
David Herrmann [Sat, 11 Apr 2015 10:13:24 +0000 (12:13 +0200)]
TODO: add
13790add4 as blocker for v220
Koen Kooi [Sat, 11 Apr 2015 08:23:26 +0000 (10:23 +0200)]
build: add support for AARCH64 EFI
Aarch64 and ARM32 lack an EFI capable objcopy, so use the ldflags + -O
binary trick gnu-efi and the Red Hat shimloader are using.
(David: rebase to systemd-git and added EFI_ prefixes)
Koen Kooi [Sat, 11 Apr 2015 08:23:25 +0000 (10:23 +0200)]
boot/util: add ticks_read() stub
Koen Kooi [Sat, 11 Apr 2015 08:23:24 +0000 (10:23 +0200)]
build: add AARCH64 efi support
This is just plumbing to add ARCH_AARCH64 EFI support for makefile tests
and defining the machine name.
Koen Kooi [Sat, 11 Apr 2015 08:23:23 +0000 (10:23 +0200)]
boot/util: use x86 ASM only on x86 platforms.
Koen Kooi [Sat, 11 Apr 2015 08:23:22 +0000 (10:23 +0200)]
build: support non-x86 EFI builds
Move the no-mmx/no-sse CFLAGS to X86-64 and IA32 defines in preparation
for ARM32 and Aarch64 support.
Peter Hutterer [Mon, 23 Mar 2015 00:18:42 +0000 (10:18 +1000)]
hwdb: set the resolution for a couple of bcm5974 touchpads
Verified for the 5,1 Macbook, the others are guesses based on the list of
supported devices of the moshi trackpad protector.
http://www.moshi.com/trackpad-protector-trackguard-macbook-pro#silver
Resolution calculated based on the min/max settings set in the kernel driver,
divided by the physical size. This is probably slightly off, but still better
than no resolution at all.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Peter Hutterer [Thu, 19 Mar 2015 04:19:58 +0000 (14:19 +1000)]
udev: builtin-keyboard: add support for EVDEV_ABS_*
Parse properties in the form
EVDEV_ABS_00="<min>:<max>:<res>:<fuzz>:<flat>"
and apply them to the kernel device. Future processes that open that device
will see the updated EV_ABS range.
This is particularly useful for touchpads that don't provide a resolution in
the kernel driver but can be fixed up through hwdb entries (e.g. bcm5974).
All values in the property are optional, e.g. a string of "::45" is valid to
set the resolution to 45.
The order intentionally orders resolution before fuzz and flat despite it
being the last element in the absinfo struct. The use-case for setting
fuzz/flat is almost non-existent, resolution is probably the most common case
we'll need.
To avoid multiple hwdb invocations for the same device, replace the
hwdb "keyboard:" prefix with "evdev:" and drop the separate 60-keyboard.rules
file. The new 60-evdev.rules is called for all event nodes
anyway, we don't need a separate rules file and second callout to the hwdb
builtin.
Peter Hutterer [Fri, 20 Mar 2015 04:00:31 +0000 (14:00 +1000)]
udev: builtin-keyboard: invert a condition
No functional changes, just to make the next patch easier to review
Peter Hutterer [Fri, 20 Mar 2015 03:17:20 +0000 (13:17 +1000)]
udev: builtin-keyboard: move actual key mapping to a helper function
No changes in the mapping, but previously we opened the device only on
successful parsing. Now we open the mapping as soon as we have a value that
looks interesting. Since errors are supposed to be the exception, not the
rule, this is probably fine.
Peter Hutterer [Fri, 20 Mar 2015 02:52:46 +0000 (12:52 +1000)]
udev: builtin-keyboard: immediately EVIOCSKEYCODE when we have a pair
Rather than building a map and looping through the map, immediately call the
ioctl when we have a successfully parsed property.
This has a side-effect: before the maximum number of ioctls was limited to the
size of the map (1024), now it is unlimited.
Peter Hutterer [Fri, 20 Mar 2015 02:48:24 +0000 (12:48 +1000)]
udev: builtin-keyboard: move fetching the device node up
No point parsing the properties if we can't get the devnode to apply them
later. Plus, this makes future additions easier to slot in.
Ronny Chevalier [Fri, 10 Apr 2015 22:25:43 +0000 (00:25 +0200)]
shared: move replace_env* from util to env-util
Ronny Chevalier [Fri, 10 Apr 2015 21:15:59 +0000 (23:15 +0200)]
shared: add terminal-util.[ch]
Ronny Chevalier [Fri, 10 Apr 2015 20:27:10 +0000 (22:27 +0200)]
shared: add random-util.[ch]
Ronny Chevalier [Fri, 10 Apr 2015 17:10:00 +0000 (19:10 +0200)]
shared: add process-util.[ch]
Ronny Chevalier [Fri, 10 Apr 2015 18:43:52 +0000 (20:43 +0200)]
shared: add formats-util.h
Thomas Hindoe Paaboel Andersen [Fri, 10 Apr 2015 21:39:13 +0000 (23:39 +0200)]
dbus: typo fix in log
Thomas Hindoe Paaboel Andersen [Fri, 10 Apr 2015 21:26:21 +0000 (23:26 +0200)]
tmpfiles: fix build with clang
Clang is not happy about using the cleanup attribute in switches
Thomas Hindoe Paaboel Andersen [Fri, 10 Apr 2015 20:58:40 +0000 (22:58 +0200)]
efivars: fix build for non-efi
Tom Gundersen [Fri, 10 Apr 2015 17:49:36 +0000 (19:49 +0200)]
shared: boot-timestamps - remove ifdef
No need to ifdef out efi code as the functions are always defined.
Tom Gundersen [Fri, 10 Apr 2015 17:44:06 +0000 (19:44 +0200)]
shared: efivars - fix compile on non-EFI systems
systemctl and logind were unconditionally using functions that were not compiled
on non-EFI systems. Add stubs returning -EOPNOTSUPP to fix compile again.
Tom Gundersen [Fri, 10 Apr 2015 17:43:36 +0000 (19:43 +0200)]
shared: efivars - is_efi_*() returns bool instead of int
There was a bug where is_efi_*() could return a negative error value, which would be treated as 'true',
just make this a bool in the helper library to avoid the problem.
Lennart Poettering [Fri, 10 Apr 2015 17:04:43 +0000 (19:04 +0200)]
Revert "sd-dhcp-client: fix strict aliasing issue"
This reverts commit
6ec8e7c763b7dfa82e25e31f6938122748d1608f.
This doesn't fix any issues, just makes the code harder to read.
Lennart Poettering [Fri, 10 Apr 2015 16:45:39 +0000 (18:45 +0200)]
sd-event: simplify sd_event_run()
Lennart Poettering [Fri, 10 Apr 2015 16:07:04 +0000 (18:07 +0200)]
tmpfiles: add specifier expansion for L and C lines, too
Lennart Poettering [Fri, 10 Apr 2015 15:35:55 +0000 (17:35 +0200)]
update TODO
Michal Sekletar [Fri, 10 Apr 2015 13:56:52 +0000 (15:56 +0200)]
bus-util: be more verbose if dbus job fails
Users might have hard time figuring out why exactly their systemctl request
failed. If dbus job fails try to figure out more details about failure by
examining Result property of the service.
https://bugzilla.redhat.com/show_bug.cgi?id=1016680
Ronny Chevalier [Fri, 10 Apr 2015 14:53:17 +0000 (16:53 +0200)]
core: set_put never returns -EEXIST
When the value is already there it returns 0.
Also add a test to ensure this
Lennart Poettering [Fri, 10 Apr 2015 14:22:22 +0000 (16:22 +0200)]
tmpfiles: enforce ordering when executing lines
Always create files first, and then adjust their ACLs, xattrs, file
attributes, never the opposite. Previously the order was not
deterministic, thus possibly first adjusting ACLs/xattrs/file
attributes before actually creating the items.
Lennart Poettering [Fri, 10 Apr 2015 14:04:16 +0000 (16:04 +0200)]
tmpfiles: eat up empty columns
Lennart Poettering [Fri, 10 Apr 2015 14:03:24 +0000 (16:03 +0200)]
tmpfiles: substitute % specifiers in arguments for writing files and xattrs
Lennart Poettering [Fri, 10 Apr 2015 12:46:05 +0000 (14:46 +0200)]
tmpfiles: warn if we get an argument on lines that don't take any
Lennart Poettering [Fri, 10 Apr 2015 12:44:52 +0000 (14:44 +0200)]
tmpfiles: mostly revert
71044f609b829d802e0eb81270e13b4f55d76476
Add a comment why returning a positive error is OK and intended in this
case.
(It's still a nasty hack to do this though!)
Lennart Poettering [Fri, 10 Apr 2015 12:43:37 +0000 (14:43 +0200)]
device-nodes: minor simplifications
Lennart Poettering [Fri, 10 Apr 2015 12:43:06 +0000 (14:43 +0200)]
util: unify how we parse mode_t strings
Lennart Poettering [Fri, 10 Apr 2015 10:21:44 +0000 (12:21 +0200)]
util: remove normalize_env_assignment(), it's unused
Ronny Chevalier [Fri, 10 Apr 2015 13:44:02 +0000 (15:44 +0200)]
shared: fix memleak
path was used for 2 purposes but it was not freed before being reused.
Lennart Poettering [Fri, 10 Apr 2015 10:16:54 +0000 (12:16 +0200)]
util: mark unquote() static, so that new code doesn't use this anymore
Lennart Poettering [Fri, 10 Apr 2015 09:56:04 +0000 (11:56 +0200)]
util: fix unicode decoding in unquote_first_word()
Lennart Poettering [Fri, 10 Apr 2015 09:27:47 +0000 (11:27 +0200)]
util: when unescaping C escape sequences support C++11 \u and \U unicode literals
We simply recode them in utf8.
Thomas Hindoe Paaboel Andersen [Thu, 9 Apr 2015 21:57:35 +0000 (23:57 +0200)]
util: fix a typo in comment
David Herrmann [Thu, 9 Apr 2015 20:19:26 +0000 (22:19 +0200)]
terminal/page: drop unneeded DISABLE_WARNING_*
The macros in shared/macro.h have been updated to no longer use fixed
variable names. This will silence any shadowing-warnings, so drop the
DISABLE_WARNING_* annotations.
David Herrmann [Wed, 8 Apr 2015 09:20:10 +0000 (11:20 +0200)]
bus: simplify bus_message_read_strv_extend()
Avoid unbound for(;;) loop and use the established coding-style:
while ((r = sd_bus_message_read*(...)) > 0) {
}
if (r < 0)
return r;
This is much easier to read and used all over the code base.
David Herrmann [Tue, 7 Apr 2015 13:20:54 +0000 (15:20 +0200)]
bus: use free_and_strdup()
Save some LOCs by replacing strdup()+error-handling+free+assign with
free_and_strdup().
David Herrmann [Tue, 7 Apr 2015 13:06:01 +0000 (15:06 +0200)]
terminal/idev: fix NULL terminator for bus-property mapping
We require a NULL entry as sentinal in bus_properties_map. Fix this!
Ronny Chevalier [Thu, 9 Apr 2015 09:23:53 +0000 (11:23 +0200)]
test-journal-flush: fix memleak
Ronny Chevalier [Thu, 9 Apr 2015 09:00:35 +0000 (11:00 +0200)]
util: cunescape - fix memleak in error path
Lennart Poettering [Thu, 9 Apr 2015 16:32:21 +0000 (18:32 +0200)]
util: add shell_maybe_quote() call for preparing a string for shell cmdline inclusion
If necessary the passed string is enclosed in "", and all special
characters escapes.
This also ports over usage in bus-util.c and job.c to use this, instead
of a incorrect local implementation that forgets to properly escape.
Lennart Poettering [Thu, 9 Apr 2015 11:12:55 +0000 (13:12 +0200)]
vconsole: some modernizations, indentation fixes
Lennart Poettering [Thu, 9 Apr 2015 11:12:26 +0000 (13:12 +0200)]
tmpfiles: properly return error code from path_set_acl()
Lennart Poettering [Thu, 9 Apr 2015 11:12:06 +0000 (13:12 +0200)]
tmpfiles: add file attribute calls to list of glob needing commands
Lennart Poettering [Thu, 9 Apr 2015 11:11:38 +0000 (13:11 +0200)]
update TODO
Lennart Poettering [Thu, 9 Apr 2015 11:10:11 +0000 (13:10 +0200)]
vconsole: stick to kernel-style function error handling
This undoes part of
8931278c8a9ef7b60f6b6963486a9e9febb20a0d.
We really should stick to kernel-style "int" return values from
functions, and not covnert success into boolean returns.
Blaž Tomažič [Fri, 3 Apr 2015 20:20:55 +0000 (22:20 +0200)]
hwdb: mouse DPI data, Roccat Lua (ROC-11-310)
https://bugs.freedesktop.org/show_bug.cgi?id=89743
Thomas Hindoe Paaboel Andersen [Wed, 8 Apr 2015 22:05:22 +0000 (00:05 +0200)]
networkd: do not assign the variable twice
Lennart Poettering [Wed, 8 Apr 2015 20:39:40 +0000 (22:39 +0200)]
update TODO
Lennart Poettering [Wed, 8 Apr 2015 20:35:52 +0000 (22:35 +0200)]
tmpfiles: rework file attribute code
- Stick to one type for the flags field: unsigned. This appears to be
what the kernel uses, and there's no point in using something else.
- compress the flags array by avoiding sparse entries
- extend some error messages to not use abbreviated words
- avoid TTOCTTOU issues by invoking fstat() after open() when applying
file flags
- add explanation why we need to check the file type with fstat().
- don't needlessly abbreviate "attribute" as "attrib", in particually as
"chattr" abbreviates it as "attr" rather than "attrib".
Lennart Poettering [Wed, 8 Apr 2015 18:47:35 +0000 (20:47 +0200)]
util: merge change_attr_fd() and chattr_fd()
Lennart Poettering [Wed, 8 Apr 2015 18:27:11 +0000 (20:27 +0200)]
fstab-generator: don't continue on OOM