Lennart Poettering [Mon, 16 Dec 2013 16:05:51 +0000 (17:05 +0100)]
bus: increase the bus socket buffer to 8 MB similar, to the log socket buffers
Lennart Poettering [Mon, 16 Dec 2013 16:05:42 +0000 (17:05 +0100)]
update TODO
Lennart Poettering [Mon, 16 Dec 2013 16:04:36 +0000 (17:04 +0100)]
util: try harder to increase the send/recv buffers of sockets
If we have the priviliges we will try SO_SNDBUFFORCE/SO_RCVBUFFORCE and
only fall back to SO_SNDBUF/SO_RCVBUF if that fails.
Zbigniew Jędrzejewski-Szmek [Mon, 16 Dec 2013 02:07:47 +0000 (21:07 -0500)]
catalog: fix language detection
Detection would fail if language was not specified in the filename
but a dot appeared somewhere higher in the path.
Lennart Poettering [Mon, 16 Dec 2013 03:59:31 +0000 (04:59 +0100)]
core: refuse doing %h, %s, %U specifier resolving in PID 1
These specifiers require NSS lookups to work, and we really shouldn't do
them from PID 1 hence. With this change they are now only supported for
user systemd instance, or when the configured user for a unit is root.
Lennart Poettering [Mon, 16 Dec 2013 03:43:42 +0000 (04:43 +0100)]
build-sys: warn if builds are not byte-by-byte reproducible due to usage of __DATE__ and suchlike
Colin Walters [Mon, 16 Dec 2013 01:23:23 +0000 (20:23 -0500)]
Fix segv introduced by
2fd069b18e525860514a70d3ea08410ca122d3e2
n->path is pointing to the value now, we set s = NULL above.
Lennart Poettering [Sun, 15 Dec 2013 23:04:40 +0000 (00:04 +0100)]
build-sys: add cppcheck target
This uses --enable=all mode. Should be taken with a grain of salt
though. While many recommendations make sense we should probably keep
"int r" always on function scope, and many of the portability warnings
really don't matter to us because we only care for Linux/glibc.
Zbigniew Jędrzejewski-Szmek [Sun, 15 Dec 2013 21:26:27 +0000 (16:26 -0500)]
Fix a few signed/unsigned format string issues
Since numbers involved are all small, behaviour was correct already.
https://bugzilla.redhat.com/show_bug.cgi?id=1043304
Zbigniew Jędrzejewski-Szmek [Sun, 15 Dec 2013 21:25:04 +0000 (16:25 -0500)]
Fix a few resource leaks in error paths
https://bugzilla.redhat.com/show_bug.cgi?id=1043304
Lennart Poettering [Sun, 15 Dec 2013 22:43:47 +0000 (23:43 +0100)]
bus: _printf_ attributes should be on prototypes not function definitions for non-static functions
Kay Sievers [Sun, 15 Dec 2013 15:56:28 +0000 (16:56 +0100)]
bus: update kdbus monitoring interface
Zbigniew Jędrzejewski-Szmek [Sun, 15 Dec 2013 04:09:14 +0000 (23:09 -0500)]
udev-builtin-path: fix printf specifiers
Zbigniew Jędrzejewski-Szmek [Sun, 15 Dec 2013 03:18:49 +0000 (22:18 -0500)]
bus: return negative errno on error
https://bugs.freedesktop.org/show_bug.cgi?id=72688
Thomas Hindoe Paaboel Andersen [Sun, 15 Dec 2013 01:29:38 +0000 (02:29 +0100)]
test-journal-syslog: use streq_ptr since we pass in NULL
Djalal Harouni [Sat, 14 Dec 2013 23:05:38 +0000 (00:05 +0100)]
systemctl: add the --plain option to the help message
Zbigniew Jędrzejewski-Szmek [Sat, 14 Dec 2013 22:30:25 +0000 (17:30 -0500)]
man: remove advice to avoid setting the same var more than once
So far the compatibility with .desktop settings hasn't been imporant
at all, and we do not want people to write convoluted unit
files.
Zbigniew Jędrzejewski-Szmek [Sat, 14 Dec 2013 22:21:55 +0000 (17:21 -0500)]
man: beef up ExecStart description
We have lots of questions from people who assume that shell syntax works
here, so let's be very explicit what is allowed and what is not. A few
examples should also help.
http://bugs.debian.org/732156
Shawn Landden [Sat, 14 Dec 2013 17:27:44 +0000 (09:27 -0800)]
build: use -ftrapv for development
We want to find these bugs if they exist.
Thomas Hindoe Paaboel Andersen [Sat, 14 Dec 2013 12:09:07 +0000 (13:09 +0100)]
Add more _printf_'s for format-nonliterals
Clang is a bit more strict wrt format-nonliterals:
http://clang.llvm.org/docs/LanguageExtensions.html#format-string-checking
Adding these extra printf attributes also makes gcc able to find more
problems. E.g. this patch uncovers a format issue in udev-builtin-path_id.c
Some parts looked intetional about breaking the format-nonliteral check.
I added some supression for warnings there.
Lennart Poettering [Sat, 14 Dec 2013 04:08:47 +0000 (05:08 +0100)]
event: some snake-oil to speed up impossible error checks
Lennart Poettering [Sat, 14 Dec 2013 04:08:15 +0000 (05:08 +0100)]
event: instead of reset the revents field when we get new revents data from epoll, OR it in
Lennart Poettering [Sat, 14 Dec 2013 04:07:13 +0000 (05:07 +0100)]
event: dynamically adjust size of events array instead of pre-allocating it possibly too large or too small
Lennart Poettering [Sat, 14 Dec 2013 04:06:40 +0000 (05:06 +0100)]
util: port last code over to new namespace utility calls
Lennart Poettering [Sat, 14 Dec 2013 04:04:49 +0000 (05:04 +0100)]
util: when joining a namespace make sure to reset all uids to 0 after
the transition
Lennart Poettering [Fri, 13 Dec 2013 21:02:47 +0000 (22:02 +0100)]
bus: when connecting to a container's kdbus instance, enter namespace first
Previously we'd open the connection in the originating namespace, which
meant most peers of the bus would not be able to make sense of the
PID/UID/... identity of us since we didn't exist in the namespace they
run in. However they require this identity for privilege decisions,
hence disallowing access to anything from the host.
Instead, when connecting to a container, create a temporary subprocess,
make it join the container's namespace and then connect from there to
the kdbus instance. This is similar to how we do it for socket
conections already.
THis also unifies the namespacing code used by machinectl and the bus
APIs.
Thomas Hindoe Paaboel Andersen [Fri, 13 Dec 2013 22:21:35 +0000 (23:21 +0100)]
getty-generator: fix stripping /dev/
Lennart Poettering [Fri, 13 Dec 2013 19:29:35 +0000 (20:29 +0100)]
bus: install systemd-bus-proxyd unit files for compatibility with dbus1
Lennart Poettering [Fri, 13 Dec 2013 19:36:02 +0000 (20:36 +0100)]
getty-generator: fix segfault when $container_ttys is not set
Lennart Poettering [Fri, 13 Dec 2013 16:39:52 +0000 (17:39 +0100)]
event: make gcc shut up
Lennart Poettering [Fri, 13 Dec 2013 16:33:20 +0000 (17:33 +0100)]
journal: downgrade comments about sd_journal_print_with_location()
Effectviely these calls are ABI anyway, so downgrade the comments a bit,
since it might actually be useful for language bindings to make use of
them.
Lennart Poettering [Fri, 13 Dec 2013 16:16:06 +0000 (17:16 +0100)]
update TODO
Lennart Poettering [Fri, 13 Dec 2013 16:15:05 +0000 (17:15 +0100)]
man: document that systemd-getty-generator can start additional container gettys on ptys
Lennart Poettering [Fri, 13 Dec 2013 15:37:58 +0000 (16:37 +0100)]
getty-generator: look add an environment variable $container_ttys set for PID 1 and start gettys on all ttys listed therein
Lennart Poettering [Fri, 13 Dec 2013 15:37:16 +0000 (16:37 +0100)]
nspawn: add new --setenv= switch to set an environment variable for the container to spawn
Lennart Poettering [Fri, 13 Dec 2013 15:01:54 +0000 (16:01 +0100)]
man: document more error codes for sd_bus_request_name()
Lukasz Skalski [Fri, 13 Dec 2013 11:12:24 +0000 (12:12 +0100)]
libsystemd-bus: the same error codes for sd_bus_release_name() (for kdbus and dbus1)
Due to this patch, sd_bus_release_name() function
returns the same code errors for kdbus and dbus1
if we try release non-existing name or foreign
name.
Lukasz Skalski [Fri, 13 Dec 2013 09:26:55 +0000 (10:26 +0100)]
build-sys: fix help text for --enable-kdbus
Lennart Poettering [Fri, 13 Dec 2013 04:14:47 +0000 (05:14 +0100)]
update TODO
Lennart Poettering [Fri, 13 Dec 2013 04:13:31 +0000 (05:13 +0100)]
bus: when a connection terminated use ECONNRESET as error
Lennart Poettering [Fri, 13 Dec 2013 04:12:46 +0000 (05:12 +0100)]
bus: rework how we attach fds to event loops
When trying a couple of addresses one after the other, make sure to
reattach the fds to the event loop.
Lennart Poettering [Fri, 13 Dec 2013 04:13:59 +0000 (05:13 +0100)]
event: add ability to change fd of an active event source
Lennart Poettering [Fri, 13 Dec 2013 03:14:25 +0000 (04:14 +0100)]
event: allow to query userdata and watchdog state
Zbigniew Jędrzejewski-Szmek [Fri, 13 Dec 2013 04:14:48 +0000 (23:14 -0500)]
man: add another nspawn example
Taken from https://bugs.freedesktop.org/show_bug.cgi?id=68369.
Lennart Poettering [Fri, 13 Dec 2013 03:03:30 +0000 (04:03 +0100)]
event: when unreffing an event source from its own handler, detach fd from epoll
The pattern of unreffing an IO event source and then closing its fd is
frequently seen in even source callbacks. Previously this likely
resultet in us removing the fd from the epoll after it was closed which
is problematic, since while we were dispatching we always kept an extra
reference to event source objects because we might still need it later.
Lennart Poettering [Fri, 13 Dec 2013 02:30:42 +0000 (03:30 +0100)]
event: be more conservative when returning errors from event handler callbacks
We really should return errors from event handlers if we have a
continous problem and don't know any other solution.
Lennart Poettering [Fri, 13 Dec 2013 01:43:35 +0000 (02:43 +0100)]
git: update .gitignore
Lennart Poettering [Fri, 13 Dec 2013 01:05:04 +0000 (02:05 +0100)]
namespace: include boot id in private tmp directories
This way it is easy to only exclude directories from the current boot
from automatic clean up in /var/tmp.
Also, pick a longer name for the directories so that are globs in
tmp.conf can be simpler yet equally accurate.
Lennart Poettering [Thu, 12 Dec 2013 21:21:25 +0000 (22:21 +0100)]
event: rework sd-event exit logic
With this change a failing event source handler will not cause the
entire event loop to fail. Instead, we just disable the specific event
source, log a message at debug level and go on.
This also introduces a new concept of "exit code" which can be stored in
the event loop and is returned by sd_event_loop(). We also rename "quit"
to "exit" everywhere else.
Altogether this should make things more robus and keep errors local
while still providing a way to return event loop errors in a clear way.
Lennart Poettering [Thu, 12 Dec 2013 21:18:09 +0000 (22:18 +0100)]
bus: fix make check
Lennart Poettering [Thu, 12 Dec 2013 21:17:38 +0000 (22:17 +0100)]
bus: properly parse NameOwnerChanged messages when caller explicitly wants to match against names coming/going
Lennart Poettering [Thu, 12 Dec 2013 21:01:40 +0000 (22:01 +0100)]
man: update sd_bus_request_name() man page
Lennart Poettering [Thu, 12 Dec 2013 20:25:47 +0000 (21:25 +0100)]
bus: make sure exit-on-idle logic works on kdbus systems that do not generate NameLost
Lennart Poettering [Thu, 12 Dec 2013 20:25:31 +0000 (21:25 +0100)]
bus: fix parsing of matches against empty strings
Thomas Hindoe Paaboel Andersen [Thu, 12 Dec 2013 22:08:47 +0000 (23:08 +0100)]
scan-build: silence some warnings
test-fileio/test-strv:
Use the streq_ptr to make build-scan not worry about passing in a null
to a nonnull function.
test-dhcp-option:
Prevent a theoretical null pointer dereference
Lennart Poettering [Thu, 12 Dec 2013 19:43:37 +0000 (20:43 +0100)]
bus: fix rewind logic
Lennart Poettering [Thu, 12 Dec 2013 19:16:06 +0000 (20:16 +0100)]
service: process watchdog timeouts with lowest priority
Kay Sievers [Thu, 12 Dec 2013 19:26:48 +0000 (20:26 +0100)]
bus: remove explicit activator-specific flags, the kdbus supports it now
Kay Sievers [Thu, 12 Dec 2013 19:19:12 +0000 (20:19 +0100)]
bus: update kdbus.h
Lennart Poettering [Thu, 12 Dec 2013 19:00:19 +0000 (20:00 +0100)]
bus: always pass valid timeout to kdbus
Lennart Poettering [Thu, 12 Dec 2013 18:58:46 +0000 (19:58 +0100)]
bus: enforce endianess and marshalling for messages we send
Shawn Landden [Thu, 12 Dec 2013 18:00:03 +0000 (10:00 -0800)]
timedatectl: work with old timedated
Which does have TimeUSec. Should we specifically check for this method
instead of assuming time=0 means it doesn't exist?
Before:
shawn@debian-T61:~/git/systemd$ ./timedatectl
Local time: Wed 1969-12-31 16:00:00 PST
Universal time: Thu 1970-01-01 00:00:00 UTC
RTC time: n/a
Timezone: America/Los_Angeles (PST, -0800)
NTP enabled: n/a
NTP synchronized: no
RTC in local TZ: no
DST active: no
Last DST change: DST ended at
Sun 1969-10-26 01:59:59 PDT
Sun 1969-10-26 01:00:00 PST
Next DST change: DST begins (the clock jumps one hour forward) at
Sun 1970-04-26 01:59:59 PST
Sun 1970-04-26 03:00:00 PDT
After:
shawn@debian-T61:~/git/systemd$ ./timedatectl
Local time: Wed 2013-12-11 14:03:21 PST
Universal time: Wed 2013-12-11 22:03:21 UTC
RTC time: n/a
Timezone: America/Los_Angeles (PST, -0800)
NTP enabled: n/a
NTP synchronized: no
RTC in local TZ: no
DST active: no
Last DST change: DST ended at
Sun 2013-11-03 01:59:59 PDT
Sun 2013-11-03 01:00:00 PST
Next DST change: DST begins (the clock jumps one hour forward) at
Sun 2014-03-09 01:59:59 PST
Sun 2014-03-09 03:00:00 PDT
Zbigniew Jędrzejewski-Szmek [Thu, 12 Dec 2013 16:42:45 +0000 (11:42 -0500)]
build-sys: use internal library for dhcp client tests
Also clean up AM_CFLAGS in a few places.
Patrik Flykt [Mon, 9 Dec 2013 21:43:32 +0000 (23:43 +0200)]
dhcp: Add function to free DHCP client data
Patrik Flykt [Mon, 9 Dec 2013 21:43:31 +0000 (23:43 +0200)]
dhcp: Add notification callback
Define a notification callback and events for stopping and client
lease expiry. Add functions to fetch IP parameters from a lease.
Patrik Flykt [Mon, 9 Dec 2013 21:43:30 +0000 (23:43 +0200)]
dhcp: Compute expire, T1 and T2 timers
Compute the default T1 and T2 timer values if they were not set by
the DHCP server. Verify that the values are reasonable.
Patrik Flykt [Mon, 9 Dec 2013 21:43:29 +0000 (23:43 +0200)]
dhcp: Process DHCP Ack/Nak message
Process a DHCP Ack/Nak in much the same way as an DHCP Offer. Factor
out header verification and process options sent. Add notification
functionality with discrete values for the outcome of the DHCP Ack/
Nak processing.
Patrik Flykt [Mon, 9 Dec 2013 21:43:28 +0000 (23:43 +0200)]
dhcp: Add maximum message size option
Add maximum message size option to keep some DHCP server implementations
from sending too big messages. See ConnMan commit
0c5c862749c05193cf4c513628328c6db02b5222.
Patrik Flykt [Mon, 9 Dec 2013 21:43:27 +0000 (23:43 +0200)]
dhcp: Send DHCP Request to acquire an IP address
Create and send a DHCP Request message reusing already existing parts
of the code. This causes factoring out IP and UDP header creation and
moving next timeout calculation to be done every time in the timer
callback function independent of DHCP state. Also add an exponential
part to the timer calculation, bail out if there are errors while
resending the DHCP message for the sixth or more times.
Patrik Flykt [Mon, 9 Dec 2013 21:43:26 +0000 (23:43 +0200)]
dhcp: Handle received DHCP Offer message
Create a function for handling the full IP, UDP and DHCP packet
and tie it to the main loop. Verify IP and UDP headers and checksum.
Creat a new lease structure with using the values supplied in the
DHCP message. Free the lease structure when client is stopped.
Split out socket handling into a creation and a sending part. As a
result modify the test code.
Patrik Flykt [Mon, 9 Dec 2013 21:43:25 +0000 (23:43 +0200)]
dhcp: Add timeout and main loop support
Require a main loop to be set when creating a DHCP client. Set up
a timer to resend DHCP Discover messages and add a 0-2 second
delay to the timeout value. Move to state Selecting after successful
sending of a Discover message.
Patrik Flykt [Mon, 9 Dec 2013 21:43:24 +0000 (23:43 +0200)]
build: Add dependency on libsystemd-bus needed for main loop
Patrik Flykt [Mon, 9 Dec 2013 21:43:23 +0000 (23:43 +0200)]
dhcp: Add function to stop the DHCP client
The client is stopped and brought back to its initial state.
Patrik Flykt [Mon, 9 Dec 2013 21:43:22 +0000 (23:43 +0200)]
dhcp: Support seconds elapsed since start of DHCP negotiation
It was noticed by Grant Erickson in ConnMan commit
95e15c09350acf58d4707056ae2614570883ef66 that:
"Certain DHCP servers, such as that implemented in Mac OS X
(< 10.7) for its "Internet Sharing" feature, refuse to issue
a DHCP lease to clients that have not set a non-zero value
in their DISCOVER or REQUEST packets."
Patrik Flykt [Mon, 9 Dec 2013 21:43:21 +0000 (23:43 +0200)]
dhcp: Add test for discover DHCP packet creation
Set a fake MAC address and emulate raw packet sending. When the buffer
containing the Discover message is received, check selected IP and
UDP headers and compute IP header and UDP message checksums. Also
send the DHCP message for option parsing and expect a successful
outcome.
Patrik Flykt [Mon, 9 Dec 2013 21:43:20 +0000 (23:43 +0200)]
build: Add libsystemd-dhcp
Patrik Flykt [Mon, 9 Dec 2013 21:43:19 +0000 (23:43 +0200)]
dhcp: Add DHCP discover sending
On starting the client, use the supplied interface mac address and create
a transaction id. Puzzle together an IP/UDP/DHCP Discover message, compute
checksums and send it out as a raw packet.
Create an additional function that constructs default options common to
all DHCP messages.
Set the DHCP Client ID option as noticed by Grant Erickson in ConnMan
commit
b18d9798b3a0ae46ed87d6d2be8d5a474bf3ab1e:
"Some Internet gateways and Wi-Fi access points are unhappy when the
DHCPv4 client-id option (61) is missing and will refuse to issue a
DHCP lease."
Patrik Flykt [Mon, 9 Dec 2013 21:43:18 +0000 (23:43 +0200)]
dhcp: Add function for sending a raw packet
Open a packet socket, create a link level header, send packet and
close socket. Adding it to a separate file makes testing of the
DHCP sending much easier, as the test program can supply any socket
to the DHCP client code.
Patrik Flykt [Mon, 9 Dec 2013 21:43:17 +0000 (23:43 +0200)]
shared: Add struct sockaddr_ll to sockaddr_union
Patrik Flykt [Mon, 9 Dec 2013 21:43:16 +0000 (23:43 +0200)]
dhcp: Add test function for computing checksum
Patrik Flykt [Mon, 9 Dec 2013 21:43:15 +0000 (23:43 +0200)]
dhcp: Add option append tests
Add checks for invalid lengths and parameters when using the option
appending function. Add also checks for adding options, see to it
that the resulting array is identical to the array of options added.
Patrik Flykt [Mon, 9 Dec 2013 21:43:14 +0000 (23:43 +0200)]
dhcp: Add tests for DHCP options, file and sname fields
Add a structure describing the DHCP file, sname and trailing options
fields. Create a messge holding these fields and call the internal
option parsing function.
In the test callback function verify that only regular options are
passed and figure out which part of the DHCP message is the one that
is being processed. As the test program knows the full contents of
the test options in the test structure, skip all non-regular fields
and verify that the option provided to the callback indeed is the
one expected. Check also if non-regular option fields are to be
ignored in the end of the option field as the callback is not called
again and the final check when the whole message has been processed
needs to be successful.
Add a boolean flag for pretty-printing, anticipate there will be a
nice option to toggle it in the future.
Patrik Flykt [Mon, 9 Dec 2013 21:43:13 +0000 (23:43 +0200)]
build: Add DHCP option test
Patrik Flykt [Mon, 9 Dec 2013 21:43:12 +0000 (23:43 +0200)]
dhcp: Add buffer length and invalid cookie tests for DHCP options
Create an initial simple test program for these two cases.
Patrik Flykt [Mon, 9 Dec 2013 21:43:11 +0000 (23:43 +0200)]
dhcp: Add option appending and parsing
Add functions to append and parse DHCP options. Not all options
are passed to the callback function, the ones not exposed are
pad, end, message type and overload. If indicated by the overload
option, file and sname fields will be examined for more options.
The option functions are internal to DHCP, add a new header files
for interal function prototypes.
Patrik Flykt [Mon, 9 Dec 2013 21:43:10 +0000 (23:43 +0200)]
build: Add initial build support
The client test program is the only one to be built so far.
Patrik Flykt [Mon, 9 Dec 2013 21:43:09 +0000 (23:43 +0200)]
dhcp: Add test for DHCP client initialization and parameter setting
Patrik Flykt [Mon, 9 Dec 2013 21:43:08 +0000 (23:43 +0200)]
dhcp: Add DHCP client initialization
Provide functionality for initializing a DHCP client struct, setting
interface index, last used address and additional options to request.
On initialization the most useful options are added by default.
Patrik Flykt [Mon, 9 Dec 2013 21:43:07 +0000 (23:43 +0200)]
dhcp: Add DHCP protocol structures and initial defines
Create a new directory to host DHCP components.
Lennart Poettering [Thu, 12 Dec 2013 14:58:49 +0000 (15:58 +0100)]
log: protect errno when we use assert_return()
Lennart Poettering [Thu, 12 Dec 2013 14:58:31 +0000 (15:58 +0100)]
core: suppress gcc warnings on selinux-less systems
Lennart Poettering [Thu, 12 Dec 2013 14:58:13 +0000 (15:58 +0100)]
update TODO
Marc-Antoine Perennou [Thu, 12 Dec 2013 04:56:13 +0000 (13:56 +0900)]
shared: add missing include
Needed for socketpair, recv
Kay Sievers [Thu, 12 Dec 2013 05:23:38 +0000 (06:23 +0100)]
bus: add SD_BUS_NAME_REPLACE_EXISTING to all activatable services, fix one flags conversion
Lennart Poettering [Thu, 12 Dec 2013 04:55:58 +0000 (05:55 +0100)]
bus: instead of exposing the dbus1 flags when acquiring a name use our own that are closer to kdbus
This turns around DO_NOT_QUEUE into QUEUE which implies a more useful
default. (And negative options are awful anyway.)
Lennart Poettering [Thu, 12 Dec 2013 04:29:56 +0000 (05:29 +0100)]
bus: properly return an error when we detect a method call timeout
Lennart Poettering [Thu, 12 Dec 2013 04:23:22 +0000 (05:23 +0100)]
core: fix bus registration on daemon reexec
Zbigniew Jędrzejewski-Szmek [Thu, 12 Dec 2013 03:00:33 +0000 (22:00 -0500)]
nspawn: complain and continue if machine has same id
If --link-journal=host or --link-journal=guest is used, this totally
cannot work and we exit with an error. If however --link-journal=auto
or --link-journal=no is used, just display a warning.
Having the same machine id can happen if booting from the same
filesystem as the host. Since other things mostly function correctly,
let's allow that.
https://bugs.freedesktop.org/show_bug.cgi?id=68369
Zbigniew Jędrzejewski-Szmek [Thu, 12 Dec 2013 01:34:19 +0000 (20:34 -0500)]
Do not log all assert_return failures
% build/journalctl help
Assertion 'match_is_valid(data, size)' failed at ../src/journal/sd-journal.c:227, function sd_journal_add_match(). Ignoring.
Callers cannot be expect to check all arguments always.
Djalal Harouni [Wed, 11 Dec 2013 23:22:48 +0000 (00:22 +0100)]
journal: pipe journalctl help output into a pager
journalctl help output might run off the screen, so be consistent
as other systemd tools do and pipe it into a pager.